On Tue, May 27, 2014 at 05:06:39PM -0500, Seth Jennings wrote: > On Sat, May 24, 2014 at 03:06:06PM -0400, Dan Streetman wrote: <snip> > > + > > +int zpool_evict(void *pool, unsigned long handle) > > +{ > > + struct zpool *zpool; > > + > > + spin_lock(&pools_lock); > > + list_for_each_entry(zpool, &pools_head, list) { > > You can do a container_of() here: > > zpool = container_of(pool, struct zpool, pool); If you do this, all of the pools_head/pools_lock is unneeded as well. Seth -- 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>