Schedule affinity_notify work while migrating IRQs during hot plug

2017-03-20 Thread Prasad Sodagudi
irq_do_set_affinity() last argument differentiates whether notify work need to queued for this irq or not. So that we can avoid double queuing of notify in the irq_set_affinity_locked() path.

Re: Schedule affinity_notify work while migrating IRQs during hot plug

2017-03-17 Thread Thomas Gleixner
On Fri, 17 Mar 2017, Sodagudi Prasad wrote: > On 2017-03-13 13:19, Thomas Gleixner wrote: > > Can you actually see the difference between these functions? There is a > > damned good reason WHY this calls irq_do_set_affinity(). > > Other option is that, adding an argument to irq_do_set_affinity() a

Re: Schedule affinity_notify work while migrating IRQs during hot plug

2017-03-17 Thread Sodagudi Prasad
On 2017-03-13 13:19, Thomas Gleixner wrote: On Mon, 13 Mar 2017, Sodagudi Prasad wrote: On 2017-02-27 09:21, Thomas Gleixner wrote: > On Mon, 27 Feb 2017, Sodagudi Prasad wrote: > > So I am thinking that, adding following sched_work() would notify clients. > > And break the world and some more.

Re: Schedule affinity_notify work while migrating IRQs during hot plug

2017-03-13 Thread Thomas Gleixner
On Mon, 13 Mar 2017, Sodagudi Prasad wrote: > On 2017-02-27 09:21, Thomas Gleixner wrote: > > On Mon, 27 Feb 2017, Sodagudi Prasad wrote: > > > So I am thinking that, adding following sched_work() would notify clients. > > > > And break the world and some more. > > > > > diff --git a/kernel/irq/m

Re: Schedule affinity_notify work while migrating IRQs during hot plug

2017-03-13 Thread Sodagudi Prasad
On 2017-02-27 09:21, Thomas Gleixner wrote: On Mon, 27 Feb 2017, Sodagudi Prasad wrote: So I am thinking that, adding following sched_work() would notify clients. And break the world and some more. diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c index 6b66959..5e4766b 100644 --- a/ker

Re: Schedule affinity_notify work while migrating IRQs during hot plug

2017-02-27 Thread Thomas Gleixner
On Mon, 27 Feb 2017, Sodagudi Prasad wrote: > So I am thinking that, adding following sched_work() would notify clients. And break the world and some more. > diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c > index 6b66959..5e4766b 100644 > --- a/kernel/irq/manage.c > +++ b/kernel/irq/manag

Re: Schedule affinity_notify work while migrating IRQs during hot plug

2017-02-27 Thread Sodagudi Prasad
On 2017-02-21 12:59, Sodagudi Prasad wrote: Hi Thomas, Currently irq_set_affinity() is called to migrate irqs from migrate_one_irq() during cpu hot plug and clients which are interested to know the irq affinity change not getting notified take_cpu_down () --> __cpu_disable() --> irq_migrate_

Schedule affinity_notify work while migrating IRQs during hot plug

2017-02-21 Thread Sodagudi Prasad
Hi Thomas, Currently irq_set_affinity() is called to migrate irqs from migrate_one_irq() during cpu hot plug and clients which are interested to know the irq affinity change not getting notified take_cpu_down () --> __cpu_disable() --> irq_migrate_all_off_this_cpu(); irq_set_affinity() is ch