Am 25.03.2014 08:29, schrieb Alex Bennée: > > Anton Blanchard <[email protected]> writes: > >> Add MSR_LE to the msr_mask for POWER8. >> > <snip> >> - pcc->msr_mask = 0x800000000284FF36ULL; >> + pcc->msr_mask = 0x800000000284FF37ULL; > <snip> > > Should we be adding some #define's for the various bit positions on this > mask? Looking at the current code it looks like a big ream of magic > numbers.
In general I concur that defines would be nice, however for 2.0 that's too risky for me as temporary maintainer and I'm not sure if these values are being pieced together by contributors or whether this is coming directly from the manual? The other issue has been that adding a new family, even after the initial round of cleanups, still requires a chunk of code to be copied, which seems prone to forgetting little bits on the new one, then maybe fixing up the original template but not the derived models, etc. Cheers, Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg
