[lkml]   [2008]   [Jul]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 72/79] sysdev: Pass the attribute to the low level sysdev show/store function

    * Greg KH <> wrote:

    > On Wed, Jul 23, 2008 at 01:04:35PM +1000, Stephen Rothwell wrote:
    > > Hi Greg,
    > >
    > > On Tue, 22 Jul 2008 14:19:19 -0700 Greg KH <> wrote:
    > > > * Andrew Morton <> wrote:
    > > >
    > > > > This wrecks use-sysdev_class-in-schedc.patch (below), which I
    > > > > merged a week ago and will now drop.
    > > > >
    > > > > Why did this patch from Andi just turn up in linux-next now,
    > > > > halfway through the merge window? It has a commit date of July
    > > > > 1 yet it has never before been sighted in linux-next.
    > > >
    > > > Andi any thoughts?
    > >
    > > The patch
    > > (sysdev-pass-the-attribute-to-the-low-level-sysdev-show-store-function.patch)
    > > has existed in the driver-core part of linux-next since July 4, but the
    > > kernel/sched.c part only got added yesterday.
    > That came from the updated version that Andi sent me to fix the
    > sparc64 build problems the original version had.
    > Stephen, thanks for diging this out.

    ok, lemme do a bit of merge window flaming here, in defense of Andrew.

    This commit history:

    commit 4a0b2b4dbe1335b8b9886ba3dc85a145d5d938ed
    Author: Andi Kleen <>
    AuthorDate: Tue Jul 1 18:48:41 2008 +0200
    Commit: Greg Kroah-Hartman <>
    CommitDate: Mon Jul 21 21:55:02 2008 -0700

    sysdev: Pass the attribute to the low level sysdev show/store function


    I converted all users in tree to the new show/store prototype. It's
    a single huge patch to avoid unbisectable sections.

    Runtime tested: x86-32, x86-64
    Compiled only: ia64, powerpc
    Not compile tested/only grep converted: sh, arm, avr32

    covers a relatively trivial patch that we'd normally not notice, but it
    is ... a ... misrepresentation of the true situation on several levels:

    1) The changelog. The updated patch Andi sent did not declare the other
    incremental changes (to sched.c) it also included freshly.

    2) The date. This patch did not originate on Jul 1 - if Andi sent a
    material update yesterday it should say Jul 21, not Jul 1.

    3) The justification. Huge atomic patches are fine and can indeed be
    much simpler than a gradual switchover, _iff_ they are done
    perfectly. If there's any doubt then they are by far not the only
    option to pursue - we've done finegrained API changeovers for years.

    ... which all we still wouldnt worry much about (the whole change is
    relatively trivial), if it had been done more carefully without wrecking
    Andrew's workflow in the middle of the merge window.

    If every upstream pull - no matter how trivial - caused such work flow
    problems for Andrew (who is sitting at the _end_ of all linux-next stuff
    so is particularly hard hit by last-minute updates - especially if they
    are not append-only) then Andrew would never be able to get his stuff
    from -mm into -git.

    If Andi cannot be bothered to do a proper finegrained workflow _or_ at
    least make it damned sure that he does not mess up other people's work
    via one huge patch modified on the last day (instead of at least sending
    an incremental patch for the last delta) then he should not be doing
    such complex changes.



     \ /
      Last update: 2008-07-23 11:07    [W:0.025 / U:7.876 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site