On 01/03/2016 09:11, "Kavanagh, Mark B" <mark.b.kavan...@intel.com> wrote:

>Hi Daniele,
>
>One minor comment below.
>
>Thanks,
>Mark
>
>>---
>> lib/dpif-netdev.c   | 137
>>++++++++++++++++++++++++++++++----------------------
>> lib/dpif-provider.h |   3 +-
>> 2 files changed, 82 insertions(+), 58 deletions(-)
>>
>>diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c
>>index c9c7d16..2c11bd3 100644
>>--- a/lib/dpif-netdev.c
>>+++ b/lib/dpif-netdev.c
>>@@ -223,7 +223,9 @@ struct dp_netdev {
>>     ovsthread_key_t per_pmd_key;
>>
>>     /* Cpu mask for pin of pmd threads. */
>>+    char *requested_pmd_cmask;
>>     char *pmd_cmask;
>>+
>>     uint64_t last_tnl_conf_seq;
>> };
>>
>>@@ -2450,79 +2452,42 @@ dpif_netdev_operate(struct dpif *dpif, struct
>>dpif_op **ops, size_t
>>n_ops)
>> /* Returns true if the configuration for rx queues or cpu mask
>>  * is changed. */
>
>This comment should probably be changed, to just 'for rx queues is
>changed.'.

You're right, changed

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to