* [PATCH 5.10-rt] locking/rtmutex: switch to EXPORT_SYMBOL() for ww_mutex_lock{,_interruptible}()
@ 2022-08-03 6:24 Yajun Deng
2022-08-18 16:32 ` Sebastian Andrzej Siewior
0 siblings, 1 reply; 3+ messages in thread
From: Yajun Deng @ 2022-08-03 6:24 UTC (permalink / raw)
To: peterz, mingo, will, bigeasy; +Cc: linux-kernel, linux-rt-users, Yajun Deng
We can use EXPORT_SYMBOL() instead of EXPORT_SYMBOL_GPL() in
ww_mutex_lock_interruptible() and ww_mutex_lock(). That match
ww_mutex_unlock() well. And also good for 3rd kernel modules.
Signed-off-by: Yajun Deng <yajun.deng@linux.dev>
---
kernel/locking/rtmutex.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/locking/rtmutex.c b/kernel/locking/rtmutex.c
index 31b374d36d04..b110fc57f733 100644
--- a/kernel/locking/rtmutex.c
+++ b/kernel/locking/rtmutex.c
@@ -2513,7 +2513,7 @@ ww_mutex_lock_interruptible(struct ww_mutex *lock, struct ww_acquire_ctx *ctx)
return ret;
}
-EXPORT_SYMBOL_GPL(ww_mutex_lock_interruptible);
+EXPORT_SYMBOL(ww_mutex_lock_interruptible);
int __sched
ww_mutex_lock(struct ww_mutex *lock, struct ww_acquire_ctx *ctx)
@@ -2533,7 +2533,7 @@ ww_mutex_lock(struct ww_mutex *lock, struct ww_acquire_ctx *ctx)
return ret;
}
-EXPORT_SYMBOL_GPL(ww_mutex_lock);
+EXPORT_SYMBOL(ww_mutex_lock);
void __sched ww_mutex_unlock(struct ww_mutex *lock)
{
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 5.10-rt] locking/rtmutex: switch to EXPORT_SYMBOL() for ww_mutex_lock{,_interruptible}()
2022-08-03 6:24 [PATCH 5.10-rt] locking/rtmutex: switch to EXPORT_SYMBOL() for ww_mutex_lock{,_interruptible}() Yajun Deng
@ 2022-08-18 16:32 ` Sebastian Andrzej Siewior
2022-08-18 18:59 ` Luis Claudio R. Goncalves
0 siblings, 1 reply; 3+ messages in thread
From: Sebastian Andrzej Siewior @ 2022-08-18 16:32 UTC (permalink / raw)
To: Yajun Deng, Luis Claudio R. Goncalves
Cc: peterz, mingo, will, linux-kernel, linux-rt-users
On 2022-08-03 14:24:30 [+0800], Yajun Deng wrote:
> We can use EXPORT_SYMBOL() instead of EXPORT_SYMBOL_GPL() in
> ww_mutex_lock_interruptible() and ww_mutex_lock(). That match
> ww_mutex_unlock() well. And also good for 3rd kernel modules.
Not that I am fan of this but it will sync the symbols with what landed
upstream as of commit
f8635d509d807 ("locking/ww_mutex: Implement rtmutex based ww_mutex API functions")
therefore it would be good to have.
Luis?
> Signed-off-by: Yajun Deng <yajun.deng@linux.dev>
Sebastian
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 5.10-rt] locking/rtmutex: switch to EXPORT_SYMBOL() for ww_mutex_lock{,_interruptible}()
2022-08-18 16:32 ` Sebastian Andrzej Siewior
@ 2022-08-18 18:59 ` Luis Claudio R. Goncalves
0 siblings, 0 replies; 3+ messages in thread
From: Luis Claudio R. Goncalves @ 2022-08-18 18:59 UTC (permalink / raw)
To: Sebastian Andrzej Siewior
Cc: Yajun Deng, peterz, mingo, will, linux-kernel, linux-rt-users
On Thu, Aug 18, 2022 at 06:32:37PM +0200, Sebastian Andrzej Siewior wrote:
> On 2022-08-03 14:24:30 [+0800], Yajun Deng wrote:
> > We can use EXPORT_SYMBOL() instead of EXPORT_SYMBOL_GPL() in
> > ww_mutex_lock_interruptible() and ww_mutex_lock(). That match
> > ww_mutex_unlock() well. And also good for 3rd kernel modules.
>
> Not that I am fan of this but it will sync the symbols with what landed
> upstream as of commit
> f8635d509d807 ("locking/ww_mutex: Implement rtmutex based ww_mutex API functions")
>
> therefore it would be good to have.
> Luis?
Looks good to me. I can add that to my next update.
Luis
> > Signed-off-by: Yajun Deng <yajun.deng@linux.dev>
>
> Sebastian
>
---end quoted text---
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-08-18 18:59 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-03 6:24 [PATCH 5.10-rt] locking/rtmutex: switch to EXPORT_SYMBOL() for ww_mutex_lock{,_interruptible}() Yajun Deng
2022-08-18 16:32 ` Sebastian Andrzej Siewior
2022-08-18 18:59 ` Luis Claudio R. Goncalves
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.