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 majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
