linux-kernel - [PATCH] net: ethernet: micrel: ksz884x: Replace semaphore proc

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Date:   Thu,  8 Jun 2017 15:33:47 +0530
From:   Binoy Jayan <binoy.jayan@...aro.org>
To:     Binoy Jayan <binoy.jayan@...aro.org>
Cc:     linux-kernel@...r.kernel.org, Arnd Bergmann <arnd@...db.de>,
        Rajendra <rnayak@...eaurora.org>,
        Mark Brown <broonie@...nel.org>,
        "David S. Miller" <davem@...emloft.net>,
        Jarod Wilson <jarod@...hat.com>,
        Masahiro Yamada <yamada.masahiro@...ionext.com>
Subject: [PATCH] net: ethernet: micrel: ksz884x: Replace semaphore proc_sem with mutex

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@...aro.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