Re: [PATCH v4 05/10] sched/topology: Define and assign sched_domain flag metadata

2020-08-07 Thread Valentin Schneider
On 06/08/20 17:18, Valentin Schneider wrote: > In the grand scheme of things I'd actually like to have this file output > the names of the flags rather than their values (since I now save them when > SCHED_DEBUG=y), but I didn't find a simple way to hack the existing SD ctl > table

Re: [PATCH v4 05/10] sched/topology: Define and assign sched_domain flag metadata

2020-08-06 Thread Ingo Molnar
* Valentin Schneider wrote: > +#ifndef SD_FLAG > +#define SD_FLAG(x, y, z) > +#endif AFAICS there's not a single use of sd_flags.h that doesn't come with its own SD_FLAG definition, so I suppose this should be: #ifndef SD_FLAG # error "Should not happen." #endif ? Also, some nits: > +/*

Re: [PATCH v4 05/10] sched/topology: Define and assign sched_domain flag metadata

2020-08-06 Thread Valentin Schneider
On 06/08/20 15:07, Ingo Molnar wrote: > * Valentin Schneider wrote: > >> +#ifndef SD_FLAG >> +#define SD_FLAG(x, y, z) >> +#endif > > AFAICS there's not a single use of sd_flags.h that doesn't come with > its own SD_FLAG definition, so I suppose this should be: > > #ifndef SD_FLAG > # error

Re: [PATCH v4 05/10] sched/topology: Define and assign sched_domain flag metadata

2020-08-04 Thread peterz
On Fri, Jul 31, 2020 at 12:54:57PM +0100, Valentin Schneider wrote: > +/* > + * Domain members share CPU capacity (i.e. SMT) > + * > + * SHARED_CHILD: Set from the base domain up until spanned CPUs no longer > share > + * CPU capacity. > + */ > +SD_FLAG(SD_SHARE_CPUCAPACITY, 6,

Re: [PATCH v4 05/10] sched/topology: Define and assign sched_domain flag metadata

2020-08-04 Thread Valentin Schneider
On 04/08/20 12:08, pet...@infradead.org wrote: > On Fri, Jul 31, 2020 at 12:54:57PM +0100, Valentin Schneider wrote: >> +/* >> + * Domain members share CPU capacity (i.e. SMT) >> + * >> + * SHARED_CHILD: Set from the base domain up until spanned CPUs no longer >> share >> + * CPU capacity. >> +

[PATCH v4 05/10] sched/topology: Define and assign sched_domain flag metadata

2020-07-31 Thread Valentin Schneider
There are some expectations regarding how sched domain flags should be laid out, but none of them are checked or asserted in sched_domain_debug_one(). After staring at said flags for a while, I've come to realize they all (except *one*) fall in either of two categories: - Shared with children: