On Thu, 7 Jan 2016 09:54:24 -0600 (CST) Christoph Lameter <cl@xxxxxxxxx> wrote: > On Thu, 7 Jan 2016, Jesper Dangaard Brouer wrote: > > > + /* Support for memcg, compiler can optimize this out */ > > + *s = cache_from_obj(*s, object); > > + > > Well the indirection on *s presumably cannot be optimized out. And the > indirection is not needed when cgroups are not active. The indirection is optimized out, because build_detached_freelist is inlined (and I marked it so for readability (even-though it was getting inlined before by GCC)). -- Best regards, Jesper Dangaard Brouer MSc.CS, Principal Kernel Engineer at Red Hat Author of http://www.iptv-analyzer.org LinkedIn: http://www.linkedin.com/in/brouer -- 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>