On Tue, Aug 18, 2015 at 10:18:51AM -0700, Scott Feldman wrote: > On Tue, Aug 18, 2015 at 7:01 AM, Andrew Lunn <and...@lunn.ch> wrote: > > On Mon, Aug 17, 2015 at 03:36:17PM -0700, sfel...@gmail.com wrote: > >> From: Scott Feldman <sfel...@gmail.com> > >> > >> > tree /sys/kernel/debug/rocker > >> /sys/kernel/debug/rocker > >> ????????? 5254001235010000 > >> ????????? fdb_tbl > >> ????????? internal_vlan_tbl > >> ????????? neigh_tbl > >> ????????? of_dpa_flow_tbl > >> ????????? of_dpa_group_tbl > >> > >> 1 directory, 5 files > >> > >> > cat /sys/kernel/debug/rocker/5254001235010000/internal_vlan_tbl > >> ifindex 5 ref_count 1 vlan 3843 > >> ifindex 7 ref_count 2 vlan 3840 > >> ifindex 4 ref_count 1 vlan 3842 > >> > >> > cat /sys/kernel/debug/rocker/5254001235010000/fdb_tbl > >> learned 1 pport 1 addr 00:02:00:00:02:00 vlan 3840 > >> learned 1 pport 2 addr 00:02:00:00:03:00 vlan 3840 > >> > >> > cat /sys/kernel/debug/rocker/5254001235010000/neigh_tbl > >> 11.0.0.9 dev sw1p2 ref_count 3 index 1 dst 00:02:00:00:01:00 ttl_check 1 > >> 11.0.0.1 dev sw1p1 ref_count 3 index 0 dst 00:02:00:00:00:00 ttl_check 1 > > > > Hi Scott > > > > David is not so keen no debugfs stuff. He already NACKed adding more > > than what is currently in DSA: > > > > https://lkml.org/lkml/2015/7/11/8 > > That patch added writable debugfs files, which I can see might be used > as a back-door to program hardware. That does seem bad.
I fully agreed with respect to write. But if you read the whole message, David is also not happy with read only. I think before you spend too much more time on this, you need some indication from David if he is going to merge it or not. Andrew -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html