On (04/04/17 14:40), Minchan Kim wrote: > > [..] > > > -struct zram_meta { > > > +struct zram { > > > struct zram_table_entry *table; > > > struct zs_pool *mem_pool; > > > -}; > > > - > > > -struct zram { > > > - struct zram_meta *meta; > > > struct zcomp *comp; > > > struct gendisk *disk; > > > /* Prevent concurrent execution of device init */ > > > > > > we still have several zram_meta_FOO() left overs in zram_drv.c > > I intentionally used that term because I don't think better name > to wrap logis which allocates table and mempool.
ah, it was intentional. um, OK, let's keep zram_meta_foo(). -ss