On Mon, Dec 05, 2011 at 03:23:51PM +1100, Dave Chinner wrote: > > + xfs_qm_dqdestroy(dqp); > > return (0); > > } > > While there, you may as well make that a "return 0;" Indeed. > > - mutex_unlock(&xqm->qm_dqfrlist_lock); > > - mutex_destroy(&xqm->qm_dqfrlist_lock); > > kmem_free(xqm); > > } > > Don't we still need that mutex_destroy() call there? We never needed it - Linux does an implicit mutex_destory when freeing memory containing a mutex. _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs
