Re: [Xen-devel] [PATCH v5 5/9] monitor: ARM SMC events

[Julien Grall <julien.grall@xxxxxxx>]

Hello,

On 04/06/2016 18:40, Tamas K Lengyel wrote:
> On Sat, Jun 4, 2016 at 3:03 AM, Edgar E. Iglesias
> <edgar.iglesias@xxxxxxxxx> wrote:
> > Forwarding SMC events for SMC insns that didn't pass the condition tests
> > doesn't make any sense to me. It'll just make the receivers job harder.
> > Why would a receiver want to do anything else than drop these?
> > If it actually does look at them it'll be looking at implementation
> > defined HW behaviour that may vary between CPU implementations.
> If for no other purposes it may be useful to log them to be able to
> study the CPU implementation's behavior.
I cannot see how you will be able to study ARM CPU implementation's 
behavior with VM event. Though I am not familiar with it.
For now, it looks like to me that forwarding conditional SMC even if the 
condition check has failed will require a lot of code in each 
introspection applications, not to mention that they will need specific 
code to distinguish ARMv7 vs ARMv8.
Anyway, I am planning to send a patch to ignore conditional SMCs if the 
condition check has failed because this is the right thing to do.
Regards,

--
Julien Grall

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