diff mbox

[v2,4/6] batman-adv: remove packed from batadv_ogm_packet

Message ID 1366401149-31600-5-git-send-email-siwu@hrz.tu-chemnitz.de (mailing list archive)
State Superseded, archived
Commit 85cc6b9361fc988f4e3b9f103ba60ee23a93bbce
Headers

Commit Message

Simon Wunderlich April 19, 2013, 7:52 p.m. UTC 
  As we decreased the struct size from 26 to 24 byte, we can remove
__packed as the compile will not add any more padding.

Signed-off-by: Simon Wunderlich <siwu@hrz.tu-chemnitz.de>
---
 packet.h |    5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

Comments

Marek Lindner April 20, 2013, 9:41 a.m. UTC | #1 
On Saturday, April 20, 2013 03:52:27 Simon Wunderlich wrote:
> As we decreased the struct size from 26 to 24 byte, we can remove
> __packed as the compile will not add any more padding.

Same typo as before: compileR

Cheers,
Marek
diff mbox

Patch

diff --git a/packet.h b/packet.h
index c90856d..88d3aa6 100644
--- a/packet.h
+++ b/packet.h
@@ -178,7 +178,10 @@  struct batadv_ogm_packet {
 	uint8_t  reserved;
 	uint8_t  tq;
 	__be16   tvlv_len;
-} __packed;
+	/* __packed is not needed as the struct size is divisible by 4,
+	 * and the largest data type in this struct has a size of 4.
+	 */
+};
 
 #define BATADV_OGM_HLEN sizeof(struct batadv_ogm_packet)