From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from cn.fujitsu.com ([59.151.112.132]:15706 "EHLO heian.cn.fujitsu.com" rhost-flags-OK-FAIL-OK-FAIL) by vger.kernel.org with ESMTP id S1751483AbbIDB3s convert rfc822-to-8bit (ORCPT ); Thu, 3 Sep 2015 21:29:48 -0400 From: Zhao Lei To: "'Omar Sandoval'" , In-Reply-To: Subject: RE: [PATCH v2 0/9] free space B-tree Date: Fri, 4 Sep 2015 09:29:45 +0800 Message-ID: <029b01d0e6b1$2e605750$8b2105f0$@cn.fujitsu.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Sender: linux-btrfs-owner@vger.kernel.org List-ID: References: Hi, Omar Sandoval [PATCH 7/9] have following compiler warning: fs/btrfs/tests/free-space-tree-tests.c: In function '__check_free_space_extents': fs/btrfs/tests/free-space-tree-tests.c:45: warning: 'offset' may be used uninitialized in this function It is just a compiler warning, and will not happened in code logic, but could you fix it to make output pretty? Thanks Zhaolei * From: linux-btrfs-owner@vger.kernel.org > [mailto:linux-btrfs-owner@vger.kernel.org] On Behalf Of Omar Sandoval > Sent: Friday, September 04, 2015 3:44 AM > To: linux-btrfs@vger.kernel.org > Cc: Omar Sandoval > Subject: [PATCH v2 0/9] free space B-tree > > Here's version 2 of the the free space B-tree patches, addressing Josef's review > from the last round, which you can find here: > http://www.spinics.net/lists/linux-btrfs/msg46713.html > > Changes from v1->v2: > > - Cleaned up a bunch of unnecessary instances of "if (ret) goto out; ret = 0" > - Added aborts in the free space tree code closer to the site the error > is encountered: where we add or remove block groups, add or remove > free space, and also when we convert formats > - Moved loading of the free space tree into caching_thread() and added a > new patch 4 in preparation for it > - Commented a bunch of stuff in the extent buffer bitmap operations and > refactored some of the complicated logic > - Added sanity tests for the extent buffer bitmap operations and free > space tree (patches 2 and 6) > - Added Josef's Reviewed-by tags > > Omar Sandoval (9): > Btrfs: add extent buffer bitmap operations > Btrfs: add extent buffer bitmap sanity tests > Btrfs: add helpers for read-only compat bits > Btrfs: refactor caching_thread() > Btrfs: introduce the free space B-tree on-disk format > Btrfs: implement the free space B-tree > Btrfs: add free space tree sanity tests > Btrfs: wire up the free space tree to the extent tree > Btrfs: add free space tree mount option > > fs/btrfs/Makefile | 5 +- > fs/btrfs/ctree.h | 107 ++- > fs/btrfs/disk-io.c | 26 + > fs/btrfs/extent-tree.c | 112 ++- > fs/btrfs/extent_io.c | 183 +++- > fs/btrfs/extent_io.h | 10 +- > fs/btrfs/free-space-tree.c | 1501 > ++++++++++++++++++++++++++++++++ > fs/btrfs/free-space-tree.h | 71 ++ > fs/btrfs/super.c | 24 +- > fs/btrfs/tests/btrfs-tests.c | 52 ++ > fs/btrfs/tests/btrfs-tests.h | 10 + > fs/btrfs/tests/extent-io-tests.c | 138 ++- > fs/btrfs/tests/free-space-tests.c | 35 +- > fs/btrfs/tests/free-space-tree-tests.c | 570 ++++++++++++ > fs/btrfs/tests/qgroup-tests.c | 20 +- > include/trace/events/btrfs.h | 3 +- > 16 files changed, 2763 insertions(+), 104 deletions(-) create mode 100644 > fs/btrfs/free-space-tree.c create mode 100644 fs/btrfs/free-space-tree.h > create mode 100644 fs/btrfs/tests/free-space-tree-tests.c > > -- > 2.5.1 > > -- > 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