summaryrefslogtreecommitdiffstats
path: root/net/bootp.h
diff options
context:
space:
mode:
authorDenis Pynkin <denis.pynkin@collabora.com>2017-07-21 19:28:42 +0300
committerJoel Stanley <joel@jms.id.au>2018-02-13 15:45:53 +1030
commit82bdfde43e0334138c6888f74533728dc124fef5 (patch)
tree9d0e0e8a97502e477093f219438196a2b37092d0 /net/bootp.h
parentf88ff976e92b60a99cd8ff057cb87c4a74c1cd54 (diff)
downloadtalos-obmc-uboot-82bdfde43e0334138c6888f74533728dc124fef5.tar.gz
talos-obmc-uboot-82bdfde43e0334138c6888f74533728dc124fef5.zip
net: Use packed structures for networking
PXE boot is broken with GCC 7.1 due option '-fstore-merging' enabled by default for '-O2': BOOTP broadcast 1 data abort pc : [<8ff8bb30>] lr : [<00004f1f>] reloc pc : [<17832b30>] lr : [<878abf1f>] sp : 8f558bc0 ip : 00000000 fp : 8ffef5a4 r10: 8ffed248 r9 : 8f558ee0 r8 : 8ffef594 r7 : 0000000e r6 : 8ffed700 r5 : 00000000 r4 : 8ffed74e r3 : 00060101 r2 : 8ffed230 r1 : 8ffed706 r0 : 00000ddd Flags: nzcv IRQs off FIQs off Mode SVC_32 Resetting CPU ... Core reason is usage of structures for network headers without packed attribute. Reviewed-by: Yauheni Kaliuta <yauheni.kaliuta@redhat.com> Signed-off-by: Denis Pynkin <denis.pynkin@collabora.com> Acked-by: Joe Hershberger <joe.hershberger@ni.com> (cherry picked from commit 704f3acfcf55343043bbed01c5fb0a0094a68e8a) Signed-off-by: Joel Stanley <joel@jms.id.au>
Diffstat (limited to 'net/bootp.h')
-rw-r--r--net/bootp.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/bootp.h b/net/bootp.h
index fcb0a64e61..567340ec5d 100644
--- a/net/bootp.h
+++ b/net/bootp.h
@@ -49,7 +49,7 @@ struct bootp_hdr {
char bp_sname[64]; /* Server host name */
char bp_file[128]; /* Boot file name */
char bp_vend[OPT_FIELD_SIZE]; /* Vendor information */
-};
+} __attribute__((packed));
#define BOOTP_HDR_SIZE sizeof(struct bootp_hdr)
OpenPOWER on IntegriCloud