1 |
On Mon, Sep 27, 2021 at 12:23 PM Mike Pagano <mpagano@g.o> wrote: |
2 |
> |
3 |
> On 9/27/21 12:10 PM, Mike Gilbert wrote: |
4 |
> > I'm looking to solicit opinions on when it is appropriate for an |
5 |
> > ebuild to check for kernel config options using linux-info.eclass. I |
6 |
> > don't think we have any guidelines documented, instead leaving it up |
7 |
> > to the "common sense" of package maintainers. |
8 |
> > |
9 |
> > Adding linux-info calls to pkg_pretend or pkg_setup causes slowdowns |
10 |
> > when running emerge, so we should do so only when there is a |
11 |
> > compensating benefit. It doesn't make sense to check for kernel |
12 |
> > options that are very commonly enabled. But what is "very common"? |
13 |
> > |
14 |
> > An obvious example would be CONFIG_INET, which controls IPv4 support |
15 |
> > in the kernel. It does not make sense to check for that in every |
16 |
> > package that uses AF_INET sockets. |
17 |
> > |
18 |
> > A less obvious example: a user filed a bug against net-misc/dhcpcd |
19 |
> > today asking that we check for CONFIG_PACKET [1]. My first thought was |
20 |
> > "why would you ever disable that?". The option description even says |
21 |
> > "if unsure, say Y". However, I suppose it is technically possible to |
22 |
> > run a Linux system with it disabled. |
23 |
> > |
24 |
> > I think a reasonable rule of thumb would be to assume we can rely on |
25 |
> > options that are enabled by "make defconfig". If the user chooses to |
26 |
> > disable them, they are responsible for anything that breaks. |
27 |
> > |
28 |
> > Thoughts? |
29 |
> > |
30 |
> > [1] https://bugs.gentoo.org/815064 |
31 |
> > |
32 |
> |
33 |
> The challenge I see is that these config checks head off bugs and issues without our intervention. |
34 |
> |
35 |
> We as kernel maintainers depend on ebuild maintainers to check these things so they don't become "kernel bugs" to figure out. |
36 |
|
37 |
I guess I'm proposing that we set some minimal standard for kernel |
38 |
configs. It's a waste of time and effort to pepper random ebuilds with |
39 |
checks for options that everyone should have enabled in the first |
40 |
place. |