[lkml]   [2005]   [Sep]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] rmmod notifier chain
>> That's funny - on one hand I'm asked to not submit huge patches (not
>> you, but by others), but on the other hand not having the consuming
>> in the same patch as the providing one is now deemed to be a
>Each patch should do a single logical thing. That doesn't mean that
>want to trickle patches in across a period of months. It means that
>bunch of spearate (and separately reviewed) patches can all go in at
>same time.
>So the split-it-up request is for reviewing (and debugging)

Forgive my non-understanding:

First, I rarely saw any kind of positive review feedback from lkml
besides the notification that you added something to your -mm tree
(negative things of course always arrive), yet no feedback at all is far
from meaning that a given patch is ever going to be accepted (as a
really good example take the tiny patch to fix the broken range check in
i386's low level NMI handler).

Second, since patches depend on one another in many cases it seemed
most natural to me to first break out things that aren't directly
related to nlkd or, if directly related, could still be viewed as
independent pieces of work. Hence I wouldn't consider it reasonable to
break up the debugger patch entirely and submit all the pieces at once,
because that could easily mean that if one intermediate piece doesn't
get accepted all the dependent pieces have been separated out

I'd be curious to know how you, considering yourself in my position,
would have approached breaking up and submitting that size a patch.


To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2005-09-09 14:46    [W:0.063 / U:3.152 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site