Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

haiya...@microsoft.com>; o...@aepfle.de; Greg Kroah- >> Hartman <g...@kroah.com>; net...@vger.kernel.org; jasow...@redhat.com; >> driverdev-devel@linuxdriverproject.org; LKML > ker...@vger.kernel.org> >> Subject: Re: [PATCH net-next] hyperv: Add handler for >&

RE: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

mloft.net>; o...@aepfle.de; jasow...@redhat.com; driverdev- > de...@linuxdriverproject.org; LKML <linux-ker...@vger.kernel.org>; > net...@vger.kernel.org > Subject: Re: [PATCH net-next] hyperv: Add handler for > RNDIS_STATUS_NETWORK_CHANGE event > > Haiyang Zhang <haiya...@micr

RE: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

Hartman <g...@kroah.com>; net...@vger.kernel.org; jasow...@redhat.com; > driverdev-devel@linuxdriverproject.org; LKML ker...@vger.kernel.org> > Subject: Re: [PATCH net-next] hyperv: Add handler for > RNDIS_STATUS_NETWORK_CHANGE event > > On Mon, Jun 23, 2014 at 10:10 PM, David Miller

RE: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

Hartman <g...@kroah.com>; net...@vger.kernel.org; > jasow...@redhat.com; driverdev-devel@linuxdriverproject.org; LKML > <linux-ker...@vger.kernel.org> > Subject: Re: [PATCH net-next] hyperv: Add handler for > RNDIS_STATUS_NETWORK_CHANGE event > > On Mon, Jun 23, 2014 at 10:10 PM, D

Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

On Mon, Jun 23, 2014 at 10:10 PM, David Miller wrote: > From: Haiyang Zhang > Date: Mon, 23 Jun 2014 16:09:59 + > >> So, what's the equivalent or similar command to "network restart" on SLES12? >> Could >> you update the command line for the

Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

On Mon, Jun 23, Haiyang Zhang wrote: Yes, there is a link down/up event from the host, we currently call netif_carrier_off() / netif_carrier_on() with these events. Will hotplug scripts be triggered by netif_carrier_off/on? Where are the scripts located at (SLES)? In

RE: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event On Mon, Jun 23, Haiyang Zhang wrote: Yes, there is a link down/up event from the host, we currently call netif_carrier_off() / netif_carrier_on() with these events. Will hotplug scripts be triggered

RE: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event On Mon, Jun 23, Haiyang Zhang wrote: I think its reasonable to expect guest config changes on this new kind of host. Would a link-down/link-up event work? I'm sure it will, there is enough code floating around

Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

On Mon, Jun 23, Haiyang Zhang wrote: I have tried some possibilities, like IF_OPER_DORMANT then IF_OPER_UP with netdev_state_change() etc. but not able to trigger DHCP review. I will look at this further... Is there a link down/up event anyway? If the interface is configured to on hotplug

Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

...@linuxdriverproject.org; linux-ker...@vger.kernel.org; da...@davemloft.net Subject: Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event On Mon, Jun 23, Haiyang Zhang wrote: I think its reasonable to expect guest config changes on this new kind of host. Would a link-down

RE: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event On Mon, Jun 23, Haiyang Zhang wrote: I have tried some possibilities, like IF_OPER_DORMANT then IF_OPER_UP with netdev_state_change() etc. but not able to trigger DHCP review. I will look at this further

RE: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

...@vger.kernel.org; jasow...@redhat.com; driverdev- de...@linuxdriverproject.org; linux-ker...@vger.kernel.org; da...@davemloft.net Subject: Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event On Mon, Jun 23, Haiyang Zhang wrote: I think its

Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

From: Olaf Hering o...@aepfle.de Date: Mon, 23 Jun 2014 15:17:23 +0200 On Mon, Jun 23, Haiyang Zhang wrote: I think its reasonable to expect guest config changes on this new kind of host. Would a link-down/link-up event work? I'm sure it will, there is enough code floating around in the

Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

From: Haiyang Zhang haiya...@microsoft.com Date: Mon, 23 Jun 2014 16:09:59 + So, what's the equivalent or similar command to network restart on SLES12? Could you update the command line for the usermodehelper when porting this patch to SLES 12? No, you are not going to keep the

Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

From: Greg KH g...@kroah.com Date: Mon, 23 Jun 2014 12:29:11 -0400 Given that this change will fail on all future distro releases, and almost all of the community distros today, I don't see how this is acceptable at all. Nor would it be any better if you switch to a systemd command line

RE: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

: Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event From: Haiyang Zhang haiya...@microsoft.com Date: Mon, 23 Jun 2014 16:09:59 + So, what's the equivalent or similar command to network restart on SLES12? Could you update the command line

Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

On Thu, Jun 19, 2014 at 06:34:36PM -0700, Haiyang Zhang wrote: @@ -589,7 +590,19 @@ void netvsc_linkstatus_callback(struct hv_device *device_obj, net_device = hv_get_drvdata(device_obj); rdev = net_device-extension; - rdev-link_state = status != 1; + switch

RE: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event On Fri, Jun 20, 2014 at 06:57:04AM +0200, Olaf Hering wrote: On Thu, Jun 19, Haiyang Zhang wrote: The RNDIS_STATUS_NETWORK_CHANGE event is received after the Hyper-V host sleep or hibernation. We refresh network

RE: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

...@vger.kernel.org Subject: Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event On Thu, Jun 19, 2014 at 06:34:36PM -0700, Haiyang Zhang wrote: @@ -589,7 +590,19 @@ void netvsc_linkstatus_callback(struct hv_device *device_obj, net_device = hv_get_drvdata(device_obj

Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

From: Haiyang Zhang haiya...@microsoft.com Date: Thu, 19 Jun 2014 18:34:36 -0700 The RNDIS_STATUS_NETWORK_CHANGE event is received after the Hyper-V host sleep or hibernation. We refresh network at this time. MS-TFS: 135162 Signed-off-by: Haiyang Zhang haiya...@microsoft.com Reviewed-by:

Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

On Thu, Jun 19, Haiyang Zhang wrote: The RNDIS_STATUS_NETWORK_CHANGE event is received after the Hyper-V host sleep or hibernation. We refresh network at this time. + char *argv[] = { /etc/init.d/network, restart, NULL }; What happens if that file does not exist? Dead network in the

Re: [PATCH net-next] hyperv: Add handler for RNDIS_STATUS_NETWORK_CHANGE event

On Fri, Jun 20, 2014 at 06:57:04AM +0200, Olaf Hering wrote: On Thu, Jun 19, Haiyang Zhang wrote: The RNDIS_STATUS_NETWORK_CHANGE event is received after the Hyper-V host sleep or hibernation. We refresh network at this time. + char *argv[] = { /etc/init.d/network, restart, NULL };