Thanks for the advice Stephen, new patch submitted!
-----Ursprüngliche Nachricht----- Von: David Miller [mailto:da...@davemloft.net] Gesendet: Dienstag, 19. April 2016 18:12 An: step...@networkplumber.org Cc: m...@pheise.de; arvid.bro...@alten.se; han...@stressinduktion.org; s...@queasysnail.net; hen...@austad.us; niko...@cumulusnetworks.com; tg...@suug.ch; linvi...@tuxdriver.com; go...@cumulusnetworks.com; d...@cumulusnetworks.com; era...@mellanox.com; a...@plumgrid.com; netdev@vger.kernel.org; Heise, Peter Betreff: Re: [PATCH net-next] net/hsr: Added support for HSR v1 From: Stephen Hemminger <step...@networkplumber.org> Date: Tue, 19 Apr 2016 08:21:00 -0700 > On Wed, 13 Apr 2016 13:52:22 +0200 > Peter Heise <m...@pheise.de> wrote: > >> diff --git a/include/uapi/linux/if_link.h >> b/include/uapi/linux/if_link.h index 9427f17..bb3a90b 100644 >> --- a/include/uapi/linux/if_link.h >> +++ b/include/uapi/linux/if_link.h >> @@ -773,6 +773,7 @@ enum { >> IFLA_HSR_SLAVE1, >> IFLA_HSR_SLAVE2, >> IFLA_HSR_MULTICAST_SPEC, /* Last byte of supervision addr */ >> + IFLA_HSR_VERSION, /* HSR version */ >> IFLA_HSR_SUPERVISION_ADDR, /* Supervision frame multicast addr */ >> IFLA_HSR_SEQ_NR, > > You added a new value into the middle of an enumeration field. > This breaks kernel ABI. Older applications (like iproute) would see > the wrong values. > > Please submit a new change which moves HSR_VERSION to the end of the > enum Good catch, Stephen. This mail has originated outside your organization, either from an external partner or the Global Internet. Keep this in mind if you answer this message. The information in this e-mail is confidential. The contents may not be disclosed or used by anyone other than the addressee. Access to this e-mail by anyone else is unauthorised. If you are not the intended recipient, please notify Airbus immediately and delete this e-mail. Airbus cannot accept any responsibility for the accuracy or completeness of this e-mail as it has been sent over public networks. If you have any concerns over the content of this message or its Accuracy or Integrity, please contact Airbus immediately. All outgoing e-mails from Airbus are checked using regularly updated virus scanning software but you should take whatever measures you deem to be appropriate to ensure that this message and any attachments are virus free.
