> @@ -120,8 +121,9 @@ pf_queue_io( > * the lock holder is either reading it from disk himself or > * completely overwriting it this behaviour is perfectly fine. > - bp = libxfs_buf_get_map(mp->m_dev, map, nmaps, LIBXFS_GETBUF_TRYLOCK); > - if (!bp) > + error = -libxfs_buf_get_map(mp->m_dev, map, nmaps, LIBXFS_GETBUF_TRYLOCK, This adds an overly long line. Otherwise looks good: Reviewed-by: Christoph Hellwig <hch@xxxxxx>