Re: [PATCH][RFC 2] cleaning up struct sock

From: Arnaldo Carvalho de Melo (acme@conectiva.com.br)
Date: Tue Dec 18 2001 - 07:01:04 EST


Em Mon, Dec 17, 2001 at 10:51:34PM -0800, David S. Miller escreveu:
> From: Arnaldo Carvalho de Melo <acme@conectiva.com.br>
> Date: Tue, 18 Dec 2001 03:35:52 -0200
>
> the only thing that still has to be done is to remove things
> like daddr, saddr, rcv_saddr, dport, sport and other ipv4 specific members
> of struct sock
>
> Actually, I'd like to keep the first couple cache lines of struct
> sock the way it is :-( For hash lookups the identity + the hash next
> pointer fit perfectly in one cache line on nearly all platforms.

fair
 
> Which brings me to...
>
> Please let me know if this is something acceptable for 2.5.
>
> What kind of before/after effect do you see in lat_tcp/lat_connect
> (from lmbench) runs?

Will see today, I concentrated on the cleanup part trying not to harm
performance by following the suggestions for the first patch (i.e., just one
allocation, etc). I'll test it later today, at the lab, UP and SMP (4 and 8
way) and submit the results here.

Apart from possible performance problems, does the patch looks OK?

- Arnaldo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Sun Dec 23 2001 - 21:00:15 EST