| Message ID | 20170619105536.GA24075@elgon.mountain (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Mon, Jun 19, 2017 at 01:55:37PM +0300, Dan Carpenter wrote: > This function is supposed to return blk_status_t error codes now but > there was a stray -ENOMEM left behind. > > Fixes: 4e4cbee93d56 ("block: switch bios to blk_status_t") > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Looks fine, Acked-by: Christoph Hellwig <hch@lst.de> -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Jun 19, 2017 at 01:55:37PM +0300, Dan Carpenter wrote: > This function is supposed to return blk_status_t error codes now but > there was a stray -ENOMEM left behind. > > Fixes: 4e4cbee93d56 ("block: switch bios to blk_status_t") > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Acked-by: David Sterba <dsterba@suse.com> The patch depends on the blk_status_t patchset, so I expect that it's going to be merged to that. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 06/21/2017 07:17 AM, David Sterba wrote: > On Mon, Jun 19, 2017 at 01:55:37PM +0300, Dan Carpenter wrote: >> This function is supposed to return blk_status_t error codes now but >> there was a stray -ENOMEM left behind. >> >> Fixes: 4e4cbee93d56 ("block: switch bios to blk_status_t") >> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > > Acked-by: David Sterba <dsterba@suse.com> > > The patch depends on the blk_status_t patchset, so I expect that it's > going to be merged to that. Added, thanks.
diff --git a/fs/btrfs/compression.c b/fs/btrfs/compression.c index e536e98fe351..2c0b7b57fcd5 100644 --- a/fs/btrfs/compression.c +++ b/fs/btrfs/compression.c @@ -584,7 +584,7 @@ blk_status_t btrfs_submit_compressed_read(struct inode *inode, struct bio *bio, __GFP_HIGHMEM); if (!cb->compressed_pages[pg_index]) { faili = pg_index - 1; - ret = -ENOMEM; + ret = BLK_STS_RESOURCE; goto fail2; } }
This function is supposed to return blk_status_t error codes now but there was a stray -ENOMEM left behind. Fixes: 4e4cbee93d56 ("block: switch bios to blk_status_t") Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html