diff mbox

__aligned(size) is preferred over __attribute__((aligned(size)))

Message ID 1424641236-7671-1-git-send-email-ameenali023@gmail.com
State Accepted, archived
Delegated to: David Miller
Headers show

Commit Message

Ameen Ali Feb. 22, 2015, 9:40 p.m. UTC
Signed-off-by: Ameen Ali <AmeenAli023@gmail.com>
---
 net/compat.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

Comments

David Miller Feb. 22, 2015, 10:02 p.m. UTC | #1
cFrom: Ameen Ali <ameenali023@gmail.com>
Date: Sun, 22 Feb 2015 23:40:36 +0200

> Signed-off-by: Ameen Ali <AmeenAli023@gmail.com>

Applied, but please use a proper Subject line next time,
in particular you should put a proper subsystem prefix.

In this case, "net: " would be an appropriate prefix.

Otherwise people scanning the shortlog in the repository
cannot tell where you are making your changes.

What if we had 100 commits all doing this same transformation?
It is the subsystem prefix that helps people see what is unique
in each such change.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/net/compat.c b/net/compat.c
index 3236b41..49c6a8f 100644
--- a/net/compat.c
+++ b/net/compat.c
@@ -508,25 +508,25 @@  COMPAT_SYSCALL_DEFINE5(getsockopt, int, fd, int, level, int, optname,
 struct compat_group_req {
 	__u32				 gr_interface;
 	struct __kernel_sockaddr_storage gr_group
-		__attribute__ ((aligned(4)));
+		__aligned(4);
 } __packed;
 
 struct compat_group_source_req {
 	__u32				 gsr_interface;
 	struct __kernel_sockaddr_storage gsr_group
-		__attribute__ ((aligned(4)));
+		__aligned(4);
 	struct __kernel_sockaddr_storage gsr_source
-		__attribute__ ((aligned(4)));
+		__aligned(4);
 } __packed;
 
 struct compat_group_filter {
 	__u32				 gf_interface;
 	struct __kernel_sockaddr_storage gf_group
-		__attribute__ ((aligned(4)));
+		__aligned(4);
 	__u32				 gf_fmode;
 	__u32				 gf_numsrc;
 	struct __kernel_sockaddr_storage gf_slist[1]
-		__attribute__ ((aligned(4)));
+		__aligned(4);
 } __packed;
 
 #define __COMPAT_GF0_SIZE (sizeof(struct compat_group_filter) - \