linux.git/fs/bio.c, branch v3.14

block: Fix cloning of discard/write same bios

2014-02-11T15:40:45+00:00

Immutable biovecs changed the way bio segments are treated in such a way that
bio_for_each_segment() cannot now do what we want for discard/write same bios,
since bi_size means something completely different for them.

Fortunately discard and write same bios never have more than a single biovec, so
bio_for_each_segment() is unnecessary and not terribly meaningful for them, but
we still have to special case them in a few places.

Signed-off-by: Kent Overstreet 
Tested-by: Richard W.M. Jones 
Signed-off-by: Jens Axboe

Revert "block: Warn and free bio if bi_end_io is not set"

2014-01-08T21:14:22+00:00

This reverts commit 95d4403889acbd98e06d41a255df76452210996a.

The patch is broken for on-stack bios, amongst other things.

block: Warn and free bio if bi_end_io is not set

2014-01-08T16:39:49+00:00

In bio_endio if bio doesn't have bi_end_io (should be an error case),
we set bio to NULL and continue silently without freeing the bio. It
would be good to have a WARN and free the bio to avoid memory leak.

Signed-off-by: Muthukumar Ratty 
Signed-off-by: Jens Axboe

block: Kill bio_pair_split()

2013-11-24T06:33:57+00:00

Signed-off-by: Kent Overstreet 
Cc: Jens Axboe

block: Introduce new bio_split()

2013-11-24T06:33:57+00:00

The new bio_split() can split arbitrary bios - it's not restricted to
single page bios, like the old bio_split() (previously renamed to
bio_pair_split()). It also has different semantics - it doesn't allocate
a struct bio_pair, leaving it up to the caller to handle completions.

Then convert the existing bio_pair_split() users to the new bio_split()
- and also nvme, which was open coding bio splitting.

(We have to take that BUG_ON() out of bio_integrity_trim() because this
bio_split() needs to use it, and there's no reason it has to be used on
bios marked as cloned; BIO_CLONED doesn't seem to have clearly
documented semantics anyways.)

Signed-off-by: Kent Overstreet 
Cc: Jens Axboe 
Cc: Martin K. Petersen 
Cc: Matthew Wilcox 
Cc: Keith Busch 
Cc: Vishal Verma 
Cc: Jiri Kosina 
Cc: Neil Brown

block: Rename bio_split() -> bio_pair_split()

2013-11-24T06:33:56+00:00

This is prep work for introducing a more general bio_split().

Signed-off-by: Kent Overstreet 
Cc: Jens Axboe 
Cc: NeilBrown 
Cc: Alasdair Kergon 
Cc: Lars Ellenberg 
Cc: Peter Osterlund 
Cc: Sage Weil

block: Generic bio chaining

2013-11-24T06:33:56+00:00

This adds a generic mechanism for chaining bio completions. This is
going to be used for a bio_split() replacement, and it turns out to be
very useful in a fair amount of driver code - a fair number of drivers
were implementing this in their own roundabout ways, often painfully.

Note that this means it's no longer to call bio_endio() more than once
on the same bio! This can cause problems for drivers that save/restore
bi_end_io. Arguably they shouldn't be saving/restoring bi_end_io at all
- in all but the simplest cases they'd be better off just cloning the
bio, and immutable biovecs is making bio cloning cheaper. But for now,
we add a bio_endio_nodec() for these cases.

Signed-off-by: Kent Overstreet 
Cc: Jens Axboe

block: Remove bi_idx hacks

2013-11-24T06:33:55+00:00

Now that drivers have been converted to the new bvec_iter primitives,
there's no need to trim the bvec before we submit it; and we can't trim
it once we start sharing bvecs.

It used to be that passing a partially completed bio (i.e. one with
nonzero bi_idx) to generic_make_request() was a dangerous thing -
various drivers would choke on such things. But with immutable biovecs
and our new bio splitting that shares the biovecs, submitting partially
completed bios has to work (and should work, now that all the drivers
have been completed to the new primitives)

Signed-off-by: Kent Overstreet 
Cc: Jens Axboe

block: Don't save/copy bvec array anymore

2013-11-24T06:33:55+00:00

Now that drivers have been converted to the bvec_iter primitives, they
shouldn't be modifying the biovec anymore and thus saving it is
unnecessary - code that was previously making a backup of the bvec array
can now just save bio->bi_iter.

Signed-off-by: Kent Overstreet 
Cc: Jens Axboe

dm: Refactor for new bio cloning/splitting

2013-11-24T06:33:55+00:00

We need to convert the dm code to the new bvec_iter primitives which
respect bi_bvec_done; they also allow us to drastically simplify dm's
bio splitting code.

Also, it's no longer necessary to save/restore the bvec array anymore -
driver conversions for immutable bvecs are done, so drivers should never
be modifying it.

Also kill bio_sector_offset(), dm was the only user and it doesn't make
much sense anymore.

Signed-off-by: Kent Overstreet 
Cc: Jens Axboe 
Cc: Alasdair Kergon 
Cc: dm-devel@redhat.com
Reviewed-by: Mike Snitzer