From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johannes Berg Subject: Re: [PATCH v2 0/2] wireless: Allow wiphy/hwsim management from user namespaces Date: Thu, 12 May 2016 11:27:01 +0200 Message-ID: <1463045221.13313.16.camel@sipsolutions.net> References: <1462811639-29906-1-git-send-email-martin@strongswan.org> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Cc: linux-wireless@vger.kernel.org, netdev@vger.kernel.org To: Martin Willi Return-path: Received: from s3.sipsolutions.net ([5.9.151.49]:40013 "EHLO sipsolutions.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751497AbcELJ1G (ORCPT ); Thu, 12 May 2016 05:27:06 -0400 In-Reply-To: <1462811639-29906-1-git-send-email-martin@strongswan.org> Sender: netdev-owner@vger.kernel.org List-ID: Hi Martin, Thanks for the update. I've applied your patch now, but I think we need to do some work around the wmediumd abstraction. Right now, you allowed=C2=A0HWSIM_CMD_REGISTER only from init_net, whic= h is mostly fine. However, it means that a wmediumd running in init_net will see frames from the radios that are generated by other namespace, but will have no way to access anything else about them, neither through hwsim netlink nor through nl80211 or sysfs. So I think you should change the "wmediumd_portid" variable to live in=C2=A0struct hwsim_net and then allow HWSIM_CMD_REGISTER to be done i= nside the net namespace. With that, and making the other few wmediumd related operations namespace aware, we would be able to run wmediumd in any net namespace and have it manage all the radios created in that namespace. johannes