Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-05 Thread Baolin Wang
Hi John, On 6 January 2017 at 03:08, John Youn wrote: > On 12/28/2016 5:29 PM, John Youn wrote: >> >> >>> Janusz Dziedzic writes: >>> On some platfroms(like x86 platform), when one core is running the >>> USB gadget >>> irq thread

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-05 Thread John Youn
On 12/28/2016 5:29 PM, John Youn wrote: > > >> Janusz Dziedzic writes: >> On some platfroms(like x86 platform), when one core is running the >> USB gadget >> irq thread handler by dwc3_thread_interrupt(), meanwhile another >> core also can >> respond other

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-05 Thread Baolin Wang
Hi, On 5 January 2017 at 19:19, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: > > [...] > > and you have triggered this with mailine? How? We don't write to GEVNT* > registers from PM code and we only allow runtime_suspend with cable >

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-05 Thread Felipe Balbi
Hi, Baolin Wang writes: [...] and you have triggered this with mailine? How? We don't write to GEVNT* registers from PM code and we only allow runtime_suspend with cable dettached. >>> >>> Sorry for late reply since I am busy on other things. I just

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-05 Thread Baolin Wang
Hi, On 5 January 2017 at 17:26, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: On 27 December 2016 at 18:52, Janusz Dziedzic wrote: > 2016-12-26 9:01 GMT+01:00 Baolin Wang

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-05 Thread Felipe Balbi
Hi, Baolin Wang writes: >>> On 27 December 2016 at 18:52, Janusz Dziedzic >>> wrote: 2016-12-26 9:01 GMT+01:00 Baolin Wang : > On some platfroms(like x86 platform), when one core is running the

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-04 Thread Baolin Wang
On 3 January 2017 at 20:33, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: >> On 28 December 2016 at 20:30, Janusz Dziedzic >> wrote: >>> 2016-12-27 13:16 GMT+01:00 Baolin Wang : Hi,

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-03 Thread Felipe Balbi
Hi, Baolin Wang writes: > On 28 December 2016 at 20:30, Janusz Dziedzic > wrote: >> 2016-12-27 13:16 GMT+01:00 Baolin Wang : >>> Hi, >>> >>> On 27 December 2016 at 19:11, Felipe Balbi wrote:

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2017-01-03 Thread Baolin Wang
On 28 December 2016 at 20:30, Janusz Dziedzic wrote: > 2016-12-27 13:16 GMT+01:00 Baolin Wang : >> Hi, >> >> On 27 December 2016 at 19:11, Felipe Balbi wrote: >>> >>> Hi, >>> >>> Baolin Wang writes:

RE: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-28 Thread John Youn
nux.intel.com>; Baolin Wang > <baolin.w...@linaro.org>; Greg KH <gre...@linuxfoundation.org>; USB > <linux-usb@vger.kernel.org>; LKML <linux-ker...@vger.kernel.org>; Linaro > Kernel Mailman List <linaro-ker...@lists.linaro.org>; Mark Brown > <broo...@ke

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-28 Thread Felipe Balbi
Hi, Janusz Dziedzic writes: On some platfroms(like x86 platform), when one core is running the USB gadget irq thread handler by dwc3_thread_interrupt(), meanwhile another core also can respond other interrupts from dwc3 controller and modify

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-28 Thread Janusz Dziedzic
2016-12-27 12:05 GMT+01:00 Felipe Balbi : > Hi, > > Lu Baolu writes: >> On 12/26/2016 04:01 PM, Baolin Wang wrote: >>> On some platfroms(like x86 platform), when one core is running the USB >>> gadget >>> irq thread handler by dwc3_thread_interrupt(),

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-28 Thread Janusz Dziedzic
2016-12-27 13:16 GMT+01:00 Baolin Wang : > Hi, > > On 27 December 2016 at 19:11, Felipe Balbi wrote: >> >> Hi, >> >> Baolin Wang writes: >>> Hi, >>> >>> On 27 December 2016 at 18:52, Janusz Dziedzic

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-27 Thread Baolin Wang
Hi, On 27 December 2016 at 19:11, Felipe Balbi wrote: > > Hi, > > Baolin Wang writes: >> Hi, >> >> On 27 December 2016 at 18:52, Janusz Dziedzic >> wrote: >>> 2016-12-26 9:01 GMT+01:00 Baolin Wang :

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-27 Thread Felipe Balbi
Hi, Baolin Wang writes: > Hi, > > On 27 December 2016 at 18:52, Janusz Dziedzic > wrote: >> 2016-12-26 9:01 GMT+01:00 Baolin Wang : >>> On some platfroms(like x86 platform), when one core is running the USB >>> gadget

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-27 Thread Felipe Balbi
Hi, Janusz Dziedzic writes: > 2016-12-26 9:01 GMT+01:00 Baolin Wang : >> On some platfroms(like x86 platform), when one core is running the USB gadget >> irq thread handler by dwc3_thread_interrupt(), meanwhile another core also >> can >>

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-27 Thread Felipe Balbi
Hi, Lu Baolu writes: > On 12/26/2016 04:01 PM, Baolin Wang wrote: >> On some platfroms(like x86 platform), when one core is running the USB gadget >> irq thread handler by dwc3_thread_interrupt(), meanwhile another core also >> can >> respond other interrupts from dwc3

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-27 Thread Baolin Wang
Hi, On 27 December 2016 at 18:52, Janusz Dziedzic wrote: > 2016-12-26 9:01 GMT+01:00 Baolin Wang : >> On some platfroms(like x86 platform), when one core is running the USB gadget >> irq thread handler by dwc3_thread_interrupt(), meanwhile

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-27 Thread Janusz Dziedzic
2016-12-26 9:01 GMT+01:00 Baolin Wang : > On some platfroms(like x86 platform), when one core is running the USB gadget > irq thread handler by dwc3_thread_interrupt(), meanwhile another core also can > respond other interrupts from dwc3 controller and modify the event

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-26 Thread Lu Baolu
Hi, On 12/27/2016 10:58 AM, Baolin Wang wrote: > Hi, > > On 27 December 2016 at 10:39, Lu Baolu wrote: >> Hi, >> >> On 12/26/2016 04:01 PM, Baolin Wang wrote: >>> On some platfroms(like x86 platform), when one core is running the USB >>> gadget >>> irq thread handler

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-26 Thread Baolin Wang
Hi, On 27 December 2016 at 10:39, Lu Baolu wrote: > Hi, > > On 12/26/2016 04:01 PM, Baolin Wang wrote: >> On some platfroms(like x86 platform), when one core is running the USB gadget >> irq thread handler by dwc3_thread_interrupt(), meanwhile another core also >> can

Re: [PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-26 Thread Lu Baolu
Hi, On 12/26/2016 04:01 PM, Baolin Wang wrote: > On some platfroms(like x86 platform), when one core is running the USB gadget > irq thread handler by dwc3_thread_interrupt(), meanwhile another core also can > respond other interrupts from dwc3 controller and modify the event buffer by >

[PATCH] usb: dwc3: gadget: Avoid race between dwc3 interrupt handler and irq thread handler

2016-12-26 Thread Baolin Wang
On some platfroms(like x86 platform), when one core is running the USB gadget irq thread handler by dwc3_thread_interrupt(), meanwhile another core also can respond other interrupts from dwc3 controller and modify the event buffer by dwc3_interrupt() function, that will cause getting the wrong