* [PATCH] net: ethernet: micrel: ksz884x: Replace semaphore proc_sem with mutex
@ 2017-06-08 10:03 Binoy Jayan
2017-06-08 14:59 ` Arnd Bergmann
0 siblings, 1 reply; 2+ messages in thread
From: Binoy Jayan @ 2017-06-08 10:03 UTC (permalink / raw)
To: Binoy Jayan
Cc: linux-kernel, Arnd Bergmann, Rajendra, Mark Brown,
David S. Miller, Jarod Wilson, Masahiro Yamada
The semaphore 'proc_sem' is used as a simple mutex, so
it should be written as one. Semaphores are going away in the future.
Signed-off-by: Binoy Jayan <binoy.jayan@linaro.org>
---
This patch is part of a bigger effort to eliminate unwanted
semaphores from the linux kernel.
drivers/net/ethernet/micrel/ksz884x.c | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/drivers/net/ethernet/micrel/ksz884x.c b/drivers/net/ethernet/micrel/ksz884x.c
index ee1c78a..9664666 100644
--- a/drivers/net/ethernet/micrel/ksz884x.c
+++ b/drivers/net/ethernet/micrel/ksz884x.c
@@ -1456,7 +1456,7 @@ struct dev_info {
* @adapter: Adapter device information.
* @port: Port information.
* @monitor_time_info: Timer to monitor ports.
- * @proc_sem: Semaphore for proc accessing.
+ * @proc_mutex: Mutex for proc accessing.
* @id: Device ID.
* @mii_if: MII interface information.
* @advertising: Temporary variable to store advertised settings.
@@ -1470,7 +1470,7 @@ struct dev_priv {
struct ksz_port port;
struct ksz_timer_info monitor_timer_info;
- struct semaphore proc_sem;
+ struct mutex proc_mutex;
int id;
struct mii_if_info mii_if;
@@ -5842,7 +5842,7 @@ static int netdev_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd)
int result = 0;
struct mii_ioctl_data *data = if_mii(ifr);
- if (down_interruptible(&priv->proc_sem))
+ if (mutex_lock_interruptible(&priv->proc_mutex))
return -ERESTARTSYS;
switch (cmd) {
@@ -5876,7 +5876,7 @@ static int netdev_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd)
result = -EOPNOTSUPP;
}
- up(&priv->proc_sem);
+ mutex_unlock(&priv->proc_mutex);
return result;
}
@@ -6805,7 +6805,7 @@ static int __init netdev_init(struct net_device *dev)
dev->features |= dev->hw_features;
- sema_init(&priv->proc_sem, 1);
+ mutex_init(&priv->proc_mutex);
priv->mii_if.phy_id_mask = 0x1;
priv->mii_if.reg_num_mask = 0x7;
--
Binoy Jayan
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] net: ethernet: micrel: ksz884x: Replace semaphore proc_sem with mutex
2017-06-08 10:03 [PATCH] net: ethernet: micrel: ksz884x: Replace semaphore proc_sem with mutex Binoy Jayan
@ 2017-06-08 14:59 ` Arnd Bergmann
0 siblings, 0 replies; 2+ messages in thread
From: Arnd Bergmann @ 2017-06-08 14:59 UTC (permalink / raw)
To: Binoy Jayan
Cc: Linux Kernel Mailing List, Rajendra, Mark Brown, David S. Miller,
Jarod Wilson, Masahiro Yamada
On Thu, Jun 8, 2017 at 12:03 PM, Binoy Jayan <binoy.jayan@linaro.org> wrote:
> The semaphore 'proc_sem' is used as a simple mutex, so
> it should be written as one. Semaphores are going away in the future.
>
> Signed-off-by: Binoy Jayan <binoy.jayan@linaro.org>
> ---
>
> This patch is part of a bigger effort to eliminate unwanted
> semaphores from the linux kernel.
Looks good,
Reviewed-by: Arnd Bergmann <arnd@arndb.de>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2017-06-08 14:59 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-08 10:03 [PATCH] net: ethernet: micrel: ksz884x: Replace semaphore proc_sem with mutex Binoy Jayan
2017-06-08 14:59 ` Arnd Bergmann
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).