On Fri, Sep 30, 2016 at 10:51 AM, Jiri Benc <jb...@redhat.com> wrote: > On Fri, 30 Sep 2016 10:43:42 -0700, pravin shelar wrote: >> There is already struct mpls_label defined in uapi mpls header file. >> Is there reason for not using it here? > > I think this should be unified as well. But it's for a different patch > not related to this patchset. This patch just moves the existing > mpls_hdr and mpls_shim_hdr definition to a new place. > > In general, I think the mpls code deserves a round of overall cleanups. > It should not block this patchset, though. It can be considered as a > step towards that goal. >
Alright, I do not oppose this patch going in as it is. Acked-by: Pravin B Shelar <pshe...@ovn.org>
