Re: [PATCH v3 1/2] block: Add iocontext priority to request

2016-10-12 Thread Jens Axboe
On 10/12/2016 11:58 AM, Adam Manzanares wrote: The 10/12/2016 08:49, Jens Axboe wrote: On 10/11/2016 02:40 PM, Adam Manzanares wrote: Patch adds an association between iocontext ioprio and the ioprio of a request. This feature is only enabled if a queue flag is set to indicate that requests

Re: [PATCH] badblocks: fix overlapping check for clearing

2016-10-12 Thread Tomasz Majchrzak
On Mon, Oct 10, 2016 at 03:32:58PM -0700, Dan Williams wrote: > > On Tue, Sep 06 2016, Tomasz Majchrzak wrote: > >> --- > >> block/badblocks.c | 6 -- > >> 1 file changed, 4 insertions(+), 2 deletions(-) > >> > >> diff --git a/block/badblocks.c b/block/badblocks.c > >> index 7be53cb..b2ffcc7

[PATCH v2 09/16] block: add reference counting for struct bsg_job

2016-10-12 Thread Johannes Thumshirn
Add reference counting to 'struct bsg_job' so we can implement a reuqest timeout handler for bsg_jobs, which is needed for Fibre Channel. Signed-off-by: Johannes Thumshirn --- block/bsg-lib.c | 7 +-- include/linux/bsg-lib.h | 2 ++ 2 files changed, 7

[PATCH v2 11/16] scsi: fc: Use bsg_destroy_job

2016-10-12 Thread Johannes Thumshirn
fc_destroy_bsgjob() and bsg_destroy_job() are now 1:1 copies, so use the later. Signed-off-by: Johannes Thumshirn --- block/bsg-lib.c | 3 ++- drivers/scsi/scsi_transport_fc.c | 20 ++-- include/linux/bsg-lib.h | 1 + 3 files

Re: [PATCH v2] badblocks: fix overlapping check for clearing

2016-10-12 Thread Jens Axboe
On 10/12/2016 04:23 AM, Tomasz Majchrzak wrote: Current bad block clear implementation assumes the range to clear overlaps with at least one bad block already stored. If given range to clear precedes first bad block in a list, the first entry is incorrectly updated. Check not only if stored

Re: [PATCH v3 1/2] block: Add iocontext priority to request

2016-10-12 Thread Jens Axboe
On 10/11/2016 02:40 PM, Adam Manzanares wrote: Patch adds an association between iocontext ioprio and the ioprio of a request. This feature is only enabled if a queue flag is set to indicate that requests should have ioprio associated with them. The queue flag is exposed as the req_prio queue

[PATCH v2 14/16] block: add bsg_job_put() and bsg_job_get()

2016-10-12 Thread Johannes Thumshirn
Add bsg_job_put() and bsg_job_get() so don't need to export bsg_destroy_job() any more. Signed-off-by: Johannes Thumshirn --- block/bsg-lib.c | 17 ++--- drivers/scsi/scsi_transport_fc.c | 4 ++-- include/linux/bsg-lib.h | 3 ++- 3

[PATCH v2 12/16] scsi: fc: use bsg_softirq_done

2016-10-12 Thread Johannes Thumshirn
bsg_softirq_done() and fc_bsg_softirq_done() are copies of each other, so ditch the fc specific one. Signed-off-by: Johannes Thumshirn --- block/bsg-lib.c | 3 ++- drivers/scsi/scsi_transport_fc.c | 16 ++-- include/linux/bsg-lib.h | 1

Re: [PATCHv3 13/41] truncate: make sure invalidate_mapping_pages() can discard huge pages

2016-10-12 Thread Jan Kara
On Wed 12-10-16 00:53:49, Kirill A. Shutemov wrote: > On Tue, Oct 11, 2016 at 05:58:15PM +0200, Jan Kara wrote: > > On Thu 15-09-16 14:54:55, Kirill A. Shutemov wrote: > > > invalidate_inode_page() has expectation about page_count() of the page > > > -- if it's not 2 (one to caller, one to

Re: [PATCH v3 1/2] block: Add iocontext priority to request

2016-10-12 Thread Adam Manzanares
The 10/12/2016 08:49, Jens Axboe wrote: > On 10/11/2016 02:40 PM, Adam Manzanares wrote: > >Patch adds an association between iocontext ioprio and the ioprio of > >a request. This feature is only enabled if a queue flag is set to > >indicate that requests should have ioprio associated with them.