Commit 57ce9774 authored by Jiri Pirko's avatar Jiri Pirko Committed by David S. Miller

netdevsim: use ida for bus device ids

Instead of increments of u32 value, use ida to manage bus device ids.
Signed-off-by: default avatarJiri Pirko <jiri@mellanox.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 23d415da
...@@ -4,6 +4,7 @@ ...@@ -4,6 +4,7 @@
*/ */
#include <linux/device.h> #include <linux/device.h>
#include <linux/idr.h>
#include <linux/kernel.h> #include <linux/kernel.h>
#include <linux/rtnetlink.h> #include <linux/rtnetlink.h>
#include <linux/slab.h> #include <linux/slab.h>
...@@ -11,7 +12,7 @@ ...@@ -11,7 +12,7 @@
#include "netdevsim.h" #include "netdevsim.h"
static u32 nsim_bus_dev_id; static DEFINE_IDA(nsim_bus_dev_ids);
static struct nsim_bus_dev *to_nsim_bus_dev(struct device *dev) static struct nsim_bus_dev *to_nsim_bus_dev(struct device *dev)
{ {
...@@ -134,14 +135,19 @@ struct nsim_bus_dev *nsim_bus_dev_new(void) ...@@ -134,14 +135,19 @@ struct nsim_bus_dev *nsim_bus_dev_new(void)
if (!nsim_bus_dev) if (!nsim_bus_dev)
return ERR_PTR(-ENOMEM); return ERR_PTR(-ENOMEM);
nsim_bus_dev->dev.id = nsim_bus_dev_id++; err = ida_alloc(&nsim_bus_dev_ids, GFP_KERNEL);
if (err < 0)
goto err_nsim_bus_dev_free;
nsim_bus_dev->dev.id = err;
nsim_bus_dev->dev.bus = &nsim_bus; nsim_bus_dev->dev.bus = &nsim_bus;
nsim_bus_dev->dev.type = &nsim_bus_dev_type; nsim_bus_dev->dev.type = &nsim_bus_dev_type;
err = device_register(&nsim_bus_dev->dev); err = device_register(&nsim_bus_dev->dev);
if (err) if (err)
goto err_nsim_bus_dev_free; goto err_nsim_bus_dev_id_free;
return nsim_bus_dev; return nsim_bus_dev;
err_nsim_bus_dev_id_free:
ida_free(&nsim_bus_dev_ids, nsim_bus_dev->dev.id);
err_nsim_bus_dev_free: err_nsim_bus_dev_free:
kfree(nsim_bus_dev); kfree(nsim_bus_dev);
return ERR_PTR(err); return ERR_PTR(err);
...@@ -150,6 +156,7 @@ struct nsim_bus_dev *nsim_bus_dev_new(void) ...@@ -150,6 +156,7 @@ struct nsim_bus_dev *nsim_bus_dev_new(void)
void nsim_bus_dev_del(struct nsim_bus_dev *nsim_bus_dev) void nsim_bus_dev_del(struct nsim_bus_dev *nsim_bus_dev)
{ {
device_unregister(&nsim_bus_dev->dev); device_unregister(&nsim_bus_dev->dev);
ida_free(&nsim_bus_dev_ids, nsim_bus_dev->dev.id);
kfree(nsim_bus_dev); kfree(nsim_bus_dev);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment