> -static void pnfs_writehdr_free(struct nfs_pgio_header *hdr) > +static void pnfs_pgio_header_free(struct nfs_pgio_header *hdr) > { > pnfs_put_lseg(hdr->lseg); > nfs_pgio_header_free(hdr); > } I think it should also be mossible to simply call pnfs_put_lseg from nfs_pgio_header_free. pnfs_put_lseg handles a NULL argument fine, is stubbed out for the non-pnfs case, and the other callers should never have it set. -- To unsubscribe from this list: send the line "unsubscribe linux-nfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html