Re: [Xen-devel] [PATCH] xen: add hypercall option to temporarily pin a vcpu

2016-02-26 Thread Dario Faggioli
On Fri, 2016-02-26 at 06:32 -0700, Jan Beulich wrote: > > > > On 26.02.16 at 14:07, wrote: > > We can argue about that being useful or not, and about it being > > (potentially) too noisy or not. I personally think it could be > > useful > > (it's XENLOG_DEBUG, after

Re: [Xen-devel] [PATCH] xen: add hypercall option to temporarily pin a vcpu

2016-02-26 Thread Jan Beulich
>>> On 26.02.16 at 13:49, wrote: > On 26/02/16 13:39, Jan Beulich wrote: > On 26.02.16 at 12:20, wrote: >>> On Fri, 2016-02-26 at 12:14 +0100, Juergen Gross wrote: On 26/02/16 11:39, Jan Beulich wrote: > >> @@ -670,7 +676,13 @@ int

Re: [Xen-devel] [PATCH] xen: add hypercall option to temporarily pin a vcpu

2016-02-26 Thread Jan Beulich
>>> On 26.02.16 at 14:07, wrote: > On Fri, 2016-02-26 at 05:39 -0700, Jan Beulich wrote: >> > > > On 26.02.16 at 12:20, wrote: >> > On Fri, 2016-02-26 at 12:14 +0100, Juergen Gross wrote: >> > > > Wouldn't it be even better to make this the

Re: [Xen-devel] [PATCH] xen: add hypercall option to temporarily pin a vcpu

2016-02-26 Thread Dario Faggioli
On Fri, 2016-02-26 at 05:39 -0700, Jan Beulich wrote: > >  > > > > On 26.02.16 at 12:20, wrote: > > On Fri, 2016-02-26 at 12:14 +0100, Juergen Gross wrote: > > >  > > > EBUSY vs. EAGAIN: by returning EAGAIN I would signal to Xen tools > > > that > > > the hypervisor is

Re: [Xen-devel] [PATCH] xen: add hypercall option to temporarily pin a vcpu

2016-02-26 Thread Juergen Gross
On 26/02/16 13:39, Jan Beulich wrote: On 26.02.16 at 12:20, wrote: >> On Fri, 2016-02-26 at 12:14 +0100, Juergen Gross wrote: >>> On 26/02/16 11:39, Jan Beulich wrote: > @@ -670,7 +676,13 @@ int cpu_disable_scheduler(unsigned int cpu) > if

Re: [Xen-devel] [PATCH] xen: add hypercall option to temporarily pin a vcpu

2016-02-26 Thread Jan Beulich
>>> On 26.02.16 at 12:20, wrote: > On Fri, 2016-02-26 at 12:14 +0100, Juergen Gross wrote: >> On 26/02/16 11:39, Jan Beulich wrote: >> > >> > > @@ -670,7 +676,13 @@ int cpu_disable_scheduler(unsigned int cpu) >> > > if ( cpumask_empty(_affinity) && >> > >

Re: [Xen-devel] [PATCH] xen: add hypercall option to temporarily pin a vcpu

2016-02-26 Thread Juergen Gross
On 26/02/16 12:20, Dario Faggioli wrote: > On Fri, 2016-02-26 at 12:14 +0100, Juergen Gross wrote: >> On 26/02/16 11:39, Jan Beulich wrote: @@ -679,6 +691,8 @@ int cpu_disable_scheduler(unsigned int cpu) v->affinity_broken = 1; } +

Re: [Xen-devel] [PATCH] xen: add hypercall option to temporarily pin a vcpu

2016-02-26 Thread Dario Faggioli
On Fri, 2016-02-26 at 12:14 +0100, Juergen Gross wrote: > On 26/02/16 11:39, Jan Beulich wrote: > >  > > > @@ -670,7 +676,13 @@ int cpu_disable_scheduler(unsigned int cpu) > > >  if ( cpumask_empty(_affinity) && > > >   cpumask_test_cpu(cpu, v->cpu_hard_affinity) ) > >

Re: [Xen-devel] [PATCH] xen: add hypercall option to temporarily pin a vcpu

2016-02-26 Thread Juergen Gross
On 26/02/16 11:39, Jan Beulich wrote: On 25.02.16 at 17:50, wrote: >> @@ -670,7 +676,13 @@ int cpu_disable_scheduler(unsigned int cpu) >> if ( cpumask_empty(_affinity) && >> cpumask_test_cpu(cpu, v->cpu_hard_affinity) ) >> { >> -

Re: [Xen-devel] [PATCH] xen: add hypercall option to temporarily pin a vcpu

2016-02-26 Thread Jan Beulich
>>> On 25.02.16 at 17:50, wrote: > @@ -670,7 +676,13 @@ int cpu_disable_scheduler(unsigned int cpu) > if ( cpumask_empty(_affinity) && > cpumask_test_cpu(cpu, v->cpu_hard_affinity) ) > { > -printk(XENLOG_DEBUG "Breaking

[Xen-devel] [PATCH] xen: add hypercall option to temporarily pin a vcpu

2016-02-25 Thread Juergen Gross
Some hardware (e.g. Dell studio 1555 laptops) require SMIs to be called on physical cpu 0 only. Linux drivers like dcdbas or i8k try to achieve this by pinning the running thread to cpu 0, but in Dom0 this is not enough: the vcpu must be pinned to physical cpu 0 via Xen, too. Add a stable