Re: linux-next: block tree build failure

[James Bottomley <James.Bottomley@HansenPartnership.com>]

On Fri, 2009-05-15 at 09:45 +0900, Tejun Heo wrote:
> Hello, James.
> 
> James Bottomley wrote:
> > On Wed, 2009-05-13 at 18:21 +0900, Tejun Heo wrote:
> >>> I have reverted commit b1f744937f1be3e6d3009382a755679133cf782d
> >>> ("block: move completion related functions back to blk-core.c") and
> >>> applied the following patch (which I realise is probably not
> >>> correct) for today.  Maybe someone can come up with a better
> >>> solution for the scsi guys and me.
> >> Hmmm... is there a SCSI tree which won't be rebased?  Then I can pull
> >> blk tree into it and SCSI tree can go on from that point on.
> > 
> > Not really ... especially now there's a proposal to redo the mvsas
> > patch.  However, block can run a postmerge tree with the changes in
> > them.
> 
> Aieee... Yeah, I guess I can maintain temporary merge tip for blk and
> scsi (that's what you're talking about, right?).

No ... postmerge trees are held by maintainers for irreconcilable tree
conflicts.  You run your standard tree, which can fire at any time in
the merge window and your postmerge one which pulls in the entangled
tree and adds your patches on top.   This can only go after the
conflicting subsystem has also been pulled into linus head.

>   Out of curiousity,
> is there any reason not to use more standard git workflow?

This is a standard workflow ... it's how we've pretty much always sorted
out entanglements ... at least it's between me and Jens, which are two
git trees ... I've had to run post merge trees against gregkh's quilt
before now.

>   Developers
> (kernel devs at least) are now pretty comfortable with git and trees
> merging and splitting off.  I can't really see much value in rebasing
> these days.

That depends.  It's not unusual for patches to get dropped or moved
around in my trees (I try not to do it often) which can't be done
without rebasing.  That pretty much precludes merging because I need a
cleanly rebaseable change set.

James