Commit d3953819 authored by Dmytro Linkin's avatar Dmytro Linkin Committed by David S. Miller

netdevsim: Add max_vfs to bus_dev

Currently there is no limit to the number of VFs netdevsim can enable.
In a real systems this value exist and used by the driver.
Fore example, some features might need to consider this value when
allocating memory.

Expose max_vfs variable to debugfs as configurable resource. If are VFs
configured (num_vfs != 0) then changing of max_vfs not allowed.
Co-developed-by: default avatarYuval Avnery <yuvalav@nvidia.com>
Signed-off-by: default avatarYuval Avnery <yuvalav@nvidia.com>
Signed-off-by: default avatarDmytro Linkin <dlinkin@nvidia.com>
Reviewed-by: default avatarJiri Pirko <jiri@nvidia.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 53c7bb55
...@@ -27,9 +27,9 @@ static struct nsim_bus_dev *to_nsim_bus_dev(struct device *dev) ...@@ -27,9 +27,9 @@ static struct nsim_bus_dev *to_nsim_bus_dev(struct device *dev)
static int nsim_bus_dev_vfs_enable(struct nsim_bus_dev *nsim_bus_dev, static int nsim_bus_dev_vfs_enable(struct nsim_bus_dev *nsim_bus_dev,
unsigned int num_vfs) unsigned int num_vfs)
{ {
nsim_bus_dev->vfconfigs = kcalloc(num_vfs, if (nsim_bus_dev->max_vfs < num_vfs)
sizeof(struct nsim_vf_config), return -ENOMEM;
GFP_KERNEL | __GFP_NOWARN);
if (!nsim_bus_dev->vfconfigs) if (!nsim_bus_dev->vfconfigs)
return -ENOMEM; return -ENOMEM;
nsim_bus_dev->num_vfs = num_vfs; nsim_bus_dev->num_vfs = num_vfs;
...@@ -39,8 +39,6 @@ static int nsim_bus_dev_vfs_enable(struct nsim_bus_dev *nsim_bus_dev, ...@@ -39,8 +39,6 @@ static int nsim_bus_dev_vfs_enable(struct nsim_bus_dev *nsim_bus_dev,
static void nsim_bus_dev_vfs_disable(struct nsim_bus_dev *nsim_bus_dev) static void nsim_bus_dev_vfs_disable(struct nsim_bus_dev *nsim_bus_dev)
{ {
kfree(nsim_bus_dev->vfconfigs);
nsim_bus_dev->vfconfigs = NULL;
nsim_bus_dev->num_vfs = 0; nsim_bus_dev->num_vfs = 0;
} }
...@@ -56,7 +54,7 @@ nsim_bus_dev_numvfs_store(struct device *dev, struct device_attribute *attr, ...@@ -56,7 +54,7 @@ nsim_bus_dev_numvfs_store(struct device *dev, struct device_attribute *attr,
if (ret) if (ret)
return ret; return ret;
rtnl_lock(); mutex_lock(&nsim_bus_dev->vfs_lock);
if (nsim_bus_dev->num_vfs == num_vfs) if (nsim_bus_dev->num_vfs == num_vfs)
goto exit_good; goto exit_good;
if (nsim_bus_dev->num_vfs && num_vfs) { if (nsim_bus_dev->num_vfs && num_vfs) {
...@@ -74,7 +72,7 @@ nsim_bus_dev_numvfs_store(struct device *dev, struct device_attribute *attr, ...@@ -74,7 +72,7 @@ nsim_bus_dev_numvfs_store(struct device *dev, struct device_attribute *attr,
exit_good: exit_good:
ret = count; ret = count;
exit_unlock: exit_unlock:
rtnl_unlock(); mutex_unlock(&nsim_bus_dev->vfs_lock);
return ret; return ret;
} }
...@@ -92,6 +90,73 @@ static struct device_attribute nsim_bus_dev_numvfs_attr = ...@@ -92,6 +90,73 @@ static struct device_attribute nsim_bus_dev_numvfs_attr =
__ATTR(sriov_numvfs, 0664, nsim_bus_dev_numvfs_show, __ATTR(sriov_numvfs, 0664, nsim_bus_dev_numvfs_show,
nsim_bus_dev_numvfs_store); nsim_bus_dev_numvfs_store);
ssize_t nsim_bus_dev_max_vfs_read(struct file *file,
char __user *data,
size_t count, loff_t *ppos)
{
struct nsim_bus_dev *nsim_bus_dev = file->private_data;
char buf[11];
size_t len;
len = snprintf(buf, sizeof(buf), "%u\n", nsim_bus_dev->max_vfs);
if (len < 0)
return len;
return simple_read_from_buffer(data, count, ppos, buf, len);
}
ssize_t nsim_bus_dev_max_vfs_write(struct file *file,
const char __user *data,
size_t count, loff_t *ppos)
{
struct nsim_bus_dev *nsim_bus_dev = file->private_data;
struct nsim_vf_config *vfconfigs;
ssize_t ret;
char buf[10];
u32 val;
if (*ppos != 0)
return 0;
if (count >= sizeof(buf))
return -ENOSPC;
mutex_lock(&nsim_bus_dev->vfs_lock);
/* Reject if VFs are configured */
if (nsim_bus_dev->num_vfs) {
ret = -EBUSY;
goto unlock;
}
ret = copy_from_user(buf, data, count);
if (ret) {
ret = -EFAULT;
goto unlock;
}
buf[count] = '\0';
ret = kstrtouint(buf, 10, &val);
if (ret) {
ret = -EIO;
goto unlock;
}
vfconfigs = kcalloc(val, sizeof(struct nsim_vf_config), GFP_KERNEL | __GFP_NOWARN);
if (!vfconfigs) {
ret = -ENOMEM;
goto unlock;
}
kfree(nsim_bus_dev->vfconfigs);
nsim_bus_dev->vfconfigs = vfconfigs;
nsim_bus_dev->max_vfs = val;
*ppos += count;
ret = count;
unlock:
mutex_unlock(&nsim_bus_dev->vfs_lock);
return ret;
}
static ssize_t static ssize_t
new_port_store(struct device *dev, struct device_attribute *attr, new_port_store(struct device *dev, struct device_attribute *attr,
const char *buf, size_t count) const char *buf, size_t count)
...@@ -311,6 +376,8 @@ static struct bus_type nsim_bus = { ...@@ -311,6 +376,8 @@ static struct bus_type nsim_bus = {
.num_vf = nsim_num_vf, .num_vf = nsim_num_vf,
}; };
#define NSIM_BUS_DEV_MAX_VFS 4
static struct nsim_bus_dev * static struct nsim_bus_dev *
nsim_bus_dev_new(unsigned int id, unsigned int port_count) nsim_bus_dev_new(unsigned int id, unsigned int port_count)
{ {
...@@ -329,15 +396,28 @@ nsim_bus_dev_new(unsigned int id, unsigned int port_count) ...@@ -329,15 +396,28 @@ nsim_bus_dev_new(unsigned int id, unsigned int port_count)
nsim_bus_dev->dev.type = &nsim_bus_dev_type; nsim_bus_dev->dev.type = &nsim_bus_dev_type;
nsim_bus_dev->port_count = port_count; nsim_bus_dev->port_count = port_count;
nsim_bus_dev->initial_net = current->nsproxy->net_ns; nsim_bus_dev->initial_net = current->nsproxy->net_ns;
nsim_bus_dev->max_vfs = NSIM_BUS_DEV_MAX_VFS;
mutex_init(&nsim_bus_dev->nsim_bus_reload_lock); mutex_init(&nsim_bus_dev->nsim_bus_reload_lock);
mutex_init(&nsim_bus_dev->vfs_lock);
/* Disallow using nsim_bus_dev */ /* Disallow using nsim_bus_dev */
smp_store_release(&nsim_bus_dev->init, false); smp_store_release(&nsim_bus_dev->init, false);
nsim_bus_dev->vfconfigs = kcalloc(nsim_bus_dev->max_vfs,
sizeof(struct nsim_vf_config),
GFP_KERNEL | __GFP_NOWARN);
if (!nsim_bus_dev->vfconfigs) {
err = -ENOMEM;
goto err_nsim_bus_dev_id_free;
}
err = device_register(&nsim_bus_dev->dev); err = device_register(&nsim_bus_dev->dev);
if (err) if (err)
goto err_nsim_bus_dev_id_free; goto err_nsim_vfs_free;
return nsim_bus_dev; return nsim_bus_dev;
err_nsim_vfs_free:
kfree(nsim_bus_dev->vfconfigs);
err_nsim_bus_dev_id_free: err_nsim_bus_dev_id_free:
ida_free(&nsim_bus_dev_ids, nsim_bus_dev->dev.id); ida_free(&nsim_bus_dev_ids, nsim_bus_dev->dev.id);
err_nsim_bus_dev_free: err_nsim_bus_dev_free:
...@@ -351,6 +431,7 @@ static void nsim_bus_dev_del(struct nsim_bus_dev *nsim_bus_dev) ...@@ -351,6 +431,7 @@ static void nsim_bus_dev_del(struct nsim_bus_dev *nsim_bus_dev)
smp_store_release(&nsim_bus_dev->init, false); smp_store_release(&nsim_bus_dev->init, false);
device_unregister(&nsim_bus_dev->dev); device_unregister(&nsim_bus_dev->dev);
ida_free(&nsim_bus_dev_ids, nsim_bus_dev->dev.id); ida_free(&nsim_bus_dev_ids, nsim_bus_dev->dev.id);
kfree(nsim_bus_dev->vfconfigs);
kfree(nsim_bus_dev); kfree(nsim_bus_dev);
} }
......
...@@ -192,6 +192,14 @@ static const struct file_operations nsim_dev_trap_fa_cookie_fops = { ...@@ -192,6 +192,14 @@ static const struct file_operations nsim_dev_trap_fa_cookie_fops = {
.owner = THIS_MODULE, .owner = THIS_MODULE,
}; };
static const struct file_operations nsim_dev_max_vfs_fops = {
.open = simple_open,
.read = nsim_bus_dev_max_vfs_read,
.write = nsim_bus_dev_max_vfs_write,
.llseek = generic_file_llseek,
.owner = THIS_MODULE,
};
static int nsim_dev_debugfs_init(struct nsim_dev *nsim_dev) static int nsim_dev_debugfs_init(struct nsim_dev *nsim_dev)
{ {
char dev_ddir_name[sizeof(DRV_NAME) + 10]; char dev_ddir_name[sizeof(DRV_NAME) + 10];
...@@ -231,6 +239,11 @@ static int nsim_dev_debugfs_init(struct nsim_dev *nsim_dev) ...@@ -231,6 +239,11 @@ static int nsim_dev_debugfs_init(struct nsim_dev *nsim_dev)
debugfs_create_bool("fail_trap_policer_counter_get", 0600, debugfs_create_bool("fail_trap_policer_counter_get", 0600,
nsim_dev->ddir, nsim_dev->ddir,
&nsim_dev->fail_trap_policer_counter_get); &nsim_dev->fail_trap_policer_counter_get);
nsim_dev->max_vfs = debugfs_create_file("max_vfs",
0600,
nsim_dev->ddir,
nsim_dev->nsim_bus_dev,
&nsim_dev_max_vfs_fops);
nsim_udp_tunnels_debugfs_create(nsim_dev); nsim_udp_tunnels_debugfs_create(nsim_dev);
return 0; return 0;
} }
......
...@@ -212,6 +212,7 @@ struct nsim_dev { ...@@ -212,6 +212,7 @@ struct nsim_dev {
struct dentry *ddir; struct dentry *ddir;
struct dentry *ports_ddir; struct dentry *ports_ddir;
struct dentry *take_snapshot; struct dentry *take_snapshot;
struct dentry *max_vfs;
struct bpf_offload_dev *bpf_dev; struct bpf_offload_dev *bpf_dev;
bool bpf_bind_accept; bool bpf_bind_accept;
bool bpf_bind_verifier_accept; bool bpf_bind_verifier_accept;
...@@ -269,6 +270,13 @@ void nsim_fib_destroy(struct devlink *devlink, struct nsim_fib_data *fib_data); ...@@ -269,6 +270,13 @@ void nsim_fib_destroy(struct devlink *devlink, struct nsim_fib_data *fib_data);
u64 nsim_fib_get_val(struct nsim_fib_data *fib_data, u64 nsim_fib_get_val(struct nsim_fib_data *fib_data,
enum nsim_resource_id res_id, bool max); enum nsim_resource_id res_id, bool max);
ssize_t nsim_bus_dev_max_vfs_read(struct file *file,
char __user *data,
size_t count, loff_t *ppos);
ssize_t nsim_bus_dev_max_vfs_write(struct file *file,
const char __user *data,
size_t count, loff_t *ppos);
#if IS_ENABLED(CONFIG_XFRM_OFFLOAD) #if IS_ENABLED(CONFIG_XFRM_OFFLOAD)
void nsim_ipsec_init(struct netdevsim *ns); void nsim_ipsec_init(struct netdevsim *ns);
void nsim_ipsec_teardown(struct netdevsim *ns); void nsim_ipsec_teardown(struct netdevsim *ns);
...@@ -308,7 +316,9 @@ struct nsim_bus_dev { ...@@ -308,7 +316,9 @@ struct nsim_bus_dev {
struct net *initial_net; /* Purpose of this is to carry net pointer struct net *initial_net; /* Purpose of this is to carry net pointer
* during the probe time only. * during the probe time only.
*/ */
unsigned int max_vfs;
unsigned int num_vfs; unsigned int num_vfs;
struct mutex vfs_lock; /* Protects vfconfigs */
struct nsim_vf_config *vfconfigs; struct nsim_vf_config *vfconfigs;
/* Lock for devlink->reload_enabled in netdevsim module */ /* Lock for devlink->reload_enabled in netdevsim module */
struct mutex nsim_bus_reload_lock; struct mutex nsim_bus_reload_lock;
......
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