RE: [PATCH v1 1/2] usbip: modifications to event handler

2016-03-22 Thread fx IWATA NOBUO
mane...@gmail.com; > shuah...@samsung.com; linux-usb@vger.kernel.org; fx MICHIMURA TADAO > Subject: Re: [PATCH v1 1/2] usbip: modifications to event handler > > On Wed, Mar 23, 2016 at 01:54:21AM +, fx IWATA NOBUO wrote: > > Hello Greg, > > > > > adds errors a

Re: [PATCH v1 1/2] usbip: modifications to event handler

2016-03-22 Thread gre...@linuxfoundation.org
On Wed, Mar 23, 2016 at 01:54:21AM +, fx IWATA NOBUO wrote: > Hello Greg, > > > adds errors and the second fixes it. > > I divided patches as > 1/2 event handler itself > 2/2 programs use event handler. > > So, programs use event handler which will be modified by 2/2 has errors at > 1/2.

RE: [PATCH v1 1/2] usbip: modifications to event handler

2016-03-22 Thread fx IWATA NOBUO
.org; fx MICHIMURA TADAO > Subject: Re: [PATCH v1 1/2] usbip: modifications to event handler > > On Tue, Mar 22, 2016 at 11:45:16PM +, fx IWATA NOBUO wrote: > > Hi, > > > > The errors are cleared by patch 2/2. > > That's not ok, you can not have one patch that

Re: [PATCH v1 1/2] usbip: modifications to event handler

2016-03-22 Thread gre...@linuxfoundation.org
On Tue, Mar 22, 2016 at 11:45:16PM +, fx IWATA NOBUO wrote: > Hi, > > The errors are cleared by patch 2/2. That's not ok, you can not have one patch that adds errors and the second fixes it. Each has to have no problems on their own. > Should I complete in patch 1/2? You need to fix patch

RE: [PATCH v1 1/2] usbip: modifications to event handler

2016-03-22 Thread fx IWATA NOBUO
.@samsung.com; gre...@linuxfoundation.org; > linux-usb@vger.kernel.org; fx IWATA NOBUO; fx MICHIMURA TADAO > Subject: Re: [PATCH v1 1/2] usbip: modifications to event handler > > Hi Nobuo, > > [auto build test ERROR on v4.5-rc7] > [also build test ERROR on next-20160321] [if y

Re: [PATCH v1 1/2] usbip: modifications to event handler

2016-03-22 Thread kbuild test robot
Hi Nobuo, [auto build test ERROR on v4.5-rc7] [also build test ERROR on next-20160321] [if your patch is applied to the wrong git tree, please drop us a note to help improving the system] url:

[PATCH v1 1/2] usbip: modifications to event handler

2016-03-22 Thread Nobuo Iwata
Modification to usbip_event.c. BEFORE) kernel threads are created in usbip_start_eh(). AFTER) one workqueue is created in new usbip_init_eh(). Event handler which was main loop of kernel thread is modified to workqueue handler. Events themselves are stored in struct usbip_device - same as