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
et...@vger.kernel.org; linux- >> ker...@vger.kernel.org; Richard Weinberger >> Subject: [PATCH] hyperv: Add netpoll support >> >> In order to have at least a netconsole to debug kernel issues on >> Windows Azure this patch implements netpoll support. >> Sending packets is easy,

Re: [PATCH] hyperv: Add netpoll support

2014-07-08 Thread Richard Weinberger
et...@vger.kernel.org; linux- >> ker...@vger.kernel.org; Richard Weinberger >> Subject: [PATCH] hyperv: Add netpoll support >> >> In order to have at least a netconsole to debug kernel issues on Windows >> Azure this patch implements netpoll support. >> Sending packets is easy,

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

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 netpoll support

2014-07-08 Thread Richard Weinberger
In order to have at least a netconsole to debug kernel issues on Windows Azure this patch implements netpoll support. Sending packets is easy, netvsc_start_xmit() does already everything needed. To receive we need to trigger the channel callback which is usally called via tasklet_schedule(). Signe