[PATCH net-next 0/3] bareudp: several code cleanup for bareudp module

STALE2350d

2 messages, 2 authors, 2020-03-09 · open the first message on its own page

[PATCH net-next 0/3] bareudp: several code cleanup for bareudp module

From: Taehee Yoo <ap420073@gmail.com>
Date: 2020-03-08 01:19:00

This patchset is to cleanup bareudp module code.

1. The first patch is to add module alias
In the current bareudp code, there is no module alias.
So, RTNL couldn't load bareudp module automatically.

2. The second patch is to add extack message.
The extack error message is useful for noticing specific errors
when command is failed.

3. The third patch is to remove unnecessary udp_encap_enable().
In the bareudp_socket_create(), udp_encap_enable() is called.
But, the it's already called in the setup_udp_tunnel_sock().
So, it could be removed.

Taehee Yoo (3):
  bareudp: add module alias
  bareudp: print error message when command fails
  bareudp: remove unnecessary udp_encap_enable() in
    bareudp_socket_create()

 drivers/net/bareudp.c | 17 +++++++++++------
 1 file changed, 11 insertions(+), 6 deletions(-)

-- 
2.17.1

Re: [PATCH net-next 0/3] bareudp: several code cleanup for bareudp module

From: David Miller <davem@davemloft.net>
Date: 2020-03-09 04:22:59

From: Taehee Yoo <ap420073@gmail.com>
Date: Sun,  8 Mar 2020 01:18:49 +0000
This patchset is to cleanup bareudp module code.

1. The first patch is to add module alias
In the current bareudp code, there is no module alias.
So, RTNL couldn't load bareudp module automatically.

2. The second patch is to add extack message.
The extack error message is useful for noticing specific errors
when command is failed.

3. The third patch is to remove unnecessary udp_encap_enable().
In the bareudp_socket_create(), udp_encap_enable() is called.
But, the it's already called in the setup_udp_tunnel_sock().
So, it could be removed.
Series applied, thanks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help