git: e06c67eef8fa - main - hptiop: Remove Giant locking around ioctls.
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Fri, 10 Dec 2021 01:38:03 UTC
The branch main has been updated by mav:
URL: https://cgit.FreeBSD.org/src/commit/?id=e06c67eef8faae3322c06056960562fde88fd7aa
commit e06c67eef8faae3322c06056960562fde88fd7aa
Author: Alexander Motin <mav@FreeBSD.org>
AuthorDate: 2021-12-10 01:35:21 +0000
Commit: Alexander Motin <mav@FreeBSD.org>
CommitDate: 2021-12-10 01:35:21 +0000
hptiop: Remove Giant locking around ioctls.
The methods have their own locking and don't require Giant.
MFC after: 2 weeks
---
sys/dev/hptiop/hptiop.c | 5 -----
1 file changed, 5 deletions(-)
diff --git a/sys/dev/hptiop/hptiop.c b/sys/dev/hptiop/hptiop.c
index 300d10ff57e1..773b30948ed9 100644
--- a/sys/dev/hptiop/hptiop.c
+++ b/sys/dev/hptiop/hptiop.c
@@ -214,8 +214,6 @@ static int hptiop_ioctl(ioctl_dev_t dev, u_long cmd, caddr_t data,
int ret = EFAULT;
struct hpt_iop_hba *hba = hba_from_dev(dev);
- mtx_lock(&Giant);
-
switch (cmd) {
case HPT_DO_IOCONTROL:
ret = hba->ops->do_ioctl(hba,
@@ -225,9 +223,6 @@ static int hptiop_ioctl(ioctl_dev_t dev, u_long cmd, caddr_t data,
ret = hptiop_rescan_bus(hba);
break;
}
-
- mtx_unlock(&Giant);
-
return ret;
}