Re: bcache: remove driver private bio splitting code

[Vojtech Pavlik <vojtech@suse.com>]

On Tue, Jan 12, 2016 at 06:00:49PM -0800, Eric Wheeler wrote:

Hello Eric,

> Have you tested the patch below in SLE12-* when bcache is backed by md
> raid5/6?
> 
> FYI: I was compared the drivers/md/bcache/io.c functions in the various
> branches here:
>   https://github.com/openSUSE/kernel
> I compared the presence of bch_generic_make_request() (which the patch
> below removes).  It looks like the branch SLE12-SP2 has the patch, but
> version before SLE12-SP2 and openSUSE-* do not (as they still have
> bch_generic_make_request).
> 
> Since the patch does exist in SLE12-SP2, I'm guessing that is been tested,
> though I am curious if it has been tested specifically when being backed
> by md raid5/6 so that queue->limits->partial_stripes_expensive is nonzero.
> 
> If you have and it is stable, then I want to get it to Kent and Jens for 
> upstream integration.

The SLES12-SP2 kernel branch is very fresh, created this week. So while
the patch was tested by Johannes before adding it, and by our per-commit
automated tests, it didn't go through fully qualified QA test cycle yet.
So I won't say it's proven stable just yet.

-- 
Vojtech Pavlik
Director SUSE Labs