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

Haiyang Zhang [off-list ref] writes:

quoted

-----Original Message-----
From: Richard Weinberger [mailto:richard.weinberger@gmail.com]
Sent: Tuesday, October 27, 2015 6:36 PM
To: David Miller <davem@davemloft.net>
Cc: Haiyang Zhang <haiyangz@microsoft.com>; olaf@aepfle.de; Greg Kroah-
Hartman [off-list ref]; netdev@vger.kernel.org; jasowang@redhat.com;
driverdev-devel@linuxdriverproject.org; LKML <linux-
kernel@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 [off-list ref]
wrote:

quoted

From: Haiyang Zhang <haiyangz@microsoft.com>
Date: Mon, 23 Jun 2014 16:09:59 +0000

quoted

So, what's the equivalent or similar command to "network restart" on

SLES12? Could

quoted

quoted

you update the command line for the usermodehelper when porting this

patch to SLES

quoted

quoted

12?

No, you are not going to keep the usermodehelper invocation in your

driver

quoted

please remove it.  It is absolutely inappropriate, and I strictly do

not want

quoted

to keep it in there because other people will copy it and then we'll

have a

quoted

real mess on our hands.

Sorry for digging up this old thread.
While talking with some guys about usermodehelper abuses I came across
this gem.
Mainline still contains that "/etc/init.d/network restart" code.
Haiyang, care to cleanup?

Hi Richard and others,

Thanks for the reminder. I will clean up the usermode helper.

Do you have suggestions of trigger DHCP refresh from kernel mode? Any 
sample code in the existing kernel code?

I think it's wrong to call dhcp refresh from kernel. What happens when
we reconnect normal hardware adapter to another network? Link goes down
and then up and userspace is supposed to react accordingly. I think we
should emulate something similar for RNDIS_STATUS_NETWORK_CHANGE.

-- 
  Vitaly