lkml.org 
[lkml]   [2014]   [Nov]   [26]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v2 net-next 3/3] tuntap: reduce the size of tun_struct by using flex array.

>
> From: Pankaj Gupta <pagupta@redhat.com>
> Date: Tue, 25 Nov 2014 00:04:04 +0530
>
> > This patch switches to flex array to implement the flow caches, it brings
> > several advantages:
> >
> > - Reduce the size of the tun_struct structure, which allows us to increase
> > the
> > upper limit of queues in future.
> > - Avoid higher order memory allocation. It will be useful when switching to
> > pure hashing in flow cache which may demand a larger size array in
> > future.
> >
> > After this patch, the size of tun_struct on x86_64 reduced from 8512 to
> > 328
> >
> > Signed-off-by: Jason Wang <jasowang@redhat.com>
> > Signed-off-by: Pankaj Gupta <pagupta@redhat.com>
> > Reviewed-by: David Gibson <dgibson@redhat.com>
>
> I see no reason to use flex arrays for this, you are preallocaing the
> memory so if anything flex array is adding an unnecessary level of
> redirection for every access in return for no real gains.
>
> Just allocate the thing normally using kzalloc() or whatever.

I agree. Will do the changes and submit v3.
> --
> To unsubscribe from this list: send the line "unsubscribe netdev" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>


\
 
 \ /
  Last update: 2014-11-26 09:21    [W:0.044 / U:1.680 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site