On Wed, Apr 29, 2020 at 01:21:49PM -0400, Brian Foster wrote: > /** > - * Remove a log items from the AIL > + * Remove a log item from the AIL. > * > + * For each log item to be removed, unlink it from the AIL, clear the IN_AIL This only works on one item, so the "For each" seems wrong. That being said I think we can just remove the comment entirely - the 'remove a log item from the AIL' is pretty obvious from the name, and the details are described in the helper actually implementing the functionality. The rest looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>
