On 11/21/19 3:17 PM, Cornelia Huck wrote:
> On Thu, 21 Nov 2019 15:04:31 +0100
> Janosch Frank <fran...@linux.ibm.com> wrote:
>
>> On 11/21/19 2:50 PM, Cornelia Huck wrote:
>>> On Wed, 20 Nov 2019 06:43:26 -0500
>>> Janosch Frank <fran...@linux.ibm.com> wrote:
>>
>>>
>>>> run_on_cpu(cs, s390_do_cpu_load_normal, RUN_ON_CPU_NULL);
>>>> break;
>>>> case S390_RESET_LOAD_NORMAL: /* Subcode 1*/
>>>
>>> missing blank before */ (introduced in a previous patch)
>>>
>>>
>>>> + subsystem_reset();
>>>> + s390_pv_prepare_reset(env);
>>>> CPU_FOREACH(t) {
>>>> if (t == cs) {
>>>> continue;
>>>> }
>>>> run_on_cpu(t, s390_do_cpu_reset, RUN_ON_CPU_NULL);
>>>> }
>>>> - subsystem_reset();
>>>> run_on_cpu(cs, s390_do_cpu_initial_reset, RUN_ON_CPU_NULL);
>>>> run_on_cpu(cs, s390_do_cpu_load_normal, RUN_ON_CPU_NULL);
>>>> break;
>>>> case S390_RESET_EXTERNAL:
>>>
>>> Annotate this with the subcode as well? (in the patch introducing it)
>>
>> I think this has no diag 308 subcode and is triggered by qemu
>
> -ENOCOFFE
>
> But even more reason to annotate this with the fact that this is
> triggered by QEMU :)
> You're too late with that idea :) I just split out the reordering and the annotation into a new commit: https://github.com/frankjaa/qemu/commit/8c53d5c8a6bbcc53496c7a2877c7cbffc435b708
signature.asc
Description: OpenPGP digital signature
