Re: [PATCH] smpboot.c: move setup_vector_irq after set_cpu_online

2015-07-02 Thread Xiao, Jin
Hi tglx, On 7/2/2015 8:50 PM, Thomas Gleixner wrote: The whole vector stuff is racy versus cpu hotplug and Jins patch merily addresses a small part of it and by doing that it breaks stuff as well. With that patch we move the vector setup after marking the cpu online, which is wrong because the

Re: [PATCH] smpboot.c: move setup_vector_irq after set_cpu_online

2015-07-02 Thread Thomas Gleixner
On Thu, 2 Jul 2015, Zhang, Yanmin wrote: > On 2015/7/2 15:02, Xiao, Jin wrote: > > Hi Joerg, > > > > On 7/2/2015 2:52 PM, Joerg Roedel wrote: > >> Hi Jin, > >> > >> On Thu, Jul 02, 2015 at 12:24:34PM +0800, xiao jin wrote: > >>> [ 106.107851] BUG: unable to handle kernel NULL pointer dereference a

Re: [PATCH] smpboot.c: move setup_vector_irq after set_cpu_online

2015-07-02 Thread Zhang, Yanmin
On 2015/7/2 15:02, Xiao, Jin wrote: > Hi Joerg, > > On 7/2/2015 2:52 PM, Joerg Roedel wrote: >> Hi Jin, >> >> On Thu, Jul 02, 2015 at 12:24:34PM +0800, xiao jin wrote: >>> [ 106.107851] BUG: unable to handle kernel NULL pointer dereference at >>> 0040 >>> [ 106.116702] IP: >>> [ 106

Re: [PATCH] smpboot.c: move setup_vector_irq after set_cpu_online

2015-07-02 Thread Xiao, Jin
Hi Joerg, On 7/2/2015 2:52 PM, Joerg Roedel wrote: Hi Jin, On Thu, Jul 02, 2015 at 12:24:34PM +0800, xiao jin wrote: [ 106.107851] BUG: unable to handle kernel NULL pointer dereference at 0040 [ 106.116702] IP: [ 106.118490] [] check_irq_vectors_for_cpu_disable+0x76/0x180 Th

Re: [PATCH] smpboot.c: move setup_vector_irq after set_cpu_online

2015-07-01 Thread Joerg Roedel
Hi Jin, On Thu, Jul 02, 2015 at 12:24:34PM +0800, xiao jin wrote: > [ 106.107851] BUG: unable to handle kernel NULL pointer dereference at > 0040 > [ 106.116702] IP: > [ 106.118490] [] > check_irq_vectors_for_cpu_disable+0x76/0x180 This might be the same issue I fixed with: