Home » Mailing list archives » linux-kernel » 2007 » September » 1

[PATCH 1/3] netlink: use the macro min(x,y) provided by <linux/kernel.h> instead

!MAILaRCHIVE_VOTE_RePLACE
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
[view in full thread]
From: Denis Cheng <crquan@...>
To: David S. Miller <davem@...>
Cc: <netdev@...>, <linux-kernel@...>, <cr_quan@...>
Subject: [PATCH 1/3] netlink: use the macro min(x,y) provided by &lt;linux/kernel.h&gt; instead
Date: Saturday, September 1, 2007 - 3:45 pm

Signed-off-by: Denis Cheng <crquan@gmail.com>
---
 net/netlink/af_netlink.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/net/netlink/af_netlink.c b/net/netlink/af_netlink.c
index 5681ce3..8bb14e3 100644
--- a/net/netlink/af_netlink.c
+++ b/net/netlink/af_netlink.c
@@ -1905,7 +1905,7 @@ static int __init netlink_proto_init(void)
 
 	order = get_bitmask_order(max) - 1 + PAGE_SHIFT;
 	max = (1UL << order) / sizeof(struct hlist_head);
-	order = get_bitmask_order(max > UINT_MAX ? UINT_MAX : max) - 1;
+	order = get_bitmask_order(min(max, (unsigned long)UINT_MAX)) - 1;
 
 	for (i = 0; i < MAX_LINKS; i++) {
 		struct nl_pid_hash *hash = &nl_table[i].hash;
-- 
1.5.3.rc7

-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
Messages in current thread:
[PATCH 1/3] netlink: use the macro min(x,y) provided by <..., Denis Cheng, (Sat Sep 1, 3:45 pm)
Re: [PATCH 1/3] netlink: use the macro min(x,y) provided by ..., David Miller, (Sun Sep 16, 7:35 pm)
[PATCH 2/3] netlink: the temp variable name max is ambiguous, Denis Cheng, (Sat Sep 1, 3:45 pm)
Re: [PATCH 2/3] netlink: the temp variable name max is ambig..., David Miller, (Sun Sep 16, 7:36 pm)
Re: [PATCH 2/3] netlink: the temp variable name max is ambig..., rae l, (Thu Sep 20, 12:56 pm)
[PATCH 3/3] netlink: use a statically allocated nl_table ins..., Denis Cheng, (Sat Sep 1, 3:45 pm)
Re: [PATCH 3/3] netlink: use a statically allocated nl_table..., David Miller, (Sun Sep 16, 7:38 pm)