Thread (8 messages) flat view 8 messages, 3 authors, 2017-11-15

Re: [PATCH] ipv6: sr: update the struct ipv6_sr_hdr

From: Edward Cree <hidden>
Date: 2017-11-14 14:14:06

On 14/11/17 12:37, David Miller wrote:
From: Ahmed Abdelsalam <redacted>
Date: Sun, 12 Nov 2017 21:37:01 +0100
quoted
diff --git a/include/uapi/linux/seg6.h b/include/uapi/linux/seg6.h
index 2f6fb0d..3f4b3ab 100644
--- a/include/uapi/linux/seg6.h
+++ b/include/uapi/linux/seg6.h
@@ -26,9 +26,9 @@ struct ipv6_sr_hdr {
 	__u8	hdrlen;
 	__u8	type;
 	__u8	segments_left;
-	__u8	first_segment;
+	__u8	last_entry;
This is user ABI and cannot be changed.

Sorry, folks should have considered these issues when the SR
changes were submitted.  This field must keep the name 'first_segment'
forever.
Surely renaming struct fields only changes the API, not the ABI?
Binaries compiled against the old struct definition will still behave the
 same (AFAICT the patch doesn't change how these fields are used), while
 sources being recompiled (so they care about the name change) can be
 changed.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help