[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 18/19] xen/mce: add support of vLMCE injection to XEN_MC_inject_v2



>>> On 23.02.17 at 10:14, <haozhong.zhang@xxxxxxxxx> wrote:
> On 02/23/17 01:26 -0700, Jan Beulich wrote:
>> >>> On 23.02.17 at 06:14, <haozhong.zhang@xxxxxxxxx> wrote:
>> > On 02/22/17 08:59 -0700, Jan Beulich wrote:
>> >> >>> On 17.02.17 at 07:39, <haozhong.zhang@xxxxxxxxx> wrote:
>> >> > @@ -1552,6 +1553,21 @@ long do_mca(XEN_GUEST_HANDLE_PARAM(xen_mc_t) 
> u_xen_mc)
>> >> >                  send_IPI_mask(cpumap, cmci_apic_vector);
>> >> >              }
>> >> >              break;
>> >> > +        case XEN_MC_INJECT_TYPE_LMCE:
>> >> > +            if ( !lmce_support )
>> >> > +            {
>> >> > +                ret = x86_mcerr("No LMCE support in platform", 
>> >> > -EINVAL);
>> >> > +                break;
>> >> > +            }
>> >> > +            /* ensure at most one CPU is specified */
>> >> 
>> >> And what use is none at all? Also - comment style (should start with
>> >> a capital).
>> >>
>> > 
>> > Do you mean the check of empty cpumap? It's checked at the beginning of 
>> > case 
>> > XEN_MC_inject_v2.
>> 
>> To be honest, I don't see any such check. But looking at that code
>> makes me notice you should also forbid the combination of
>> XEN_MC_INJECT_CPU_BROADCAST and XEN_MC_INJECT_TYPE_LMCE.
>>
> 
> I mean the following check. Of course, the additional check you
> suggested must go before it.
>         if ( op->u.mc_inject_v2.flags & XEN_MC_INJECT_CPU_BROADCAST )
>             cpumap = &cpu_online_map;
>         else
>         {
>             ret = xenctl_bitmap_to_cpumask(&cmv, &op->u.mc_inject_v2.cpumap);
>             if ( ret )
>                 break;
>             cpumap = cmv;
>             if ( !cpumask_intersects(cpumap, &cpu_online_map) )  <=== I mean 
> this one exactly

I've been blind, so I'm sorry for the noise.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.