* Re: [PATCH] clean up fs/devfs/base.c [not found] <20031224034110.GA25709@kroah.com> @ 2003-12-24 4:00 ` Ian Kent 2003-12-24 17:10 ` Greg KH 0 siblings, 1 reply; 7+ messages in thread From: Ian Kent @ 2003-12-24 4:00 UTC (permalink / raw) To: Greg KH; +Cc: Andrew Morton, Linus Torvalds, ULMO, linux-kernel On Tue, 23 Dec 2003, Greg KH wrote: > On Tue, Dec 23, 2003 at 06:38:20PM -0800, Andrew Morton wrote: > > > > Now would be a good time for someone to feed the whole thing through indent > > though. > > As much as I enjoy using the devfs code as my "bad coding style" > example, here's a patch against 2.6.0 that cleans up the devfs code to > follow the proper kernel coding style. I think it needs a little bit more than that Greg. I would like to (will) spend some time on it over the next couple of days. Ian ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] clean up fs/devfs/base.c 2003-12-24 4:00 ` [PATCH] clean up fs/devfs/base.c Ian Kent @ 2003-12-24 17:10 ` Greg KH 2003-12-24 18:30 ` Andrew Morton 2003-12-25 3:51 ` Ian Kent 0 siblings, 2 replies; 7+ messages in thread From: Greg KH @ 2003-12-24 17:10 UTC (permalink / raw) To: Ian Kent; +Cc: Andrew Morton, Linus Torvalds, ULMO, linux-kernel On Wed, Dec 24, 2003 at 12:00:13PM +0800, Ian Kent wrote: > On Tue, 23 Dec 2003, Greg KH wrote: > > > On Tue, Dec 23, 2003 at 06:38:20PM -0800, Andrew Morton wrote: > > > > > > Now would be a good time for someone to feed the whole thing through indent > > > though. > > > > As much as I enjoy using the devfs code as my "bad coding style" > > example, here's a patch against 2.6.0 that cleans up the devfs code to > > follow the proper kernel coding style. > > I think it needs a little bit more than that Greg. Oh, I know it does. That was just a "convert to proper coding format in 15 minutes" patch. You should start with that and work from there. But, if you are going to do this, remember to submit in incremental changes. I suggest this patch go in, as it is just a reformatting, nothing else. Then you can work from there, changing the actual logic of the code. thanks, greg k-h ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] clean up fs/devfs/base.c 2003-12-24 17:10 ` Greg KH @ 2003-12-24 18:30 ` Andrew Morton 2003-12-25 4:07 ` Ian Kent 2003-12-26 3:12 ` Ian Kent 2003-12-25 3:51 ` Ian Kent 1 sibling, 2 replies; 7+ messages in thread From: Andrew Morton @ 2003-12-24 18:30 UTC (permalink / raw) To: Greg KH; +Cc: raven, torvalds, ULMO, linux-kernel Greg KH <greg@kroah.com> wrote: > > On Wed, Dec 24, 2003 at 12:00:13PM +0800, Ian Kent wrote: > > On Tue, 23 Dec 2003, Greg KH wrote: > > > > > On Tue, Dec 23, 2003 at 06:38:20PM -0800, Andrew Morton wrote: > > > > > > > > Now would be a good time for someone to feed the whole thing through indent > > > > though. > > > > > > As much as I enjoy using the devfs code as my "bad coding style" > > > example, here's a patch against 2.6.0 that cleans up the devfs code to > > > follow the proper kernel coding style. > > > > I think it needs a little bit more than that Greg. > > Oh, I know it does. That was just a "convert to proper coding format in > 15 minutes" patch. You should start with that and work from there. > But, if you are going to do this, remember to submit in incremental > changes. I suggest this patch go in, as it is just a reformatting, > nothing else. Then you can work from there, changing the actual logic > of the code. Yup, just whitespace fixes please. I don't think I have the energy for a big cleanup exercise right now, and it's not really appropriate. ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] clean up fs/devfs/base.c 2003-12-24 18:30 ` Andrew Morton @ 2003-12-25 4:07 ` Ian Kent 2003-12-26 3:12 ` Ian Kent 1 sibling, 0 replies; 7+ messages in thread From: Ian Kent @ 2003-12-25 4:07 UTC (permalink / raw) To: Andrew Morton; +Cc: Greg KH, torvalds, ULMO, linux-kernel On Wed, 24 Dec 2003, Andrew Morton wrote: > > > > Oh, I know it does. That was just a "convert to proper coding format in > > 15 minutes" patch. You should start with that and work from there. > > But, if you are going to do this, remember to submit in incremental > > changes. I suggest this patch go in, as it is just a reformatting, > > nothing else. Then you can work from there, changing the actual logic > > of the code. > > Yup, just whitespace fixes please. I don't think I have the energy for a > big cleanup exercise right now, and it's not really appropriate. > OK. Didn't see this yet. I'll test what I have and get an initial patch to you. Ian ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] clean up fs/devfs/base.c 2003-12-24 18:30 ` Andrew Morton 2003-12-25 4:07 ` Ian Kent @ 2003-12-26 3:12 ` Ian Kent 2003-12-27 11:19 ` Ian Kent 1 sibling, 1 reply; 7+ messages in thread From: Ian Kent @ 2003-12-26 3:12 UTC (permalink / raw) To: Andrew Morton; +Cc: Greg KH, Linus Torvalds, Kernel Mailing List On Wed, 24 Dec 2003, Andrew Morton wrote: > > Yup, just whitespace fixes please. I don't think I have the energy for a > big cleanup exercise right now, and it's not really appropriate. > OK. Got side tracked for a while. White space only (just about) patch is on kernel.org at: /pub/linux/kernel/perope/raven/devfs/linux-2.6.0-devfs-1.patch It compiles, links and basic functionality tested OK with devfsd 1.3.25, against 2.6.0. Ian ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] clean up fs/devfs/base.c 2003-12-26 3:12 ` Ian Kent @ 2003-12-27 11:19 ` Ian Kent 0 siblings, 0 replies; 7+ messages in thread From: Ian Kent @ 2003-12-27 11:19 UTC (permalink / raw) To: Andrew Morton; +Cc: Greg KH, Linus Torvalds, Kernel Mailing List On Fri, 26 Dec 2003, Ian Kent wrote: > On Wed, 24 Dec 2003, Andrew Morton wrote: > > > > > Yup, just whitespace fixes please. I don't think I have the energy for a > > big cleanup exercise right now, and it's not really appropriate. > > > > OK. Got side tracked for a while. > > White space only (just about) patch is on kernel.org at: > > /pub/linux/kernel/perope/raven/devfs/linux-2.6.0-devfs-1.patch > > It compiles, links and basic functionality tested OK with devfsd > 1.3.25, against 2.6.0. > Have done a 2nd pass of devfs for 'almost white space only' changes. It is in linux-2.6.0-devfs-2.patch. I have tested it as I did the first. It should be applied over the first. Have run out of time now so that will have to be it for now. Ian ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH] clean up fs/devfs/base.c 2003-12-24 17:10 ` Greg KH 2003-12-24 18:30 ` Andrew Morton @ 2003-12-25 3:51 ` Ian Kent 1 sibling, 0 replies; 7+ messages in thread From: Ian Kent @ 2003-12-25 3:51 UTC (permalink / raw) To: Greg KH; +Cc: Andrew Morton, Linus Torvalds, ULMO, linux-kernel On Wed, 24 Dec 2003, Greg KH wrote: > > I think it needs a little bit more than that Greg. > > Oh, I know it does. That was just a "convert to proper coding format in > 15 minutes" patch. You should start with that and work from there. > But, if you are going to do this, remember to submit in incremental > changes. I suggest this patch go in, as it is just a reformatting, > nothing else. Then you can work from there, changing the actual logic > of the code. Yep. I have started with the patch you provided. Thanks. I'm a bit of a white space obsessive compulsive type so I have made a fair few more changes. I tried hard to resist changing the code but some of those long lines just had to go. I'm having difficulty resisting breaking up base.c. I just can't bare 3000+ lines in one source file. I'm thinking I should just go ahead, do a limited test and then get a couple of people that use devfs to test it before submitting it. Mind, I'm not changing the logic, just the layout and structure. You don't agree I think. I'm making patches as I go so I can do the incremental thing without being held up. I have the time to do this now, over the Xmas break, so I'll go ahead and return to fix things if I need to. Lets not loose sight of the reason I volunteered to provide maintenance only, best effort only fixes during the deprecation process. That is, as you know, to quell the unrest on the list regarding devfs, so that people like yourself can get on with udev. Simple things like replacing OBSOLETE with DEPRECATED in fs/Kconfig and rewording last paragraph to better reflect Andrews actual intent would help a lot. So it's Xmas day and I shouldn't be so serious. Hope everyone has a great day. Indeed, some of us won't get to it till tomorrow, fantastic. Ian ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2003-12-27 11:18 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20031224034110.GA25709@kroah.com>
2003-12-24 4:00 ` [PATCH] clean up fs/devfs/base.c Ian Kent
2003-12-24 17:10 ` Greg KH
2003-12-24 18:30 ` Andrew Morton
2003-12-25 4:07 ` Ian Kent
2003-12-26 3:12 ` Ian Kent
2003-12-27 11:19 ` Ian Kent
2003-12-25 3:51 ` Ian Kent
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox