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

Re: [Xen-devel] [PATCH v13 for-xen-4.5 11/21] x86/VPMU: Interface for setting PMU mode and flags



>>> On 13.10.14 at 20:57, <boris.ostrovsky@xxxxxxxxxx> wrote:
> On 10/13/2014 09:51 AM, Jan Beulich wrote:
>> So this time round you don't retain any state between retries at all.
>> How is this process expected to ever complete on a large and loaded
>> enough system?
> 
> Since we now leave completion of the mode change request to user we 
> can't guarantee that it will ever happen.

So what is all this good for if there's no guarantee it would ever
complete?

> So if we were to keep the 
> state of who has passed through the context switch we'd have to prevent 
> VPMUs on those CPUs from being loaded again. We don't have such 
> mechanism now (unless we introduce a per-CPU VPMU mode which is not a 
> good idea imo).

Or have a global "vPMU unusable" or "vPMU being turned off" state.

> With continuations we could leave VPMU mode in new state and revert it 
> to old if continuations failed but with things left to user we can't do 
> this.
> 
> This has been the case for the last few versions btw.

You mean the user space invoked continuations? Yes, but v12
still had the retaining of some global state, so what you do now
is yet another variant. Anyway - I don't think this can be left as
is, and it's not really clear why you thought this would be better
than the earlier variants.

Jan


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


 


Rackspace

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