* Re: [PATCH 0/14] JANITOR: Remove include/linux/sys.h
2013-10-29 9:53 [PATCH 0/14] JANITOR: Remove include/linux/sys.h Mika Kukkonen
@ 2013-10-29 13:36 ` Dan Carpenter
2013-10-30 10:59 ` Mika Kukkonen
` (2 subsequent siblings)
3 siblings, 0 replies; 5+ messages in thread
From: Dan Carpenter @ 2013-10-29 13:36 UTC (permalink / raw)
To: kernel-janitors
On Tue, Oct 29, 2013 at 11:53:48AM +0200, Mika Kukkonen wrote:
> Hello,
>
> File include/linux/sys.h has a comment stating that the "file is no longer used or needed",
> but it is still included in multiple places under several architectures. It does contain
> bunch of #define's but those have been #ifdef'd out, and grepping did not show any uses
> of those #define's in the source tree outside this file.
>
> The uses of the file seem partially be accidental remains of some old use, but almost all
> are in architecture specific code with only one outside in file net/core/pktgen.c. So this
> patchset has been divided to 14 parts, addressing 12 architectures (c6x, cris, frv, h8300,
> microblaze, mips, mn10300, parisc, powerpc, sh, unicore32 and x86) and the outlier in net
> and then the final patch removes the file itself. All patches can be applied invidually
> with the obvious exception of the last one which actually removes the file and should be
> last.
>
Kernel Janitors doesn't have an a patch tree. In theory these could all
go through trivial@kernel.org or they could go to the subsystem
maintainers. You pretty much have broken it up by subsystem already.
The subjects need to be different. Each should reflect the subsystem
conventions. Find the correct subject by doing:
git log --oneline arch/c6x/
git log --oneline arch/cris/
The complication with doing it by the subsystem is that the last patch
as to go in last. I would suggest sending the first 13 patches right
away and holding off until v3.13-rc1 has been released in 4 weeks time.
By then the earlier patches should have been merged. Otherwise wait
until v3.14 to apply the last patch. It's not a race, and it will make
it easiest if the 13 have gone to Linus before we take the final step.
regards,
dan carpenter
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH 0/14] JANITOR: Remove include/linux/sys.h
2013-10-29 9:53 [PATCH 0/14] JANITOR: Remove include/linux/sys.h Mika Kukkonen
2013-10-29 13:36 ` Dan Carpenter
@ 2013-10-30 10:59 ` Mika Kukkonen
2013-10-30 11:03 ` Julia Lawall
2013-10-30 11:29 ` Mika Kukkonen
3 siblings, 0 replies; 5+ messages in thread
From: Mika Kukkonen @ 2013-10-30 10:59 UTC (permalink / raw)
To: kernel-janitors
On Tue, Oct 29, 2013 at 01:36:09PM +0000, Dan Carpenter wrote:
>
> Kernel Janitors doesn't have an a patch tree. In theory these could all
> go through trivial@kernel.org or they could go to the subsystem
> maintainers. You pretty much have broken it up by subsystem already.
Yes, but I chickened out and did not want to CC 14 subsystem maintainers. ;-)
> The complication with doing it by the subsystem is that the last patch
> as to go in last. I would suggest sending the first 13 patches right
> away and holding off until v3.13-rc1 has been released in 4 weeks time.
> By then the earlier patches should have been merged. Otherwise wait
> until v3.14 to apply the last patch. It's not a race, and it will make
> it easiest if the 13 have gone to Linus before we take the final step.
Well, this is in no way urgent issue, so I'll revisit it in the next release
cycle if nothing happends before that. Thanks for your feedback!
--MiKu
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 0/14] JANITOR: Remove include/linux/sys.h
2013-10-29 9:53 [PATCH 0/14] JANITOR: Remove include/linux/sys.h Mika Kukkonen
2013-10-29 13:36 ` Dan Carpenter
2013-10-30 10:59 ` Mika Kukkonen
@ 2013-10-30 11:03 ` Julia Lawall
2013-10-30 11:29 ` Mika Kukkonen
3 siblings, 0 replies; 5+ messages in thread
From: Julia Lawall @ 2013-10-30 11:03 UTC (permalink / raw)
To: kernel-janitors
On Wed, 30 Oct 2013, Mika Kukkonen wrote:
> On Tue, Oct 29, 2013 at 01:36:09PM +0000, Dan Carpenter wrote:
> >
> > Kernel Janitors doesn't have an a patch tree. In theory these could all
> > go through trivial@kernel.org or they could go to the subsystem
> > maintainers. You pretty much have broken it up by subsystem already.
>
> Yes, but I chickened out and did not want to CC 14 subsystem maintainers. ;-)
I've deleted the mail with the patches, but if it would be useful for the
maintainers to see the patches, you should send them to them. That is,
send each patch to the people indicated by get_maintainers.
julia
>
> > The complication with doing it by the subsystem is that the last patch
> > as to go in last. I would suggest sending the first 13 patches right
> > away and holding off until v3.13-rc1 has been released in 4 weeks time.
> > By then the earlier patches should have been merged. Otherwise wait
> > until v3.14 to apply the last patch. It's not a race, and it will make
> > it easiest if the 13 have gone to Linus before we take the final step.
>
> Well, this is in no way urgent issue, so I'll revisit it in the next release
> cycle if nothing happends before that. Thanks for your feedback!
>
> --MiKu
> --
> To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 0/14] JANITOR: Remove include/linux/sys.h
2013-10-29 9:53 [PATCH 0/14] JANITOR: Remove include/linux/sys.h Mika Kukkonen
` (2 preceding siblings ...)
2013-10-30 11:03 ` Julia Lawall
@ 2013-10-30 11:29 ` Mika Kukkonen
3 siblings, 0 replies; 5+ messages in thread
From: Mika Kukkonen @ 2013-10-30 11:29 UTC (permalink / raw)
To: kernel-janitors
On Wed, Oct 30, 2013 at 11:03:57AM +0000, Julia Lawall wrote:
> On Wed, 30 Oct 2013, Mika Kukkonen wrote:
>
> > On Tue, Oct 29, 2013 at 01:36:09PM +0000, Dan Carpenter wrote:
> > >
> > > Kernel Janitors doesn't have an a patch tree. In theory these could all
> > > go through trivial@kernel.org or they could go to the subsystem
> > > maintainers. You pretty much have broken it up by subsystem already.
> >
> > Yes, but I chickened out and did not want to CC 14 subsystem maintainers. ;-)
>
> I've deleted the mail with the patches, but if it would be useful for the
> maintainers to see the patches, you should send them to them. That is,
> send each patch to the people indicated by get_maintainers.
Yeah, I'll probably do that, but I got bunch other janitorial stuff(*) in pipeline,
so I will probably delay this to next merge window as this is in no way urgent.
--MiKu
(*) Basically I am trying to make "make W=1" usefull, I just happened to stumble
upon this <linux/sys.h> while working on that.
^ permalink raw reply [flat|nested] 5+ messages in thread