LVS
lvs-devel
Google
 
Web LinuxVirtualServer.org

Re: [PATCH 1/6] ipvs: timeout tables do not need GFP_ATOMIC allocation

To: "Julian Anastasov" <ja@xxxxxx>
Subject: Re: [PATCH 1/6] ipvs: timeout tables do not need GFP_ATOMIC allocation
Cc: "Simon Horman" <horms@xxxxxxxxxxxx>, lvs-devel@xxxxxxxxxxxxxxx
From: "Hans Schillstrom" <hans@xxxxxxxxxxxxxxx>
Date: Fri, 13 Apr 2012 22:14:19 +0200 (CEST)
Hello
All 6 patches look fine to me, and are all called from process context

>---- Original Message ----
>From: Julian Anastasov <ja@xxxxxx>
>To: "Simon Horman" <horms@xxxxxxxxxxxx>
>Cc: lvs-devel@xxxxxxxxxxxxxxx
>Sent: Fri, Apr 13, 2012, 3:49 PM
>Subject: [PATCH 1/6] ipvs: timeout tables do not need GFP_ATOMIC allocation
>
>They are called only on initialization.
>
>Signed-off-by: Julian Anastasov <ja@xxxxxx>

Signed-off-by: Hans Schillstrom <hans@xxxxxxxxxxxxxxx>

>---
> net/netfilter/ipvs/ip_vs_proto.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
>diff --git a/net/netfilter/ipvs/ip_vs_proto.c 
>b/net/netfilter/ipvs/ip_vs_proto.c
>index 8531293..f549ff7 100644
>--- a/net/netfilter/ipvs/ip_vs_proto.c
>+++ b/net/netfilter/ipvs/ip_vs_proto.c
>@@ -197,7 +197,7 @@ void ip_vs_protocol_timeout_change(struct netns_ipvs 
>*ipvs, int flags)
> int *
> ip_vs_create_timeout_table(int *table, int size)
> {
>-      return kmemdup(table, size, GFP_ATOMIC);
>+      return kmemdup(table, size, GFP_KERNEL);
> }
> 
> 
>-- 
>1.7.3.4
>
>--
>To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
>the body of a message to majordomo@xxxxxxxxxxxxxxx
>More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe lvs-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

<Prev in Thread] Current Thread [Next in Thread>