From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: <"dex@dex.chen"@ruckuswireless.com> Received: from anubis (59-124-251-135.HINET-IP.hinet.net [59.124.251.135]) by dpdk.org (Postfix) with ESMTP id 189A911F5 for ; Thu, 2 Jul 2015 12:12:18 +0200 (CEST) Received: by anubis (Postfix, from userid 1000) id B29C31483305; Thu, 2 Jul 2015 18:12:16 +0800 (CST) From: Dex Chen To: dev@dpdk.org Date: Thu, 2 Jul 2015 18:12:13 +0800 Message-Id: <1435831933-13339-1-git-send-email-dex.chen@ruckuswireless.com> X-Mailer: git-send-email 2.1.4 Subject: [dpdk-dev] [PATCH] kni: allow per-net instances X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 02 Jul 2015 10:12:20 -0000 There is a global variable 'device_in_use' which is used to make sure only one instance is using /dev/kni device. If you were using LXC, you will find there is only one instance of KNI example could be run even differnt namespaces were created. In order to have /dev/kni used simultaneously in different namespaces, making all of global variables as per network namespace variables. With regard to single kernel thread mode, there will be one kernel thread for each of network namespace. Signed-off-by: Dex Chen --- lib/librte_eal/linuxapp/kni/kni_misc.c | 129 ++++++++++++++++++++++----------- 1 file changed, 85 insertions(+), 44 deletions(-) diff --git a/lib/librte_eal/linuxapp/kni/kni_misc.c b/lib/librte_eal/linuxapp/kni/kni_misc.c index 2e9fa89..5ba8ab8 100644 --- a/lib/librte_eal/linuxapp/kni/kni_misc.c +++ b/lib/librte_eal/linuxapp/kni/kni_misc.c @@ -28,6 +28,9 @@ #include #include #include +#include +#include +#include #include #include "kni_dev.h" @@ -90,18 +93,48 @@ static unsigned multiple_kthread_on = 0; #define KNI_DEV_IN_USE_BIT_NUM 0 /* Bit number for device in use */ -static volatile unsigned long device_in_use; /* device in use flag */ -static struct task_struct *kni_kthread; +static int kni_net_id; -/* kni list lock */ -static DECLARE_RWSEM(kni_list_lock); +struct kni_net { + volatile unsigned long device_in_use; /* device in use flag */ + struct task_struct *kni_kthread; + struct rw_semaphore kni_list_lock; + struct list_head kni_list_head; +}; + +static __net_init int kni_init_net(struct net *net) +{ + struct kni_net *knet = net_generic(net, kni_net_id); -/* kni list */ -static struct list_head kni_list_head = LIST_HEAD_INIT(kni_list_head); + /* Clear the bit of device in use */ + clear_bit(KNI_DEV_IN_USE_BIT_NUM, &knet->device_in_use); + + init_rwsem(&knet->kni_list_lock); + INIT_LIST_HEAD(&knet->kni_list_head); + + return 0; +} + +static __net_exit void kni_exit_net(struct net *net) +{ + /* + * Nothing to do here. + * Assuming all cleanup jobs were done in kni_release(). + */ +} + +static struct pernet_operations kni_net_ops = { + .init = kni_init_net, + .exit = kni_exit_net, + .id = &kni_net_id, + .size = sizeof(struct kni_net), +}; static int __init kni_init(void) { + int rc; + KNI_PRINT("######## DPDK kni module loading ########\n"); if (kni_parse_kthread_mode() < 0) { @@ -114,8 +147,9 @@ kni_init(void) return -EPERM; } - /* Clear the bit of device in use */ - clear_bit(KNI_DEV_IN_USE_BIT_NUM, &device_in_use); + rc = register_pernet_subsys(&kni_net_ops); + if (rc) + goto out; /* Configure the lo mode according to the input parameter */ kni_net_config_lo_mode(lo_mode); @@ -123,11 +157,16 @@ kni_init(void) KNI_PRINT("######## DPDK kni module loaded ########\n"); return 0; + +out: + misc_deregister(&kni_misc); + return rc; } static void __exit kni_exit(void) { + unregister_pernet_subsys(&kni_net_ops); misc_deregister(&kni_misc); KNI_PRINT("####### DPDK kni module unloaded #######\n"); } @@ -151,19 +190,22 @@ kni_parse_kthread_mode(void) static int kni_open(struct inode *inode, struct file *file) { - /* kni device can be opened by one user only, test and set bit */ - if (test_and_set_bit(KNI_DEV_IN_USE_BIT_NUM, &device_in_use)) + struct net *net = current->nsproxy->net_ns; + struct kni_net *knet = net_generic(net, kni_net_id); + + /* kni device can be opened by one user only per netns, test and set bit */ + if (test_and_set_bit(KNI_DEV_IN_USE_BIT_NUM, &knet->device_in_use)) return -EBUSY; /* Create kernel thread for single mode */ if (multiple_kthread_on == 0) { KNI_PRINT("Single kernel thread for all KNI devices\n"); /* Create kernel thread for RX */ - kni_kthread = kthread_run(kni_thread_single, NULL, + knet->kni_kthread = kthread_run(kni_thread_single, (void *)knet, "kni_single"); - if (IS_ERR(kni_kthread)) { + if (IS_ERR(knet->kni_kthread)) { KNI_ERR("Unable to create kernel threaed\n"); - return PTR_ERR(kni_kthread); + return PTR_ERR(knet->kni_kthread); } } else KNI_PRINT("Multiple kernel thread mode enabled\n"); @@ -176,17 +218,19 @@ kni_open(struct inode *inode, struct file *file) static int kni_release(struct inode *inode, struct file *file) { + struct net *net = current->nsproxy->net_ns; + struct kni_net *knet = net_generic(net, kni_net_id); struct kni_dev *dev, *n; /* Stop kernel thread for single mode */ if (multiple_kthread_on == 0) { /* Stop kernel thread */ - kthread_stop(kni_kthread); - kni_kthread = NULL; + kthread_stop(knet->kni_kthread); + knet->kni_kthread = NULL; } - down_write(&kni_list_lock); - list_for_each_entry_safe(dev, n, &kni_list_head, list) { + down_write(&knet->kni_list_lock); + list_for_each_entry_safe(dev, n, &knet->kni_list_head, list) { /* Stop kernel thread for multiple mode */ if (multiple_kthread_on && dev->pthread != NULL) { kthread_stop(dev->pthread); @@ -199,10 +243,10 @@ kni_release(struct inode *inode, struct file *file) kni_dev_remove(dev); list_del(&dev->list); } - up_write(&kni_list_lock); + up_write(&knet->kni_list_lock); /* Clear the bit of device in use */ - clear_bit(KNI_DEV_IN_USE_BIT_NUM, &device_in_use); + clear_bit(KNI_DEV_IN_USE_BIT_NUM, &knet->device_in_use); KNI_PRINT("/dev/kni closed\n"); @@ -210,15 +254,16 @@ kni_release(struct inode *inode, struct file *file) } static int -kni_thread_single(void *unused) +kni_thread_single(void *data) { + struct kni_net *knet = data; int j; struct kni_dev *dev; while (!kthread_should_stop()) { - down_read(&kni_list_lock); + down_read(&knet->kni_list_lock); for (j = 0; j < KNI_RX_LOOP_NUM; j++) { - list_for_each_entry(dev, &kni_list_head, list) { + list_for_each_entry(dev, &knet->kni_list_head, list) { #ifdef RTE_KNI_VHOST kni_chk_vhost_rx(dev); #else @@ -227,7 +272,7 @@ kni_thread_single(void *unused) kni_net_poll_resp(dev); } } - up_read(&kni_list_lock); + up_read(&knet->kni_list_lock); #ifdef RTE_KNI_PREEMPT_DEFAULT /* reschedule out for a while */ schedule_timeout_interruptible(usecs_to_jiffies( \ @@ -305,8 +350,9 @@ kni_check_param(struct kni_dev *kni, struct rte_kni_device_info *dev) } static int -kni_ioctl_create(unsigned int ioctl_num, unsigned long ioctl_param) +kni_ioctl_create(struct net *net, unsigned int ioctl_num, unsigned long ioctl_param) { + struct kni_net *knet = net_generic(net, kni_net_id); int ret; struct rte_kni_device_info dev_info; struct pci_dev *pci = NULL; @@ -314,7 +360,6 @@ kni_ioctl_create(unsigned int ioctl_num, unsigned long ioctl_param) struct net_device *net_dev = NULL; struct net_device *lad_dev = NULL; struct kni_dev *kni, *dev, *n; - struct net *net; printk(KERN_INFO "KNI: Creating kni...\n"); /* Check the buffer size, to avoid warning */ @@ -339,14 +384,14 @@ kni_ioctl_create(unsigned int ioctl_num, unsigned long ioctl_param) } /* Check if it has been created */ - down_read(&kni_list_lock); - list_for_each_entry_safe(dev, n, &kni_list_head, list) { + down_read(&knet->kni_list_lock); + list_for_each_entry_safe(dev, n, &knet->kni_list_head, list) { if (kni_check_param(dev, &dev_info) < 0) { - up_read(&kni_list_lock); + up_read(&knet->kni_list_lock); return -EINVAL; } } - up_read(&kni_list_lock); + up_read(&knet->kni_list_lock); net_dev = alloc_netdev(sizeof(struct kni_dev), dev_info.name, #ifdef NET_NAME_UNKNOWN @@ -358,13 +403,7 @@ kni_ioctl_create(unsigned int ioctl_num, unsigned long ioctl_param) return -EBUSY; } - net = get_net_ns_by_pid(task_pid_vnr(current)); - if (IS_ERR(net)) { - free_netdev(net_dev); - return PTR_ERR(net); - } dev_net_set(net_dev, net); - put_net(net); kni = netdev_priv(net_dev); @@ -494,16 +533,17 @@ kni_ioctl_create(unsigned int ioctl_num, unsigned long ioctl_param) wake_up_process(kni->pthread); } - down_write(&kni_list_lock); - list_add(&kni->list, &kni_list_head); - up_write(&kni_list_lock); + down_write(&knet->kni_list_lock); + list_add(&kni->list, &knet->kni_list_head); + up_write(&knet->kni_list_lock); return 0; } static int -kni_ioctl_release(unsigned int ioctl_num, unsigned long ioctl_param) +kni_ioctl_release(struct net *net, unsigned int ioctl_num, unsigned long ioctl_param) { + struct kni_net *knet = net_generic(net, kni_net_id); int ret = -EINVAL; struct kni_dev *dev, *n; struct rte_kni_device_info dev_info; @@ -521,8 +561,8 @@ kni_ioctl_release(unsigned int ioctl_num, unsigned long ioctl_param) if (strlen(dev_info.name) == 0) return ret; - down_write(&kni_list_lock); - list_for_each_entry_safe(dev, n, &kni_list_head, list) { + down_write(&knet->kni_list_lock); + list_for_each_entry_safe(dev, n, &knet->kni_list_head, list) { if (strncmp(dev->name, dev_info.name, RTE_KNI_NAMESIZE) != 0) continue; @@ -539,7 +579,7 @@ kni_ioctl_release(unsigned int ioctl_num, unsigned long ioctl_param) ret = 0; break; } - up_write(&kni_list_lock); + up_write(&knet->kni_list_lock); printk(KERN_INFO "KNI: %s release kni named %s\n", (ret == 0 ? "Successfully" : "Unsuccessfully"), dev_info.name); @@ -552,6 +592,7 @@ kni_ioctl(struct inode *inode, unsigned long ioctl_param) { int ret = -EINVAL; + struct net *net = current->nsproxy->net_ns; KNI_DBG("IOCTL num=0x%0x param=0x%0lx \n", ioctl_num, ioctl_param); @@ -563,10 +604,10 @@ kni_ioctl(struct inode *inode, /* For test only, not used */ break; case _IOC_NR(RTE_KNI_IOCTL_CREATE): - ret = kni_ioctl_create(ioctl_num, ioctl_param); + ret = kni_ioctl_create(net, ioctl_num, ioctl_param); break; case _IOC_NR(RTE_KNI_IOCTL_RELEASE): - ret = kni_ioctl_release(ioctl_num, ioctl_param); + ret = kni_ioctl_release(net, ioctl_num, ioctl_param); break; default: KNI_DBG("IOCTL default \n"); -- 2.1.4