On 02/02/2017 13:00, "Ben Pfaff" <[email protected]> wrote:
>On Fri, Jan 27, 2017 at 05:10:04PM -0800, Daniele Di Proietto wrote:
>> Currently we parse the 'other_config' column in Openvswitch table in
>> bridge.c. We extract the values (just 'pmd-cpu-mask' for now) and we
>> pass them down to the datapath, via different layers.
>>
>> If we want to pass other values to dpif-netdev.c (like we recently
>> discussed) we would have to touch ofproto.c, ofproto-dpif.c and dpif.c.
>>
>> This patch sends the entire other_config column to dpif-netdev, so that
>> dpif-netdev can extract the values it's interested in.
>>
>> No functional change.
>>
>> Signed-off-by: Daniele Di Proietto <[email protected]>
>> ---
>> I don't like that dpif-netdev receives the whole other_config column,
>> because it contains other values which are completely unrelated, but
>> unfortunately there's no better place in the database for datapath
>> specific configuration.
>
>Acked-by: Ben Pfaff <[email protected]>
Thanks for the review, pushed to master
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
