Thread (4 messages) flat view 4 messages, 2 authors, 2012-01-26

Re: [patch v1, kernel 3.2.1] RTNETLINK adjusting values of min_ifinfo_dump_size

From: David Miller <davem@davemloft.net>
Date: 2012-01-26 02:36:14
Also in: lkml

From: Stefan Gula <redacted>
Date: Wed, 25 Jan 2012 09:43:15 +0100 (CET)
From: Stefan Gula <redacted>

This patch extends the rtnetlink.c. The problem is that calcit, which
returns the value min_ifinfo_dump_size, is called without ability to
get device info as well (struct net_device *dev is not accessible from
it). This create a problem if required size of struct sk_buff *skb is
not enough big to comply with all data returned by fill_info function
for the interface that required size can vary over time, e.g. some
dynamic structures will be putted to skb like hlists or lists. To do
this patch extends the do_setlink function with recalculating of
min_ifinfo_dump_size accordingly to device type and get_size
functions.

Signed-off-by: Stefan Gula <redacted>
This commit message is terrible, and the reason I know it's terrible
is because even after reading it I still had to go read the code
in order to understand what your change even does or why it's even
necessary.

How about this:

	Setting link parameters on a netdevice changes the value
	of if_nlmsg_size(), therefore it is necessary to recalculate
	min_ifinfo_dump_size.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help