[Skiboot] [PATCH v2 02/12] hdat: Add processor version check for hs service ntuple
Oliver O'Halloran
oohall at gmail.com
Tue Dec 20 15:05:41 AEDT 2016
On Tue, Dec 20, 2016 at 2:34 PM, Stewart Smith
<stewart at linux.vnet.ibm.com> wrote:
> Vasant Hegde <hegdevasant at linux.vnet.ibm.com> writes:
>> Host services ntuple is deprecated on P9.
>>
>> Signed-off-by: Vasant Hegde <hegdevasant at linux.vnet.ibm.com>
>> ---
>> hdata/spira.c | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/hdata/spira.c b/hdata/spira.c
>> index 592197e..59ffc1d 100644
>> --- a/hdata/spira.c
>> +++ b/hdata/spira.c
>> @@ -1023,6 +1023,10 @@ static void hostservices_parse(void)
>> unsigned int size;
>> unsigned int ntuples_size;
>>
>> + /* Deprecated on P9 */
>> + if (proc_gen >= proc_gen_p9)
>> + return;
>> +
>
> I'd prefer we print an error rather than failing silently.
Why? Ignoring a deprecated tuple is perfectly fine and the FSP/HB
sometimes populate the fields anyway.
>
> --
> Stewart Smith
> OPAL Architect, IBM.
>
More information about the Skiboot
mailing list