On Tue, Dec 03, 2013 at 01:23:01PM +0400, Vladimir Davydov wrote: > Actually, I'm not going to modify the list_lru structure, because I > think it's good as it is. I'd like to substitute it with a new > structure, memcg_list_lru, only in those places where this functionality > (per-memcg scanning) is really needed. This new structure would look > like this: > > struct memcg_list_lru { > struct list_lru global_lru; > struct list_lru **memcg_lrus; > struct list_head list; > void *old_lrus; > } > > Since old_lrus and memcg_lrus can be NULL under normal operation, in > memcg_list_lru_destroy() I'd have to check either the list or the > global_lru field, i.e. it would look like: > > if (!list.next) > /* has not been initialized */ > return; > > or ... or just use hlist_head. -- To unsubscribe, send a message with 'unsubscribe linux-mm' in the body to majordomo@xxxxxxxxx. For more info on Linux MM, see: http://www.linux-mm.org/ . Don't email: <a href=mailto:"dont@xxxxxxxxx"> email@xxxxxxxxx </a>