[Skiboot] [PATCH] phb4/capp: Update the expected Eye-catcher for CAPP ucode lid
Vaibhav Jain
vaibhav at linux.ibm.com
Mon Sep 3 18:52:55 AEST 2018
Thanks for reviewing this patch Andrew,
Andrew Donnellan <andrew.donnellan at au1.ibm.com> writes:
> On 03/09/18 17:43, Vaibhav Jain wrote:
>> Currently on P9 load_capp_code() expects CAPP ucode lid header to have
>> eye-catcher magic of 'CAPPPSLL'. However the ucode lid available on a
>> FSP base system has eye-catcher magic of 'CAPPLIDH'. This prevents
>> skiboot from loading the ucode with this error message:
>>
>> CAPP: ucode header invalid
>>
>> We fix this issue by updating load_capp_ucode() to use the eye-catcher
>> value of 'CAPPLIDH' instead of 'CAPPPSLL'.
>>
>> Signed-off-by: Vaibhav Jain <vaibhav at linux.ibm.com>
>
> Should this go to stable?
Yes, will resend marking it to stable.
> Is there anywhere where this change is documented? What about non-FSP
> systems?
Yes, as discussed offline CAPP ucode only lids for FSP based systems
have CAPPLIDH as the header magic. Hence this needs to be changed.
Openpower based systems dont use a lid and pnor partition data
containing capp ucode has magic 'CAPPULID'. So this condition is already
handled in capp_load_ucode() and openpower based systems are not
effected by this patch.
--
Vaibhav Jain <vaibhav at linux.vnet.ibm.com>
Linux Technology Center, IBM India Pvt. Ltd.
More information about the Skiboot
mailing list