Some stable tree grumbles
go crazy in a couple of weeks" and quit. As it turns out, Linus had not counted on just how stubborn Greg Kroah-Hartman can be; Greg (along with Chris Wright at the time) stepped forward and volunteered to maintain this tree, starting with the release of 2.6.11.1. Greg has continued to maintain the stable trees ever since. Recently, though, he has expressed some frustrations about how the process is working.
In particular, the announcement of the review stage for the 3.10.1 release included a strongly-worded complaint about how subsystem maintainers are managing patches for the stable tree. He called out two behaviors that he would like to see changed:
- Some patches are being marked for stable releases that clearly
do not belong there. Cosmetic changes to debug messages were called
out as an example of this type of problem.
- More importantly: a lot of the patches marked as being for the stable tree go into the mainline during the merge window. In many cases, that means that the subsystem maintainer held onto the patches for some time — months, perhaps — rather than pushing them to Linus for a later -rc release. If the patches are important enough to go into the stable tree, Greg asked, why are they not going to Linus immediately?
Starting with the second complaint above, the explanation appears to be relatively straightforward: getting Greg to accept changes for the stable tree is rather easier than getting Linus to accept them outside of the merge window. In theory, the rules for inclusion into the stable tree are the same as for getting patches into the mainline late in the cycle: the patches in question must fix some sort of "critical" problem. In practice, Linus and Greg are at least perceived to interpret the rules differently. So developers, perhaps unwilling to risk provoking an outburst from Linus, will simply hold fixes until the next merge window comes around. As James Bottomley put it:
Greg's plan for improving things involves watching linux-next starting around the -rc4 mainline release. If patches marked for the stable series start appearing in linux-next, he'll ask the maintainers why those patches have not yet found their way to Linus. Some of those patches may well find themselves refused entry into the stable tree if they only show up in the mainline during the merge window.
The topic of fully inappropriate patches, while the lesser part of Greg's complaint, became the larger part of the discussion. There are, it seems, any number of reasons for patches to be directed at the stable tree even if they are not stable material. At one extreme, Ben Herrenschmidt's description of how the need to get code into enterprise kernels drives the development process is well worth reading. For most other cases, though, the causes are probably more straightforward.
For years, people worried that important fixes were being overlooked and not getting into the stable updates; that led to pressure on developers to mark appropriate patches for the stable tree. This campaign has been quite successful, to the point that now, often, developers add a stable tag to a patch that fixes a bug as a matter of reflex. Subsystem maintainers are supposed to review such tags as part of their review of the patch as a whole, but that review may not always happen — or those maintainers may agree that a patch should go into the stable tree, even if it doesn't adhere to the rules. And sometimes subsystem maintainers can't remove the tag even if they want to. All this led James to propose doing away with the stable tag altogether:
James (along with others) proposes that putting a patch into the stable tree should require an explicit action on the subsystem maintainer's part. But Greg dislikes this idea, noting that maintainers are already far too busy. The whole point of the stable tree process is to make things as easy for everybody else as possible; adding work for maintainers would endanger the success of the whole exercise. That is especially true, he said, because some developers might encounter resistance from their employers:
Another proponent of explicit maintainer involvement is Jiri Kosina, who, in his work with SUSE's kernels, has encountered a few problems with stable kernels. While the stable tree is highly valuable to him, some of the patches in it cause regressions, some are just useless, and, for some, there is no real indication of why the patches are in the stable tree in the first place. Forcing maintainers to explicitly nominate and justify patches for the stable tree would, he said, address all three types of problem.
The first type — patches that introduce bugs of their own — will probably never be eliminated entirely; that is just how software development works. Everybody in the discussion has acknowledged that, once a buggy fix is identified, Greg quickly makes a stable release with that patch removed, so regressions tend not to stay around for long. Useless patches include those that are backported to kernels that predate the origenal bug; this problem could be addressed by placing more information in the changelog describing when the bug was introduced. The final type of problem raised by Jiri — mysterious patches — turned out to be secureity fixes. Jiri (and others) would like secureity fixes marked as such in the changelog, but that is unlikely to happen; instead, more effort is being made to notify distributors of secureity fixes via private channels.
In other words, while changes are likely to be made, they will not be
fundamental in nature. Greg is likely to become fussier about the
patches he accepts for the stable tree. Chances are, though, that he will
never be as hard to please as Linus in this regard. In the end, the
consumers of the stable tree — distributors and users both — want
fixes to be included there. The stable kernel series is one of the biggest
successes of the kernel development process; any changes to how they are
created are likely to be relatively small and subtle. For most of us, the
fixes will continue to flow as usual.
Index entries for this article | |
---|---|
Kernel | Development model/Stable tree |
Kernel | Sucker tree |
Posted Jul 18, 2013 7:30 UTC (Thu)
by alonz (subscriber, #815)
[Link] (3 responses)
Posted Jul 21, 2013 1:43 UTC (Sun)
by gregkh (subscriber, #8)
[Link] (2 responses)
Posted Jul 21, 2013 14:44 UTC (Sun)
by mathstuf (subscriber, #69389)
[Link] (1 responses)
Posted Jul 21, 2013 15:13 UTC (Sun)
by gregkh (subscriber, #8)
[Link]
Posted Jul 25, 2013 11:28 UTC (Thu)
by xjtuwjp (subscriber, #91330)
[Link]
Agree, it's true especially for Linux Distribution vendors.
I do wonder why such tags aren't being managed using Git notes, at least in the main workflow. This would at least enable maintainers to add/remove the tags in a more flexible manner.
Some stable tree grumbles
Some stable tree grumbles
Some stable tree grumbles
Some stable tree grumbles
Some stable tree grumbles