3 ms·

Not sure if there was much more to it than the thread linked to, but it was basically Linus being Linus. He said stuff that made sense in a pretty blunt fashion

Not sure if there was much more to it than the thread linked to, but it was basically Linus being Linus. He said stuff that made sense in a pretty blunt fashion. There were flags proposed that allowed the seal to be ignored. >So you say "we can't munmap in this *one* place, but all others ignore the sealing". Later was the spice. >And dammit, once something is sealed, it is SEALED. None of this crazy "one place honors the sealing, random other places do not". And later, even spicier, Linus says that seals cannot be ignored and that is non-negotiable. Any further suggestions to ignore a seal via a flag would result in the person being added to Linus' ignore list. (He, of course, said this with some profanities and capitals sprinkled in.)

js2 2y ago
Wasn't just Linus. Earlier, from Theo de Raadt: > I don't think you understand the problem space well enough to come up with your own solution for it. I spent a year on this, and ship a complete system using it. You are asking such simplistic questions above it shocks me. https://lwn.net/ml/linux-kernel/95482.1697587015@cvs.openbsd.org/ https://lwn.net/ml/linux-kernel/95482.1697587015@cvs.openbsd... Via https://lwn.net/Articles/948129/ https://lwn.net/Articles/948129/
- 0xbadcafebee 2y ago
  Not a great perspective... "It took me a year [or more] to understand this. The fact that you don't understand it shocks me." Dude, not everybody's as smart or experienced as you. Here's an opportunity to be a mentor.
  - nativeit 2y ago
    > Google has no shortage of experienced developers who could have reviewed this submission before it was posted publicly, but that does not appear to have happened, with the result that a relatively inexperienced developer was put into a difficult position. Feedback on the proposal was resisted rather than listened to. The result was an interaction that pleased nobody.
    - refulgentis 2y ago
      > Google has no shortage of experienced developers who could have reviewed this submission before it was posted publicly, You'd be surprised. My understanding from folks on Chrome OS is they've already shedded most, if not all, of the most experienced old hands. (n.b. Chrome OS was absorbed by Android and development is, by and large, ceased on it according to same sources directly, and indirectly via Blind.)
  - vlovich123 2y ago
    My reading of this is a lot more generous to the maintainers and a lot less sympathetic to the author than yours is. The maintainers highlighted the problems and the author came back basically with "I don't believe you so let's go with my approach to stay more general" - it's one thing to disagree, it's another to straight up not acknowledge the feedback. The author ate a lot of very senior people's time arguing instead of listening to them and learning from their experience and that was justifiably frustrating forcing much more direct feedback. The kind of mistake the author made - having to enforce at each individual syscall level instead of it being a protection on the memory itself enforced on all accesses - indicates a poor understanding of how to think about security and build security APIs which is a problem when you're proposing a security API. It's particularly impressive how misguided the patch is given that they took inspiration from the OpenBSD API implementation, changed both API & implementation, & then argued with both Linus and Theo who started Linux & OpenBSD respectively and were trying to give direct feedback about how OpenBSD is different and why it took the approach it did. Hopefully the author has taken the more forceful feedback as a learning opportunity about listening to feedback when the people giving it to you have a lot more experience. Or their team is coaching them about what went wrong now that this became so visible to learn what they got wrong. From Matthew Wilcox who is another senior Linux maintainer: > I concur with Theo & Linus. You don't know what you're doing. I think the underlying idea of mimmutable() is good, but how you've split it up and how you've implemented it is terrible. It's delivered directly and bluntly but it's not mean or personal. The author proposed a bad patch & argued from a position of ignorance.
    - jorvi 2y ago
      > The maintainers highlighted the problems and the author came back basically with "I don't believe you so let's go with my approach to stay more general" - it's one thing to disagree, it's another to straight up not acknowledge the feedback. Isn't that the Linux kernel in a nutshell? Of the top of my head I can name: - The zram maintainer that's for a few years been blocking the patches that add the zpool api to zram. This would go a long way to unifying zram and zswap in the future. - TuxOnIce being permablocked by some heels-in-sand maintainers until the kernel diverged to much and the patch writer gave up. This one would have fixed hibernation on Linux.. in other words, hibernation still sucks because of these maintainers. - Of course the trench warfare all the C-proficient maintainers are waging to chase all the Rust-in-Linux people away, lest they have to learn Rust. ~80% of CVEs are memory-related, so you could say that in say.. 10 years time, ~80% of the CVEs happening in the Linux kernel are the legacy of Ted & co.
  - terribleperson 2y ago
    The time of the people who maintain the free and open source software we rely on is not free. From the people I've talked to, maintainers of successful projects are overworked and underappreciated. Mentorship from one of those people would be valuable, but arguing with them about the implementation of something you don't understand isn't how you get that mentorship.
    - Onavo 2y ago
      > Mentorship from one of those people would be valuable, but arguing with them about the implementation of something you don't understand isn't how you get that mentorship. Actually that's exactly how you can provoke them into explaining themselves. Too many experienced people sit in their ivory towers and basically appeal to their authority, "I wrote a kernel therefore I know everything, <insert profanity here>". There's a reason why Cunningham's Law exist, "The best way to get the right answer on the Internet is not to ask a question, it's to post the wrong answer."
- deleted 2y ago
  [deleted]
- Affric 2y ago
  Thank you. That was beautiful. Demonstrated the difference in design/engineering philosophies from two of the greats.
- benreesman 2y ago
  You’re asking me how a watch works, let’s just try to keep an eye on the time. https://youtu.be/vkYqs9iuJqY?feature=shared&t=109 https://youtu.be/vkYqs9iuJqY?feature=shared&t=109
santiagobasulto 2y ago
Is that considered "spicy"? Is the sensitivity threshold maybe too low?
- f1shy 2y ago
  Extremely too low. My personal opinion: people who cannot take that kind of criticism, has no place in such a project. Period.
f1shy 2y ago
And the reason why SW development sucks in enterprise, is the lack of people that can speak clearly like Linus.

js2 2y ago

Wasn't just Linus. Earlier, from Theo de Raadt: > I don't think you understand the problem space well enough to come up with your own solution for it. I spent a year on this, and ship a complete system using it. You are asking such simplistic questions above it shocks me. https://lwn.net/ml/linux-kernel/95482.1697587015@cvs.openbsd.org/ https://lwn.net/ml/linux-kernel/95482.1697587015@cvs.openbsd... Via https://lwn.net/Articles/948129/ https://lwn.net/Articles/948129/

0xbadcafebee 2y ago
Not a great perspective... "It took me a year [or more] to understand this. The fact that you don't understand it shocks me." Dude, not everybody's as smart or experienced as you. Here's an opportunity to be a mentor.
- nativeit 2y ago
  > Google has no shortage of experienced developers who could have reviewed this submission before it was posted publicly, but that does not appear to have happened, with the result that a relatively inexperienced developer was put into a difficult position. Feedback on the proposal was resisted rather than listened to. The result was an interaction that pleased nobody.
  - refulgentis 2y ago
    > Google has no shortage of experienced developers who could have reviewed this submission before it was posted publicly, You'd be surprised. My understanding from folks on Chrome OS is they've already shedded most, if not all, of the most experienced old hands. (n.b. Chrome OS was absorbed by Android and development is, by and large, ceased on it according to same sources directly, and indirectly via Blind.)
- vlovich123 2y ago
  My reading of this is a lot more generous to the maintainers and a lot less sympathetic to the author than yours is. The maintainers highlighted the problems and the author came back basically with "I don't believe you so let's go with my approach to stay more general" - it's one thing to disagree, it's another to straight up not acknowledge the feedback. The author ate a lot of very senior people's time arguing instead of listening to them and learning from their experience and that was justifiably frustrating forcing much more direct feedback. The kind of mistake the author made - having to enforce at each individual syscall level instead of it being a protection on the memory itself enforced on all accesses - indicates a poor understanding of how to think about security and build security APIs which is a problem when you're proposing a security API. It's particularly impressive how misguided the patch is given that they took inspiration from the OpenBSD API implementation, changed both API & implementation, & then argued with both Linus and Theo who started Linux & OpenBSD respectively and were trying to give direct feedback about how OpenBSD is different and why it took the approach it did. Hopefully the author has taken the more forceful feedback as a learning opportunity about listening to feedback when the people giving it to you have a lot more experience. Or their team is coaching them about what went wrong now that this became so visible to learn what they got wrong. From Matthew Wilcox who is another senior Linux maintainer: > I concur with Theo & Linus. You don't know what you're doing. I think the underlying idea of mimmutable() is good, but how you've split it up and how you've implemented it is terrible. It's delivered directly and bluntly but it's not mean or personal. The author proposed a bad patch & argued from a position of ignorance.
  - jorvi 2y ago
    > The maintainers highlighted the problems and the author came back basically with "I don't believe you so let's go with my approach to stay more general" - it's one thing to disagree, it's another to straight up not acknowledge the feedback. Isn't that the Linux kernel in a nutshell? Of the top of my head I can name: - The zram maintainer that's for a few years been blocking the patches that add the zpool api to zram. This would go a long way to unifying zram and zswap in the future. - TuxOnIce being permablocked by some heels-in-sand maintainers until the kernel diverged to much and the patch writer gave up. This one would have fixed hibernation on Linux.. in other words, hibernation still sucks because of these maintainers. - Of course the trench warfare all the C-proficient maintainers are waging to chase all the Rust-in-Linux people away, lest they have to learn Rust. ~80% of CVEs are memory-related, so you could say that in say.. 10 years time, ~80% of the CVEs happening in the Linux kernel are the legacy of Ted & co.
- terribleperson 2y ago
  The time of the people who maintain the free and open source software we rely on is not free. From the people I've talked to, maintainers of successful projects are overworked and underappreciated. Mentorship from one of those people would be valuable, but arguing with them about the implementation of something you don't understand isn't how you get that mentorship.
  - Onavo 2y ago
    > Mentorship from one of those people would be valuable, but arguing with them about the implementation of something you don't understand isn't how you get that mentorship. Actually that's exactly how you can provoke them into explaining themselves. Too many experienced people sit in their ivory towers and basically appeal to their authority, "I wrote a kernel therefore I know everything, <insert profanity here>". There's a reason why Cunningham's Law exist, "The best way to get the right answer on the Internet is not to ask a question, it's to post the wrong answer."
deleted 2y ago
[deleted]
Affric 2y ago
Thank you. That was beautiful. Demonstrated the difference in design/engineering philosophies from two of the greats.
benreesman 2y ago
You’re asking me how a watch works, let’s just try to keep an eye on the time. https://youtu.be/vkYqs9iuJqY?feature=shared&t=109 https://youtu.be/vkYqs9iuJqY?feature=shared&t=109

santiagobasulto 2y ago

Is that considered "spicy"? Is the sensitivity threshold maybe too low?

f1shy 2y ago
Extremely too low. My personal opinion: people who cannot take that kind of criticism, has no place in such a project. Period.

f1shy 2y ago

And the reason why SW development sucks in enterprise, is the lack of people that can speak clearly like Linus.