[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20080103.204705.35413740.davem@davemloft.net>
Date: Thu, 03 Jan 2008 20:47:05 -0800 (PST)
From: David Miller <davem@...emloft.net>
To: dada1@...mosbay.com
Cc: netdev@...r.kernel.org, acme@...hat.com
Subject: Re: [NET]: prot_inuse cleanups and optimizations
From: Eric Dumazet <dada1@...mosbay.com>
Date: Thu, 3 Jan 2008 18:41:54 +0100
> 1) Cleanups (all functions are prefixed by sock_prot_inuse)
>
> sock_prot_inc_use(prot) -> sock_prot_inuse_add(prot,-1)
> sock_prot_dec_use(prot) -> sock_prot_inuse_add(prot,-1)
> sock_prot_inuse() -> sock_prot_inuse_get()
>
> New functions :
>
> sock_prot_inuse_init() and sock_prot_inuse_free() to abstract pcounter use.
>
> 2) if CONFIG_PROC_FS=n, we can zap 'inuse' member from "struct proto",
> since nobody wants to read the inuse value.
>
> This saves 1372 bytes on i386/SMP and some cpu cycles.
>
> Signed-off-by: Eric Dumazet <dada1@...mosbay.com>
I hate the ifdef, but it's in a header so what can I do? :-)
Applied, thanks Eric.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists