Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Juergen Gross
On 11/03/16 13:57, Peter Zijlstra wrote: > On Fri, Mar 11, 2016 at 01:48:12PM +0100, Juergen Gross wrote: >> On 11/03/16 13:42, Peter Zijlstra wrote: >>> how about something like: >>> >>> struct xen_callback_struct { >>> struct work_struct work; >>> struct completion done; >

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Peter Zijlstra
On Fri, Mar 11, 2016 at 01:48:12PM +0100, Juergen Gross wrote: > On 11/03/16 13:42, Peter Zijlstra wrote: > > how about something like: > > > > struct xen_callback_struct { > > struct work_struct work; > > struct completion done; int (*func)(void*); >

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Juergen Gross
On 11/03/16 13:42, Peter Zijlstra wrote: > On Fri, Mar 11, 2016 at 01:19:50PM +0100, Peter Zijlstra wrote: >> On Fri, Mar 11, 2016 at 12:59:30PM +0100, Juergen Gross wrote: >>> +int call_sync_on_phys_cpu(unsigned cpu, int (*func)(void *), void *par) >>> +{ >>> + cpumask_var_t old_mask; >>> + in

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Peter Zijlstra
On Fri, Mar 11, 2016 at 01:43:53PM +0100, Juergen Gross wrote: > On 11/03/16 13:19, Peter Zijlstra wrote: > > On Fri, Mar 11, 2016 at 12:59:30PM +0100, Juergen Gross wrote: > >> +int call_sync_on_phys_cpu(unsigned cpu, int (*func)(void *), void *par) > >> +{ > >> + cpumask_var_t old_mask; > >> +

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Juergen Gross
On 11/03/16 13:19, Peter Zijlstra wrote: > On Fri, Mar 11, 2016 at 12:59:30PM +0100, Juergen Gross wrote: >> +int call_sync_on_phys_cpu(unsigned cpu, int (*func)(void *), void *par) >> +{ >> +cpumask_var_t old_mask; >> +int ret; >> + >> +if (cpu >= nr_cpu_ids) >> +return -EI

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Peter Zijlstra
On Fri, Mar 11, 2016 at 01:19:50PM +0100, Peter Zijlstra wrote: > On Fri, Mar 11, 2016 at 12:59:30PM +0100, Juergen Gross wrote: > > +int call_sync_on_phys_cpu(unsigned cpu, int (*func)(void *), void *par) > > +{ > > + cpumask_var_t old_mask; > > + int ret; > > + > > + if (cpu >= nr_cpu_ids)

Re: [Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Peter Zijlstra
On Fri, Mar 11, 2016 at 12:59:30PM +0100, Juergen Gross wrote: > +int call_sync_on_phys_cpu(unsigned cpu, int (*func)(void *), void *par) > +{ > + cpumask_var_t old_mask; > + int ret; > + > + if (cpu >= nr_cpu_ids) > + return -EINVAL; > + > + if (!alloc_cpumask_var(&old_

[Xen-devel] [PATCH 2/6] sched: add function to execute a function synchronously on a physical cpu

2016-03-11 Thread Juergen Gross
On some hardware models (e.g. Dell Studio 1555 laptop) some hardware related functions (e.g. SMIs) are to be executed on physical cpu 0 only. Instead of open coding such a functionality multiple times in the kernel add a service function for this purpose. This will enable the possibility to take sp