On Wed, Nov 30, 2016 at 08:24:32AM +0800, Qu Wenruo wrote: > > > At 11/30/2016 12:10 AM, David Sterba wrote: > > On Mon, Nov 28, 2016 at 09:40:07AM +0800, Qu Wenruo wrote: > >> Goldwyn Rodrigues has exposed and fixed a bug which underflows btrfs > >> qgroup reserved space, and leads to non-writable fs. > >> > >> This reminds us that we don't have enough underflow check for qgroup > >> reserved space. > >> > >> For underflow case, we should not really underflow the numbers but warn > >> and keeps qgroup still work. > >> > >> So add more check on qgroup reserved space and add WARN_ON() and > >> btrfs_warn() for any underflow case. > >> > >> Signed-off-by: Qu Wenruo <quwen...@cn.fujitsu.com> > >> Reviewed-by: David Sterba <dste...@suse.com> > > > > One of the warnings is visible during xfstests > > (btrfs_qgroup_free_refroot), is there a fix? Either a patch in > > mailinglist or work in progress. If not, I'm a bit reluctant to add it > > to 4.10 as we'd get that reported from users for sure. > > Fix WIP, ETA would be in this week.
Good, thanks. > At least, this warning is working and helped us to find bugs. No doubt about that, I'll keep the patch in 4.10 queue but will not add it to the first pull that I'm about to send today. -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
