Re: [Xen-devel] [PATCH v3 10/10] xen/arm: Enable errata for secondary CPU on hotplug after the boot

[Mirela Simonovic <mirela.simonovic@xxxxxxxxxx>]

Hi Julien,

On Thu, May 10, 2018 at 3:29 PM, Julien Grall <julien.grall@xxxxxxx> wrote:
> Hi,
>
> On 05/10/2018 02:24 PM, Mirela Simonovic wrote:
>>
>> On Thu, May 10, 2018 at 1:57 PM, Mirela Simonovic
>> <mirela.simonovic@xxxxxxxxxx> wrote:
>
>
>> I have tested the tuned scenario where enabling capabilities fails -
>> the erroneous CPU is stopped/powered down and the system continues to
>> work fine without it. Although I still don't understand why indeed I
>> needed to debug this I'll add the fix in v4.
>
>
> I don't understand your last sentence. What did you mean?
>

I still don't understand how can this scenario happen practically at
runtime (a scenario without me tuning the failure for testing). Memory
allocation failure shouldn't be possible at this point and any
workaround related failure would have happened at boot. I don't see
anything else or my understanding may not be correct...
However, if some new capability would be introduced one day and it
could fail, it would be good if we don't have to go back and fix this
patch.
At the end, it doesn't matter since the system will be able to
properly deal with such a scenario with the fix.

Thanks,
Mirela

> Cheers,
>
> --
> Julien Grall

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel