Skip to content

Commit a4d656d

Browse files
author
Lai Jiangshan
committed
block/rsxx: use ida_simple_get()
Signed-off-by: Lai Jiangshan <laijs@cn.fujitsu.com>
1 parent 3d5c112 commit a4d656d

File tree

1 file changed

+4
-16
lines changed

1 file changed

+4
-16
lines changed

drivers/block/rsxx/core.c

Lines changed: 4 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,6 @@ MODULE_PARM_DESC(sync_start, "On by Default: Driver load will not complete "
5858
"until the card startup has completed.");
5959

6060
static DEFINE_IDA(rsxx_disk_ida);
61-
static DEFINE_SPINLOCK(rsxx_ida_lock);
6261

6362
/* --------------------Debugfs Setup ------------------- */
6463

@@ -823,19 +822,10 @@ static int rsxx_pci_probe(struct pci_dev *dev,
823822
card->dev = dev;
824823
pci_set_drvdata(dev, card);
825824

826-
do {
827-
if (!ida_pre_get(&rsxx_disk_ida, GFP_KERNEL)) {
828-
st = -ENOMEM;
829-
goto failed_ida_get;
830-
}
831-
832-
spin_lock(&rsxx_ida_lock);
833-
st = ida_get_new(&rsxx_disk_ida, &card->disk_id);
834-
spin_unlock(&rsxx_ida_lock);
835-
} while (st == -EAGAIN);
836-
837-
if (st)
825+
st = ida_simple_get(&rsxx_disk_ida, 0, 0, GFP_KERNEL);
826+
if (st < 0)
838827
goto failed_ida_get;
828+
card->disk_id = st;
839829

840830
st = pci_enable_device(dev);
841831
if (st)
@@ -1036,9 +1026,7 @@ static int rsxx_pci_probe(struct pci_dev *dev,
10361026
failed_dma_mask:
10371027
pci_disable_device(dev);
10381028
failed_enable:
1039-
spin_lock(&rsxx_ida_lock);
1040-
ida_remove(&rsxx_disk_ida, card->disk_id);
1041-
spin_unlock(&rsxx_ida_lock);
1029+
ida_simple_remove(&rsxx_disk_ida, card->disk_id);
10421030
failed_ida_get:
10431031
kfree(card);
10441032

0 commit comments

Comments
 (0)