On Tue, Sep 29, 2015 at 10:38 PM, Ben Pfaff <b...@nicira.com> wrote:
> On Tue, Sep 22, 2015 at 05:51:12PM -0700, Andy Zhou wrote:
>> dpdk datapath needs to run as root. Block the --user
>> option for now. It is likely we will revisit this issue for possibly
>> supporting --user option for dpdk datapath process as well.
>>
>> Signed-off-by: Andy Zhou <az...@nicira.com>
>
> Acked-by: Ben Pfaff <b...@nicira.com>
>
>> +    /* Reject --user option */
>> +    int i;
>> +    for (i =0; i< argc; i++) {
>
> I'd write the above line as:
>     for (int i = 0; i < argc; i++) {
>
>> +         if (!strcmp(argv[i], "--user")) {
>> +            VLOG_ERR("Can not specify both --dpdk and --user, aborting.");
>> +        }
>> +    }
>> +
>
> Thanks,
>
> Ben.
Applied with the white space fixes.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to