lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <56146A74.8050807@gmail.com> Date: Tue, 06 Oct 2015 17:42:28 -0700 From: Florian Fainelli <f.fainelli@...il.com> To: Vivien Didelot <vivien.didelot@...oirfairelinux.com>, netdev@...r.kernel.org CC: linux-kernel@...r.kernel.org, kernel@...oirfairelinux.com, "David S. Miller" <davem@...emloft.net>, Scott Feldman <sfeldma@...il.com>, Jiri Pirko <jiri@...nulli.us>, Andrew Lunn <andrew@...n.ch>, Neil Armstrong <narmstrong@...libre.com>, Sergei Shtylyov <sergei.shtylyov@...entembedded.com> Subject: Re: [PATCH net-next 0/3] net: remove dsa.h from netdevice.h On 06/10/15 14:54, Vivien Didelot wrote: > In order to push switchdev objects down to DSA drivers, I need to include > switchdev.h in dsa.h. But compilation fails because of a circular dependency > issue, since dsa.h is also included in linux/netdevice.h. Just for the record, what does this circular dependency looks like? Last I tried something in that front, I ended-up forward declaring the 'switchdev_obj' structure, but that was weeks ago before the whole restructuring, so could be pointless now. > > dsa.h is included for two purposes: to have the definition of struct > dsa_switch_tree; and to have access to dsa_uses_tagged_protocol(), used by > netdev_uses_dsa(). > > This patchset forward declares struct dsa_switch_tree; adds a new uses_hw_tag > boolean to struct net_device, which is more explicit than the DSA helper, and > finally get rid of the dsa.h include. > > With this patchset, switchdev.h can safely be included in dsa.h. > > Vivien Didelot (3): > net: dsa: add uses_hw_tag > net: dsa: include dsa.h in dsa_priv.h > net: remove dsa.h include from linux/netdevice.h > > include/linux/netdevice.h | 9 ++++++--- > include/net/dsa.h | 5 ----- > net/dsa/dsa.c | 1 + > net/dsa/dsa_priv.h | 1 + > 4 files changed, 8 insertions(+), 8 deletions(-) > -- Florian -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists