From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Date: Mon, 10 Aug 2009 20:43:32 +0200 References: <1249595428-21594-1-git-send-email-arnd@arndb.de> <4A7FC27F.1090904@trash.net> In-Reply-To: <4A7FC27F.1090904@trash.net> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Message-Id: <200908102043.32937.arnd@arndb.de> Subject: Re: [Bridge] [PATCH] macvlan: add tap device backend List-Id: Linux Ethernet Bridging List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Patrick McHardy Cc: Herbert Xu , "Michael S. Tsirkin" , "Fischer, Anna" , netdev@vger.kernel.org, bridge@lists.linux-foundation.org, linux-kernel@vger.kernel.org, Or Gerlitz , Edge Virtual Bridging , davem@davemloft.net On Monday 10 August 2009, Patrick McHardy wrote: > macvlan will remove all macvlan/vtap devices when the underlying > device in unregistered, at which time you need to release the > device references you're holding. I'd suggest to change the > macvlan_device_event() handler to use > > vlan->dev->rtnl_link_ops->dellink(vlan->dev) > > instead of macvlan_dellink() so the macvtap_dellink callback > is invoked. Ok, will do that. Thanks, Arnd <><