lkml.org 
[lkml]   [2009]   [May]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 22/24] sysfs: Make sysfs_rename_link atomic
    Eric W. Biederman wrote:
    > Tejun Heo <tj@kernel.org> writes:
    >
    >> Eric W. Biederman wrote:
    >>> From: Eric W. Biederman <ebiederm@xmission.com>
    >>>
    >>> Use the existing sysfs_rename to make sysfs_rename_link an atomic
    >>> operation that does less work. While I am at add additional sanity
    >>> checking to ensure it is a symlink I am renaming.
    >>>
    >>> Acked-by: Kay Sievers <kay.sievers@vrfy.org>
    >>> Signed-off-by: Eric W. Biederman <ebiederm@aristanetworks.com>
    >> It would be really nice to merge or group this together with the first
    >> three patches. Other than that,
    >
    > Perfection is the enemy of the good on that one. That just convolutes
    > things unnecessarily, makes the patches harder to review, and requires
    > additional testing.
    >
    > I much prefer to work in a tree without rewinding.

    Everything is a matter of degree and I don't think my bar here was too
    high. Unless patches are in some exported tree and when the content
    of the patchset changes (you're changing the behavior of the same code
    two times), reordering patches is what's usually done. When the end
    results stay the same, reshuffling isn't that much of work, is it?

    Well, I'm not the maintainer and things like this are mostly upto the
    maintainer, so let's leave it to be decided by Greg.

    Thanks.

    --
    tejun


    \
     
     \ /
      Last update: 2009-05-30 12:51    [W:4.165 / U:0.052 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site