[lkml]   [2009]   [Apr]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 1/1] vfs: umount_begin BKL pushdown v2

    * Christoph Hellwig <> wrote:

    > > You've also not explained why you have done it this way. It
    > > would cost you almost nothing to apply these bits into a
    > > separate branch and merge that branch into your main tree. Lots
    > > of other maintainer are doing that.
    > Having a separate kill the BKL tree is a stupid idea. Locking
    > changes need deep subsystem knowledge and should always go through
    > the subsystem trees.

    Here you are missing the small inconvenient fact that having the
    kill-the-BKL tree is what got this work underway. It is what got
    developers interested, it is what is concentrating the effort, and
    it is that is producing the patches.

    _Nobody_ ever suggested that VFS patches should not go upstream via
    the VFS tree. We are _happy_ that BKL removal patches are finally
    flowing through the VFS tree.

    The _only_ very minimal courtesy i was asking for was to also be
    'allowed' to carry those fixes that we WROTE, with the same commit
    ID - so that if the kill-the-BKL tree goes upstream sometime in the
    (apparently far) future (well after the VFS bits go upstream), it
    will look nice and wont have duplicate commits. We are patient, and
    we'd like to maintain a tidy tree.

    But i didnt even get a _reply_ to that initial request - Al just
    committed it straight into the VFS tree and ignored my question
    somewhat rudely.

    The thing is, for years you never cared about the BKL being deep
    embedded in the guts of the VFS. But the minute someone _else_ does
    what arguably you should have done long ago, you stand in the way
    and hinder that effort by first proclaiming that this tree should
    not be doing such changes and then forcing it into an ugly (future)

    Exactly how does such kind of behavior help Linux, in your opinion?


     \ /
      Last update: 2009-04-25 00:53    [W:0.054 / U:2.604 seconds]
    ©2003-2017 Jasper Spaans. hosted at Digital OceanAdvertise on this site