diff options
| author | Marcelo Ricardo Leitner <marcelo.leitner@gmail.com> | 2017-02-07 11:37:56 -0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2017-02-07 14:11:04 -0500 |
| commit | 85c727b5948344c8d559e2fda8925e9ddd41c29a (patch) | |
| tree | ca4a55955cedd8871f53b00e859b3d51b9489ace | |
| parent | bb580ad698aeb4e5455d701c228c50355f84c056 (diff) | |
sctp: drop __packed from almost all SCTP structures
__packed is considered harmful as it potentially generates code that
doesn't perform well and its usage should be avoided as much as
possible.
This patch drops __packed from all SCTP structures except one, which is
sctp_signed_cookie. In there it's required, as per changelog on
commit 9834a2bb4970 ("[SCTP]: Fix sctp_cookie alignment in the packet.").
After this patch, no alignment changes neither in x86 or x86_64 and
no exceptions were noticed during testing on both archs.
Code size for SCTP module also didn't change with this patch.
Cc: David Miller <davem@davemloft.net>
Cc: David Laight <David.Laight@ACULAB.COM>
Signed-off-by: Marcelo Ricardo Leitner <marcelo.leitner@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
| -rw-r--r-- | include/linux/sctp.h | 80 | ||||
| -rw-r--r-- | include/net/sctp/structs.h | 2 |
2 files changed, 41 insertions, 41 deletions
diff --git a/include/linux/sctp.h b/include/linux/sctp.h |