Re: [PATCH 6/9] irda: use sock slab cache

[Arnaldo Carvalho de Melo <acme@conectiva.com.br>]

Christoph Hellwig escreveu:
> On Thu, Jan 20, 2005 at 01:47:10AM -0200, Arnaldo Carvalho de Melo wrote:
> 
>>>	I'm just curious about the overhead of adding a specific slab
>>>for IrDA sockets. Most users never create any (using IrCOMM), or
>>>maximum one (using Obex), so it's not like it will get a lot of use
>>>(except here, of course).
>>
>>Well, lets start with something that may sound funny: when this series
>>of patches is finished the overhead will _decrease_ for most people.
>>
>>Why? Today we have in most machines five slab caches of this nature:
>>udp_sock, raw_sock, tcp_sock, unix_sock (PF_LOCAL) and the generic,
>>sock, that only is used by the protocols that are using 
>>kmalloc(pritave_sock) +
>>sk_protinfo.
> 
> 
> But as Jean sais this type of socket is used very little, as are a few
> other probably (raw, pfkey?), so maybe those should just use kmalloc +
> kfree instead of their own slab?

OK, rethinking the scheme:

Now: rename the zero_it sk_alloc parameter to obj_size, kmalloc the whole
aggregate sock in sk_alloc (e.g. struct irda_sock in the patch that originated
this thread) and set sk->sk_slab to NULL, not using sk_cachep (the generic
"sock" slab), at sk_free time we use this logic:

	if (sk->sk_slab == NULL)
		kfree(sk);
	else
		kmem_cache_free(sk->sk_slab, sk);

This works well for the transitional case, i.e. for the protocols that
still use the "sock" generic slab cache, for the ones that already
use private slab caches (tcp, udp, etc) and for the ones that will
be using this new scheme, with just kmalloc/kfree aggregate, derived
from struct sock, i.e. we can even leave the parameter as "zero_it"
as is, as it will be removed in the future, final scheme, described
in the next paragraph.

Future: rename sk->sk_prot->slab_obj_size to sk->sk_prot->obj_size,
leave sk->sk_prot->slab as NULL, this will make sk_alloc use kmalloc to
allocate sk->sk_prot->obj_size bytes, and sk_free, will just use kfree
when sk->sk_prot->slab is NULL.

What do you guys think? Sane? :-)

David, please don't apply this series of patches, I'll rework it
with this new scheme if nobody pokes any hole in it and resent.

Best Regards,

- Arnaldo