linux-stable.git/drivers/md/bcache/request.c, branch linux-4.13.y

bcache: fix sequential large write IO bypass

2017-09-27T12:43:21+00:00

commit c81ffa32a214c84b08900fbc9d432187bd948eba upstream.

Sequential write IOs were tested with bs=1M by FIO in writeback cache
mode, these IOs were expected to be bypassed, but actually they did not.
We debug the code, and find in check_should_bypass():
    if (!congested &&
        mode == CACHE_MODE_WRITEBACK &&
        op_is_write(bio_op(bio)) &&
        (bio-＞bi_opf & REQ_SYNC))
        goto rescale
that means, If in writeback mode, a write IO with REQ_SYNC flag will not
be bypassed though it is a sequential large IO, It's not a correct thing
to do actually, so this patch remove these codes.

Signed-off-by: tang.junhui 
Reviewed-by: Kent Overstreet 
Reviewed-by: Eric Wheeler 
Signed-off-by: Jens Axboe 
Signed-off-by: Greg Kroah-Hartman

bcache: do not subtract sectors_to_gc for bypassed IO

2017-09-27T12:43:21+00:00

commit 69daf03adef5f7bc13e0ac86b4b8007df1767aab upstream.

Since bypassed IOs use no bucket, so do not subtract sectors_to_gc to
trigger gc thread.

Signed-off-by: tang.junhui 
Acked-by: Coly Li 
Reviewed-by: Eric Wheeler 
Reviewed-by: Christoph Hellwig 
Signed-off-by: Jens Axboe 
Signed-off-by: Greg Kroah-Hartman

block: switch bios to blk_status_t

2017-06-09T15:27:32+00:00

Replace bi_error with a new bi_status to allow for a clear conversion.
Note that device mapper overloaded bi_error with a private value, which
we'll have to keep arround at least for now and thus propagate to a
proper blk_status_t value.

Signed-off-by: Christoph Hellwig 
Signed-off-by: Jens Axboe

block: Use pointer to backing_dev_info from request_queue

2017-02-02T15:20:48+00:00

We will want to have struct backing_dev_info allocated separately from
struct request_queue. As the first step add pointer to backing_dev_info
to request_queue and convert all users touching it. No functional
changes in this patch.

Reviewed-by: Christoph Hellwig 
Signed-off-by: Jan Kara 
Signed-off-by: Jens Axboe

block: add a op_is_flush helper

2017-01-27T16:01:45+00:00

This centralizes the checks for bios that needs to be go into the flush
state machine.

Signed-off-by: Christoph Hellwig 
Reviewed-by: Martin K. Petersen 
Reviewed-by: Bart Van Assche 
Signed-off-by: Jens Axboe

bcache: Make gc wakeup sane, remove set_task_state()

2016-12-17T20:01:55+00:00

Signed-off-by: Kent Overstreet

block: bio: pass bvec table to bio_init()

2016-11-22T15:57:21+00:00

Some drivers often use external bvec table, so introduce
this helper for this case. It is always safe to access the
bio->bi_io_vec in this way for this case.

After converting to this usage, it will becomes a bit easier
to evaluate the remaining direct access to bio->bi_io_vec,
so it can help to prepare for the following multipage bvec
support.

Signed-off-by: Ming Lei 
Reviewed-by: Christoph Hellwig 

Fixed up the new O_DIRECT cases.

Signed-off-by: Jens Axboe

block,fs: use REQ_* flags directly

2016-11-01T15:43:26+00:00

Remove the WRITE_* and READ_SYNC wrappers, and just use the flags
directly.  Where applicable this also drops usage of the
bio_set_op_attrs wrapper.

Signed-off-by: Christoph Hellwig 
Signed-off-by: Jens Axboe

bcache: use op_is_sync to check for synchronous requests

2016-11-01T15:43:26+00:00

(and remove one layer of masking for the op_is_write call next to it).

Signed-off-by: Christoph Hellwig 
Signed-off-by: Jens Axboe

block: export bio_free_pages to other modules

2016-09-22T13:48:03+00:00

bio_free_pages is introduced in commit 1dfa0f68c040
("block: add a helper to free bio bounce buffer pages"),
we can reuse the func in other modules after it was
imported.

Cc: Christoph Hellwig 
Cc: Jens Axboe 
Cc: Mike Snitzer 
Cc: Shaohua Li 
Signed-off-by: Guoqing Jiang 
Acked-by: Kent Overstreet 
Signed-off-by: Jens Axboe