On Fri, May 31, 2013 at 7:59 PM, Rajahalme, Jarno (NSN - FI/Espoo)
<[email protected]> wrote:
> On May 31, 2013, at 11:23 , ext Jesse Gross wrote:
>
>> On Fri, May 31, 2013 at 4:53 PM, Fan Du <[email protected]> wrote:
>>> [ovs-dev] [PATCH] Use EEXIST as return error when adding vport
>>>
>>> Hi, Jesse
>>>
>>> Those two patches attempts to use EEXIST as return error when trying
>>> to add an vport which is actually already in place as the reason to user.
>>> And furthermore, we warn user in any reason other than EEXIST too.
>>
>> It's not sufficient to update both the userspace and kernel because
>> there's no requirement that the same versions be paired together.
>> Therefore, after this patch the kernel won't work with old userspace
>> versions and userspace will not work with old kernels.
>>
>> I don't think there is really a good way to solve this but the change
>> seems mostly aesthetic to me at this point.
>
> Maybe there should be a "known (aesthetic) issues" file somewhere, where 
> things like this could be recorded.
> Could be also useful if there comes a time when things are cleaned up without 
> worries about old versions.

I guess, although in this case I don't really see a strong argument
for either one or the other.
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to