On 11/17/2016 01:17 PM, Christoph Hellwig wrote:
Ok, the updated tree is here:
http://git.infradead.org/users/hch/block.git/shortlog/refs/heads/new-dio
This moves your O_SYNC patch earlier and makes it not directly
assign bi_opf.
Then comes the main direct I/O path with the various folds, and
a use after free fix ported from the iomap code in the completion
handler.
Last comes the I/O completion handler split - I don't think it's
worth it, but if you prefer it that way go ahead and fold it in.
The last patch seems to have some issues:
+ bool is_sync = is_sync = is_sync_kiocb(iocb);
and it's reverting part of the dio->is_sync that was part of my union
waiter/iocb change. So I'll leave that out for now, for those reasons.
I couldn't do the bio_kmalloc conversion as bio_kmalloc seems
to not like the front pad.
Yeah, !bs/bio_kmalloc() doesn't use the front pad...
--
Jens Axboe
--
To unsubscribe from this list: send the line "unsubscribe linux-block" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
