Hi Greg, On Wed, 2010-04-28 at 16:59 +0200, ext Greg KH wrote: > On Wed, Apr 28, 2010 at 12:02:57PM +0200, Carsten Munk wrote: > > Hi, > > > > I noticed the following lines in > > http://meego.gitorious.org/meego-os-base/kernel-source/blobs/master/README#line120 > > : > > > > * Unless the author identified in the From: tag has a @intel.com or > > @linux.intel.com address, the patch must include a Signed-off-by: or > > Acked-by: header which identifies the person in one of these domains > > who feels responsible for the patch inside the company. > > > > Would it be possible to reword this in a manner so it does not sounds > > as biased towards Intel employees? Ie, treating non-member > > contributions with same process and review as member contributions. > > > > A suggestion could be referring to upstream approval by subsystem > > maintainer or by someone responsible for the subsystem/architecture in > > the MeeGo kernel maintainer team. > > Heh, that file is copied directly from the suse kernel tree, with only > s/suse/intel/ applied to it there. The reason that we (Novell) require > this, is that we want someone to "own" the kernel patch so that we know > who to blame if something goes wrong.
Agreed! > As Intel is ultimately responsible for distributing and maintaining this > kernel tree, having someone within intel to "own" each patch in the same > manner makes lots of sense as well. In Moblin days this was a perfect argument. But things should change when this kernel became part of MeeGo project which is a joint effort between Intel and Nokia. Is this insider information which is not made public that only Intel is ultimately responsible for the kernel? If we are going to integrate patches which enable different functionality on Nokia N900 into MeeGo kernel, then I don't think Intel would like to "own" those patches. And I don't think they would liked to be blamed for the same ;) > So I would recommend just leaving it alone, unless you want to maintain > the whole kernel package for intel yourself? :) I disagree :( There should be room to accommodate kernel maintainers apart from Intel. Cheers, Ameya. _______________________________________________ MeeGo-dev mailing list [email protected] http://lists.meego.com/listinfo/meego-dev
