b.a.t.m.a.n.lists.open-mesh.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/1] pull request for net: batman-adv 2020-11-24
@ 2020-11-24 13:44 Simon Wunderlich
  2020-11-24 13:44 ` [PATCH 1/1] batman-adv: set .owner to THIS_MODULE Simon Wunderlich
  2020-11-25  0:50 ` [PATCH 0/1] pull request for net: batman-adv 2020-11-24 Jakub Kicinski
  0 siblings, 2 replies; 3+ messages in thread
From: Simon Wunderlich @ 2020-11-24 13:44 UTC (permalink / raw)
  To: davem, kuba; +Cc: netdev, b.a.t.m.a.n

Hi David, hi Jakub,

here is a bugfix for batman-adv which we would like to have integrated into net.

Please pull or let me know of any problem!

Thank you,
      Simon

The following changes since commit f8394f232b1eab649ce2df5c5f15b0e528c92091:

  Linux 5.10-rc3 (2020-11-08 16:10:16 -0800)

are available in the Git repository at:

  git://git.open-mesh.org/linux-merge.git tags/batadv-net-pullrequest-20201124

for you to fetch changes up to 14a2e551faea53d45bc11629a9dac88f88950ca7:

  batman-adv: set .owner to THIS_MODULE (2020-11-15 11:43:56 +0100)

----------------------------------------------------------------
Here is a batman-adv bugfix:

 - set module owner to THIS_MODULE, by Taehee Yoo

----------------------------------------------------------------
Taehee Yoo (1):
      batman-adv: set .owner to THIS_MODULE

 net/batman-adv/log.c | 1 +
 1 file changed, 1 insertion(+)

^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH 1/1] batman-adv: set .owner to THIS_MODULE
  2020-11-24 13:44 [PATCH 0/1] pull request for net: batman-adv 2020-11-24 Simon Wunderlich
@ 2020-11-24 13:44 ` Simon Wunderlich
  2020-11-25  0:50 ` [PATCH 0/1] pull request for net: batman-adv 2020-11-24 Jakub Kicinski
  1 sibling, 0 replies; 3+ messages in thread
From: Simon Wunderlich @ 2020-11-24 13:44 UTC (permalink / raw)
  To: davem, kuba
  Cc: netdev, b.a.t.m.a.n, Taehee Yoo, Sven Eckelmann, Simon Wunderlich

From: Taehee Yoo <ap420073@gmail.com>

If THIS_MODULE is not set, the module would be removed while debugfs is
being used.
It eventually makes kernel panic.

Fixes: c6c8fea29769 ("net: Add batman-adv meshing protocol")
Signed-off-by: Taehee Yoo <ap420073@gmail.com>
Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Simon Wunderlich <sw@simonwunderlich.de>
---
 net/batman-adv/log.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/net/batman-adv/log.c b/net/batman-adv/log.c
index a67b2b091447..c0ca5fbe5b08 100644
--- a/net/batman-adv/log.c
+++ b/net/batman-adv/log.c
@@ -180,6 +180,7 @@ static const struct file_operations batadv_log_fops = {
 	.read           = batadv_log_read,
 	.poll           = batadv_log_poll,
 	.llseek         = no_llseek,
+	.owner          = THIS_MODULE,
 };
 
 /**
-- 
2.20.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH 0/1] pull request for net: batman-adv 2020-11-24
  2020-11-24 13:44 [PATCH 0/1] pull request for net: batman-adv 2020-11-24 Simon Wunderlich
  2020-11-24 13:44 ` [PATCH 1/1] batman-adv: set .owner to THIS_MODULE Simon Wunderlich
@ 2020-11-25  0:50 ` Jakub Kicinski
  1 sibling, 0 replies; 3+ messages in thread
From: Jakub Kicinski @ 2020-11-25  0:50 UTC (permalink / raw)
  To: Simon Wunderlich; +Cc: davem, netdev, b.a.t.m.a.n

On Tue, 24 Nov 2020 14:44:16 +0100 Simon Wunderlich wrote:
> Hi David, hi Jakub,
> 
> here is a bugfix for batman-adv which we would like to have integrated into net.

Pulled, thanks!

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-11-25  0:50 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-24 13:44 [PATCH 0/1] pull request for net: batman-adv 2020-11-24 Simon Wunderlich
2020-11-24 13:44 ` [PATCH 1/1] batman-adv: set .owner to THIS_MODULE Simon Wunderlich
2020-11-25  0:50 ` [PATCH 0/1] pull request for net: batman-adv 2020-11-24 Jakub Kicinski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).