Re: [PATCH] hyperv: Add netpoll support

2014-07-09 Thread Richard Weinberger
Am 09.07.2014 01:43, schrieb Francois Romieu: > Richard Weinberger : >> Am 09.07.2014 00:47, schrieb Francois Romieu: > [...] >>> What are you taking about ? netconsole does not need to receive. >> >> Isn't netconsole is only one user of netpoll ? > > Out of tree users are irrelevant. See netpoll

Re: [PATCH] hyperv: Add netpoll support

2014-07-08 Thread Francois Romieu
Richard Weinberger : > Am 09.07.2014 00:47, schrieb Francois Romieu: [...] > > What are you taking about ? netconsole does not need to receive. > > Isn't netconsole is only one user of netpoll ? Out of tree users are irrelevant. See netpoll related comments in cd6362befe4cc7bf589a5236d2a780af2d4

Re: [PATCH] hyperv: Add netpoll support

2014-07-08 Thread Richard Weinberger
Am 09.07.2014 00:47, schrieb Francois Romieu: > Richard Weinberger : > [...] >> This won't work as netpoll runs with IRQs disabled. >> ->ndo_poll_controller() has to make sure that SKBs can be received and >> transmitted >> while IRQs are off. I thought calling the channel callback by hand would

Re: [PATCH] hyperv: Add netpoll support

2014-07-08 Thread Francois Romieu
Richard Weinberger : [...] > This won't work as netpoll runs with IRQs disabled. > ->ndo_poll_controller() has to make sure that SKBs can be received and > transmitted > while IRQs are off. I thought calling the channel callback by hand would be > enough to receive SKBs. What are you taking abou

Re: [PATCH] hyperv: Add netpoll support

2014-07-08 Thread Richard Weinberger
Am 09.07.2014 00:47, schrieb Francois Romieu: > Richard Weinberger : > [...] >> This won't work as netpoll runs with IRQs disabled. >> ->ndo_poll_controller() has to make sure that SKBs can be received and >> transmitted >> while IRQs are off. I thought calling the channel callback by hand would

Re: [PATCH] hyperv: Add netpoll support

2014-07-08 Thread Richard Weinberger
Am 08.07.2014 22:03, schrieb KY Srinivasan: > The VCPU the channel is bound to is available in the channel state. You could > use the following code > Fragment to ensure that the call is made on the "right" cpu: > > smp_call_function_single(dev->channel->target_cpu, >

RE: [PATCH] hyperv: Add netpoll support

2014-07-08 Thread KY Srinivasan
> -Original Message- > From: Richard Weinberger [mailto:rich...@nod.at] > Sent: Tuesday, July 8, 2014 11:39 AM > To: KY Srinivasan; Haiyang Zhang > Cc: de...@linuxdriverproject.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: Re: [PATCH] hyperv

RE: [PATCH] hyperv: Add netpoll support

2014-07-08 Thread Haiyang Zhang
> -Original Message- > From: Richard Weinberger [mailto:rich...@nod.at] > Sent: Tuesday, July 8, 2014 2:40 PM > To: Haiyang Zhang; KY Srinivasan > Cc: de...@linuxdriverproject.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject: Re: [PATCH] hyperv

Re: [PATCH] hyperv: Add netpoll support

2014-07-08 Thread Richard Weinberger
Am 08.07.2014 19:55, schrieb Haiyang Zhang: > > >> -Original Message- >> From: Richard Weinberger [mailto:rich...@nod.at] >> Sent: Tuesday, July 8, 2014 5:32 AM >> To: KY Srinivasan; Haiyang Zhang >> Cc: de...@linuxdriverproject.org; net...@vger.kernel.org; linux- >> ker...@vger.kernel.or

Re: [PATCH] hyperv: Add netpoll support

2014-07-08 Thread Richard Weinberger
Am 08.07.2014 20:01, schrieb KY Srinivasan: > > >> -Original Message- >> From: Richard Weinberger [mailto:rich...@nod.at] >> Sent: Tuesday, July 8, 2014 2:32 AM >> To: KY Srinivasan; Haiyang Zhang >> Cc: de...@linuxdriverproject.org; net...@vger.kernel.org; linux- >> ker...@vger.kernel.or

RE: [PATCH] hyperv: Add netpoll support

2014-07-08 Thread KY Srinivasan
> -Original Message- > From: Richard Weinberger [mailto:rich...@nod.at] > Sent: Tuesday, July 8, 2014 2:32 AM > To: KY Srinivasan; Haiyang Zhang > Cc: de...@linuxdriverproject.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; Richard Weinberger > Subject: [PATCH] hyperv: Add n

RE: [PATCH] hyperv: Add netpoll support

2014-07-08 Thread Haiyang Zhang
> -Original Message- > From: Richard Weinberger [mailto:rich...@nod.at] > Sent: Tuesday, July 8, 2014 5:32 AM > To: KY Srinivasan; Haiyang Zhang > Cc: de...@linuxdriverproject.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; Richard Weinberger > Subject: [PATCH] hyperv: Add n