Thread (5 messages) flat view 5 messages, 2 authors, 2017-04-10

Re: [PATCH net-next v2] can: initial support for network namespaces

From: Mario Kicherer <hidden>
Date: 2017-04-10 07:51:09
Also in: linux-can

Hello Oliver,
You didn't include the statistics here:

+       struct s_stats *can_stats;      /* packet statistics */
+       struct s_pstats *can_pstats;    /* receive list statistics */

which need to be per-net too, right?
I was not sure how this information is used - e.g., maybe just as debug 
information that the system is working. Hence, I just left them as they
were.
You do a kzalloc(sizeof(struct dev_rcv_lists), GFP_KERNEL) in
can_pernet_init().

Doesn't it need a kfree(net->can.can_rx_alldev_list) then??
Yes, I missed this. Thanks!

Unfortunately, I don't have access to my CAN hardware right now, hence
I cannot test a corresponding patch at the moment. :/

Best regards,
Mario
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help