On Mon, Nov 05, 2018 at 04:36:34PM +0000, Filipe Manana wrote: > On Mon, Nov 5, 2018 at 4:34 PM David Sterba <dste...@suse.cz> wrote: > > On Mon, Nov 05, 2018 at 04:30:35PM +0000, Filipe Manana wrote: > > > On Mon, Nov 5, 2018 at 4:29 PM David Sterba <dste...@suse.cz> wrote: > > > > On Wed, Oct 24, 2018 at 01:48:40PM +0100, Filipe Manana wrote: > > > > > > Ah ok makes sense. Well in that case lets just make > > > > > > btrfs_read_locked_inode() > > > > > > take a path, and allocate it in btrfs_iget, that'll remove the ugly > > > > > > > > > > > > if (path != in_path) > > > > > > > > > > You mean the following on top of v4: > > > > > > > > > > https://friendpaste.com/6XrGXb5p0RSJGixUFYouHg > > > > > > > > > > Not much different, just saves one such if statement. I'm ok with > > > > > that. > > > > > > > > Now in misc-next with v4 and the friendpaste incremental as > > > > > > > > https://github.com/kdave/btrfs-devel/commit/efcfd6c87d28b3aa9bcba52d7c1e1fc79a2dad69 > > > > > > Please don't add the incremental. It's buggy. It was meant to figure > > > out what Josef was saying. That's why I haven't sent a V5. > > > > Ok dropped, I'll will wait for a proper patch. > > It's V4, the last sent version. Just forget the incremental. > Thanks.
For the record, V4 has been merged to master in 4.20-rc2.
