On Thu Aug 19, 2010 at 10:24:00AM +0200, Jerome Marchand <[email protected]> wrote: > On 08/19/2010 02:28 AM, Tyler Hicks wrote: > > On Tue Aug 17, 2010 at 05:24:05PM +0200, Jerome Marchand > > <[email protected]> wrote: > >> > >> Some ecryptfs init functions are not prefixed by __init and thus not > >> freed after initialization. This patch saved about 1kB in ecryptfs > >> module. > >> > >> Signed-off-by: Jerome Marchand <[email protected]> > >> --- > > > > Thanks Jerome - We could also get away with marking > > ecryptfs_init_kmem_caches() and do_sysfs_registration() with __init, > > right? > > > > Tyler > > > > I guess we could. But it doesn't really matter, these are static functions > which > are called from only one place, so they're inlined anyway. I don't know what > the > convention is in such case. >
Good point - this has been applied to git://git.kernel.org/pub/scm/linux/kernel/git/ecryptfs/ecryptfs-2.6.git#next Thanks, Tyler _______________________________________________ Mailing list: https://launchpad.net/~ecryptfs-devel Post to : [email protected] Unsubscribe : https://launchpad.net/~ecryptfs-devel More help : https://help.launchpad.net/ListHelp
