On May 31, 2013, at 11:23 , ext Jesse Gross wrote: > On Fri, May 31, 2013 at 4:53 PM, Fan Du <fan...@windriver.com> 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. Jarno _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
