On 9/20/18 10:39 AM, Jens Axboe wrote:

On 9/20/18 12:29 AM, Christoph Hellwig wrote:

quoted

On Sat, Sep 15, 2018 at 08:47:13AM -0600, Jens Axboe wrote:

quoted

quoted

quoted

this series moves various helpers related to merging based on physical
addresses from the public headers into block/, moves the Xen special
case from arch hooks into common code, cleans up the code a bit, and
removes not nessecary includes from the block headers.

---end quoted text---

It's a good cleanup, I like it. Would prefer if the arm/xen folks could
ack the first bits,

Konread, can you look at the series (or delegate).

Konrad is out this (and last) week, this looks good to me. Including
patch 13, although it's hard to say whether it may break some builds.

-boris

quoted

quoted

and 13/13 should probably just to in differently.

I can't parse this setence.

The irony is thick :-)

Anyway, s/to/go for that sentence. I don't mind carrying it if Konrad
is happy with it, but it could go in separately after this series is
merged.