lkml.org 
[lkml]   [2006]   [Jan]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 8/11] Time: i386 Conversion - part 4: ACPI PM variable renaming and config change.
john stultz <johnstul@us.ibm.com> wrote:
>
> Andrew, All,
> The conversion of i386 to use the generic timeofday subsystem
> has been split into 6 parts. This patch, the fourth of six, renames
> some ACPI PM variables and removes the CONFIG_X86_PM_TIMER option.
>
> It applies on top of my timeofday-arch-i386-part3 patch. This patch is
> part the timeofday-arch-i386 patchset, so without the following parts
> it is not expected to compile.
>
> Andrew, please consider for inclusion into your tree.

Problems:

- The changelogs aren't terribly good.

When preparing changelogs, please think of how they will look in the
final git record. This means that metainfo concerning preceding patch
dependencies, the number of patches in the series, etc is irrelevant.
The sequence numbering in the Subject: (which you have done correctly) is
sufficient.

Text such as "please apply", while nice, just has to be removed by
myself. I'm not offended if it doesn't get sent ;)

Alert readers will note that I also strip out text such as "this patch
does X" from the changelogs. Because such text is redundant once the
patch is merged up. We _know_ it's a patch - otherwise it wouldn't be in
the git tree.

And once we strip out the above irrelevencies, your changelogs are
awfully spartan for such a substantial piece of work. Isn't there more
to be said?

- The fact that the kernel won't compile without the succeeding patches
in the series is unfortunate. If someone is trying to locate an
unrelated regression via `git bisect' and the bisection process happens
to land them at one of your it-wont-compile points then they have a big
problem. It can pretty much screw up the whole bisection process.

So please try to ensure that the kernel will compile and probably run
at all points of a patch series. If that's too hard then just fold the
separate patches into one larger patch (with appropriately expanded
changelog) to achieve the desired result.


Anyway, I'll tenatively merge these patches into next -mm so they can get a
bit of testing. Which causes a problem, because you don't then have a tree
against which to raise a new patch series.

So perhaps it would be best if you were to

a) Tell me which patches to fold into which other patches to generate a
series which compiles at every stage and

b) Send me a new set of changelogs for the resulting patch series.

OK?
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2006-01-04 01:39    [from the cache]
©2003-2011 Jasper Spaans