Christoph Hellwig <hch@xxxxxxxxxxxxx> wrote: > On Wed, Feb 15, 2023 at 01:17:56PM +0000, David Howells wrote: > > Probably not, but I don't want to fiddle with that right now. I can send a > > follow up patch for it. > > Honestly, I think this rush for 6.3 inclusion is a really bad idea. > > This series fundamentally changes how splice reads work, and has only > been out for about a week. It hasn't even been Cc'ed to Al Sorry, what?! Al has been To'd or cc'd on every patch. > and Linus I don't know that it's necessary to cc Linus on everything. Jens is the splice maintainer, I thought. > which generally have a good knowledge of the splice code and an opinion > on it. > > I think it is a good change, but I'd feel much more comfortable with > it for the next merge window rather than rushing it. The lack of iov_iter_extract_pages() is blocking other things I want to work on - and will push those out another 3 months further beyond this. I'm fine with dropping the block layer changes and most of the splice changes, but I do want to try to get patches 1-3, 10 and 11: mm: Pass info, not iter, into filemap_get_pages() splice: Add a func to do a splice from a buffered file without ITER_PIPE splice: Add a func to do a splice from an O_DIRECT file without ITER_PIPE iov_iter: Add a function to extract a page list from an iterator iov_iter: Define flags to qualify page extraction. upstream through the cifs tree if you, Jens and Steve French have no objection, with my cifs iteratorisation patches on top. It shouldn't affect anything other than cifs in this merge window, barring the change to the flags to iov_iter_get_pages*(). David
