Live data from Hacker News

“Please don't waste maintainers' time on your KPI grabbing patches”

lkml.org

251–260 of 277 posts

Re: “Please don't waste maintainers' time on your KPI grabbing patches”

#251
post #15

Earlier quoted context omitted.

I guess if you have hundreds of trivial patches it'd take ages to review and merge them, and due to kpis Huawei are being deliberately inefficient

Is this an issue with the workflow that the kernel uses in terms of time spent per patch? In most workflows this seems like it'd take under a minute to review, approve and merge each one although the CI/CD might add some computing cost.

It takes more time because they're not just being rubber stamped.

Re: “Please don't waste maintainers' time on your KPI grabbing patches”

#252
post #105

Earlier quoted context omitted.

Hate is a strong word. Dismissing it as simply 'hate' is emotive language (and dishonest, I think) in that it tries to guide the reader to the conclusion that it is completely unreasonable. A more honest word to use would be 'mistrust'

"Hate" is a word that has a variety of meanings, and I was using it as a suffix - it's basically a synonym for '-aggro', a way to describing a pattern of oppositional or antagonistic behavior. (My usage is colloquial. If I were saying what you have mistakenly read it to mean, I would have mentioned hatred of China", or maybe "China hatred".) I am not using it "dishonestly", and I think that the conclusion it guides p…

[deleted]

Re: “Please don't waste maintainers' time on your KPI grabbing patches”

#253
post #129

Earlier quoted context omitted.

KPI = Key Performance Indicator. It's basically a list of things that are considered during your evaluation. The more items off that check list you cross (if they're one-time items, like "ensure an uptime of X"), or the more of each you do (if they're things like "upstream patches" or "patents granted"), the better your score is. It's a shit system that does exactly the opposite of what it's supposed to do IMHO, but…

KPIs are not a shit system, they are actually important to running any nontrivial organization. The issue isn't whether KPIs are good or bad but what you do with them. KPIs don't need to be tied to actual people, usually they are tied to teams, systems, projects, processes, etc. For example, a number of users resigning from our services might be a KPI. KPIs are an important tool to understand what is going on and whe…

I have seen many examples of KPI’s not working as intended. Do you have any real world examples of KPI’s that actually worked in practice (and not just in the make-believe heads of managers)?

Re: “Please don't waste maintainers' time on your KPI grabbing patches”

#254
post #105

Earlier quoted context omitted.

Hate is a strong word. Dismissing it as simply 'hate' is emotive language (and dishonest, I think) in that it tries to guide the reader to the conclusion that it is completely unreasonable. A more honest word to use would be 'mistrust'

"Hate" is a word that has a variety of meanings, and I was using it as a suffix - it's basically a synonym for '-aggro', a way to describing a pattern of oppositional or antagonistic behavior. (My usage is colloquial. If I were saying what you have mistakenly read it to mean, I would have mentioned hatred of China", or maybe "China hatred".) I am not using it "dishonestly", and I think that the conclusion it guides p…

You're right, I read the phrase a little too literally. Apologies!

Re: “Please don't waste maintainers' time on your KPI grabbing patches”

#255
post #241

Earlier quoted context omitted.

I'm not sure what that means, or what basis there is for any of it ? The basis for my comment was Torvalds' own statements and actions, and he's a pretty good source for Torvalds and for effective management and communication in the Linux community. Do you know him personally? Are you a maintainer?

It means that Linus used to be fairly brusk when he was younger (basis: e.g. flipping people off and telling people to "fuck off"), and recently he has apologized for his behavior (basis: the link you shared). To me this is him mellowing out in his old age. His younger self might not have agreed with this mellowing out, seeing it as unnecessary (basis: his responses previous to the apology to other people who called…

> To me ...

My question remains: What is the basis of that? Do you work with him? Read his biography at least (assuming there is one)?

Re: “Please don't waste maintainers' time on your KPI grabbing patches”

#256
post #207

Earlier quoted context omitted.

As others have pointed out, that's speculation. I have trouble understanding why it bothers people so much that Linus is rude sometimes. You don't have to interact with him if you don't want. You can even contribute to the kernel without interacting with him. Whatever he's been doing has been working for going on 3 decades now, I don't see a burning need to change it because it bothers some outsiders. There's this we…

> I have trouble understanding why it bothers people so much that Linus is rude sometimes. Can people actually point to instances where he was rude?

Well...I like Linus just fine as kernel maintainer, but I remember a dozen times (over 2 decades, to be fair) when he was, uhh, very direct. It's often not as aggressive as people make out, but he can definitely get a bit mean when people keep making the same mistakes.

Re: “Please don't waste maintainers' time on your KPI grabbing patches”

#257

Earlier quoted context omitted.

> they tried to fix "borken" to "broken" and the maintainer was not happy […] this does come down to not being familiar with this particular bit of slang - but they push back and argue a bit which doesn't help I did not know "borken" either, but I am aware of "borked" and "broken". Based on that email thread someone else already attempted to fix this in the past. Maybe it's an indication that the so-called "joke" is…

Agreed. While the kernel developers can obviously do whatever they want in terms of in-house jokes, the claim that "this is just normal usage, Google it" is a bit ridiculous. They could have just said, "we know it's wrong, we just thought it's funny". ("Borken" is a city in Germany, that's my only association with it )

[deleted]

Re: “Please don't waste maintainers' time on your KPI grabbing patches”

#258
post #248

Earlier quoted context omitted.

Actually I'm being silly, there is a good solution: Workplace Democracy. You can fool the higher ups come promotion time, but you can't fool all the people all the time. Mondragon please get in the tech biz.

Didn't Donal Trump fool a lot of people in a democracy? And how does this have anything to do with metrics?

No, he didn't fool us.

Re: “Please don't waste maintainers' time on your KPI grabbing patches”

#259
post #248

Earlier quoted context omitted.

Didn't Donal Trump fool a lot of people in a democracy? And how does this have anything to do with metrics?

No, he didn't fool us.

yeah right, he didn't fool anyone

/s

Re: “Please don't waste maintainers' time on your KPI grabbing patches”

#260
post #162
post #38

I have worked in team with some kernel developers at Samsung, years ago, so let me put some perspective for those who do not understand the dynamics. In some companies, the amount of patents or Linux kernel patches you get accepted is direct measure of your success. As you know, whatever you measure becomes a target -- these guys feel very pressed to get ANY kernel commits accepted, no matter how small, irrelevant or…

"When a measure becomes a target, it ceases to be a good measure."

I sometimes wonder if something like this has happened with money.
Post reply on HN