diff for duplicates of <1524697697.4100.23.camel@HansenPartnership.com> diff --git a/a/1.txt b/N1/1.txt index 6aa7f95..ef8ceea 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -2,37 +2,37 @@ On Wed, 2018-04-25 at 19:00 -0400, Mikulas Patocka wrote: > > On Wed, 25 Apr 2018, James Bottomley wrote: > -> > > > Do we really need the new config option? This could just be -> > > > manually tunable via fault injection IIUC. -> > > +> > > > Do we really need the new config option?A A This could just be +> > > > manuallyA tunable via fault injection IIUC. +> > >A > > > We do, because we want to enable it in RHEL and Fedora debugging -> > > kernels, so that it will be tested by the users. -> > > +> > > kernels,A so that it will be tested by the users. +> > >A > > > The users won't use some extra magic kernel options or debugfs > files. -> > +> >A > > If it can be enabled via a tunable, then the distro can turn it on -> > without the user having to do anything. If you want to present the +> > without the user having to do anything.A If you want to present the > > user with a different boot option, you can (just have the tunable > set > > on the command line), but being tunable driven means that you don't > > have to choose that option, you could automatically enable it under > a -> > range of circumstances. I think most sane distributions would want +> > range of circumstances.A I think most sane distributions would want > > that flexibility. -> > +> >A > > Kconfig proliferation, conversely, is a bit of a nightmare from > both > > the user and the tester's point of view, so we're trying to avoid > it > > unless absolutely necessary. -> > +> >A > > James > > BTW. even developers who compile their own kernel should have this -> enabled by a CONFIG option - because if the developer sees the option -> when browsing through menuconfig, he may enable it. If he doesn't see -> the option, he won't even know that such an option exists. +> enabledA by a CONFIG option - because if the developer sees the option +> whenA browsing through menuconfig, he may enable it. If he doesn't see +> theA option, he won't even know that such an option exists. I may be an atypical developer but I'd rather have a root canal than browse through menuconfig options. The way to get people to learn diff --git a/a/content_digest b/N1/content_digest index f317fa0..9e3dd68 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -42,37 +42,37 @@ "> \n" "> On Wed, 25 Apr 2018, James Bottomley wrote:\n" "> \n" - "> > > > Do we really need the new config option?\302\240\302\240This could just be\n" - "> > > > manually\302\240 tunable via fault injection IIUC.\n" - "> > >\302\240\n" + "> > > > Do we really need the new config option?A A This could just be\n" + "> > > > manuallyA tunable via fault injection IIUC.\n" + "> > >A \n" "> > > We do, because we want to enable it in RHEL and Fedora debugging\n" - "> > > kernels,\302\240so that it will be tested by the users.\n" - "> > >\302\240\n" + "> > > kernels,A so that it will be tested by the users.\n" + "> > >A \n" "> > > The users won't use some extra magic kernel options or debugfs\n" "> files.\n" - "> >\302\240\n" + "> >A \n" "> > If it can be enabled via a tunable, then the distro can turn it on\n" - "> > without the user having to do anything.\302\240 If you want to present the\n" + "> > without the user having to do anything.A If you want to present the\n" "> > user with a different boot option, you can (just have the tunable\n" "> set\n" "> > on the command line), but being tunable driven means that you don't\n" "> > have to choose that option, you could automatically enable it under\n" "> a\n" - "> > range of circumstances.\302\240 I think most sane distributions would want\n" + "> > range of circumstances.A I think most sane distributions would want\n" "> > that flexibility.\n" - "> >\302\240\n" + "> >A \n" "> > Kconfig proliferation, conversely, is a bit of a nightmare from\n" "> both\n" "> > the user and the tester's point of view, so we're trying to avoid\n" "> it\n" "> > unless absolutely necessary.\n" - "> >\302\240\n" + "> >A \n" "> > James\n" "> \n" "> BTW. even developers who compile their own kernel should have this\n" - "> enabled\302\240by a CONFIG option - because if the developer sees the option\n" - "> when\302\240browsing through menuconfig, he may enable it. If he doesn't see\n" - "> the\302\240option, he won't even know that such an option exists.\n" + "> enabledA by a CONFIG option - because if the developer sees the option\n" + "> whenA browsing through menuconfig, he may enable it. If he doesn't see\n" + "> theA option, he won't even know that such an option exists.\n" "\n" "I may be an atypical developer but I'd rather have a root canal than\n" "browse through menuconfig options. The way to get people to learn\n" @@ -83,4 +83,4 @@ "\n" James -d95cd72b8fb0a9952c4f2ee181efd0c4430698c9b50083573636d57119af1164 +e33ce645ae68b70046264e074ee24b0dc7a51c003dbbe4b229b513728dc1fc74
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.