From: Al Viro <viro@ZenIV.linux.org.uk>
To: Trond Myklebust <Trond.Myklebust@netapp.com>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
linux-nfs@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: Merge of the 'write_inode' branch from the VFS tree
Date: Fri, 5 Mar 2010 15:48:23 +0000 [thread overview]
Message-ID: <20100305154823.GV30031@ZenIV.linux.org.uk> (raw)
In-Reply-To: <1267802800.5174.6.camel@localhost.localdomain>
On Fri, Mar 05, 2010 at 10:26:40AM -0500, Trond Myklebust wrote:
> Hi Al,
>
> When you submitted the VFS changes for this merge window, I was hoping
> you would include the 'write_inode' branch. I've been waiting for them
> in order to push the NFS writeback improvements to Linus.
>
> Would you be willing to either push the write_inode branch to Linus or
> to Ack my doing so as part of the NFS push?
Said branch has managed to grow conflicts with XFS commits already in
the mainline ;-/ With commits postdating the write_inode ones by a week
or so and having the same author.
I'm going to push the next VFS pile in about half an hour and get to the
write_inode situation. I'm not sure what's the best course here. Note
that since you've pulled it, you also have conflicts with what's in the
mainline. I can do *another* backmerge (already had one due to gfs2 trivial
conflicts) and push the result. Which will suck, since XFS conflicts
are not entirely trivial and we'll get a really ugly merge node, with
conflict resolution both hidden and not quite obvious.
Or I can do a new branch, put updated pair of patches there (hch has sent
the updated variants my way) and ask you to rebuild NFS tree. Which will
also suck, since it adds PITA for you and you are completely innocent in
that clusterfuck.
Suggestions? I'd love to get out of that mess with minimal PITA for
everyone involved and minimally messed tree...
One thing for sure - I'm not going to do that kind of "guaranteed to be
unchanged" shared branches again, TYVM.
next prev parent reply other threads:[~2010-03-05 15:48 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-05 15:26 Merge of the 'write_inode' branch from the VFS tree Trond Myklebust
2010-03-05 15:48 ` Al Viro [this message]
2010-03-05 17:40 ` [git pull] vfs part 3 (write_inode mess) Al Viro
2010-03-08 20:22 ` Steve Dickson
2010-03-09 8:52 ` Dave Chinner
2010-03-10 23:07 ` Trond Myklebust
2010-03-05 18:02 ` Merge of the 'write_inode' branch from the VFS tree Trond Myklebust
2010-03-05 18:29 ` Al Viro
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20100305154823.GV30031@ZenIV.linux.org.uk \
--to=viro@zeniv.linux.org.uk \
--cc=Trond.Myklebust@netapp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-nfs@vger.kernel.org \
--cc=torvalds@linux-foundation.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox