lkml.org 
[lkml]   [2020]   [Nov]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH 000/141] Fix fall-through warnings for Clang
    On Mon, Nov 23, 2020 at 4:58 PM James Bottomley
    <James.Bottomley@hansenpartnership.com> wrote:
    >
    > Well, I used git. It says that as of today in Linus' tree we have 889
    > patches related to fall throughs and the first series went in in
    > october 2017 ... ignoring a couple of outliers back to February.

    I can see ~10k insertions over ~1k commits and 15 years that mention a
    fallthrough in the entire repo. That is including some commits (like
    the biggest one, 960 insertions) that have nothing to do with C
    fallthrough. A single kernel release has an order of magnitude more
    changes than this...

    But if we do the math, for an author, at even 1 minute per line change
    and assuming nothing can be automated at all, it would take 1 month of
    work. For maintainers, a couple of trivial lines is noise compared to
    many other patches.

    In fact, this discussion probably took more time than the time it
    would take to review the 200 lines. :-)

    > We're also complaining about the inability to recruit maintainers:
    >
    > https://www.theregister.com/2020/06/30/hard_to_find_linux_maintainers_says_torvalds/
    >
    > And burn out:
    >
    > http://antirez.com/news/129

    Accepting trivial and useful 1-line patches is not what makes a
    voluntary maintainer quit... Thankless work with demanding deadlines is.

    > The whole crux of your argument seems to be maintainers' time isn't
    > important so we should accept all trivial patches

    I have not said that, at all. In fact, I am a voluntary one and I
    welcome patches like this. It takes very little effort on my side to
    review and it helps the kernel overall. Paid maintainers are the ones
    that can take care of big features/reviews.

    > What I'm actually trying to articulate is a way of measuring value of
    > the patch vs cost ... it has nothing really to do with who foots the
    > actual bill.

    I understand your point, but you were the one putting it in terms of a
    junior FTE. In my view, 1 month-work (worst case) is very much worth
    removing a class of errors from a critical codebase.

    > One thesis I'm actually starting to formulate is that this continual
    > devaluing of maintainers is why we have so much difficulty keeping and
    > recruiting them.

    That may very well be true, but I don't feel anybody has devalued
    maintainers in this discussion.

    Cheers,
    Miguel

    \
     
     \ /
      Last update: 2020-11-23 20:06    [W:4.090 / U:22.372 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site