On 7:13 15/05, Christoph Hellwig wrote: > > FYI, generic/475 always fail on me for btrfs, due to the warnings on > transaction abort. > > Anyway, I have come up with a version that seems to mostly work. > > The main change is that btrfs_sync_file stashes away the journal handle. Unfortunately, this will not fly. I tested this week and found transactions begin in writepage() etc. We could work on stashing journal handle in those functions as well, but it looks hackish. We might revert to it as a last resort. In the meantime, I am trying to revert the allocations in case of an error. > I also had to merge parts of the ->iomap_end patch into the main iomap > one. I also did some cleanups to my iomap changes while looking over it. > Let me know what you thing, the tree is here: > > git://git.infradead.org/users/hch/misc.git btrfs-dio > > Gitweb: > > http://git.infradead.org/users/hch/misc.git/shortlog/refs/heads/btrfs-dio -- Goldwyn
