From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from prv3-mh.provo.novell.com ([137.65.250.26]:55487 "EHLO prv3-mh.provo.novell.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751901AbbLLVeT (ORCPT ); Sat, 12 Dec 2015 16:34:19 -0500 Subject: Re: FAILED: patch "[PATCH] Btrfs: fix regression running delayed references when using" failed to apply to 4.2-stable tree To: Greg KH References: <1449852873196149@kroah.com> <566C2FEA.2060409@suse.com> <20151212195749.GA10064@kroah.com> <20151212200342.GA30934@kroah.com> Cc: derek@valedon.co.uk, floyd.net@gmail.com, flux-btrfs@inside.org, mail@eliasprobst.eu, malte@tnxip.de, quwenruo@cn.fujitsu.com, stephane_btrfs@lesimple.fr, stable@vger.kernel.org, clm@fb.com From: Filipe Manana Message-ID: <566C9331.30703@suse.com> Date: Sat, 12 Dec 2015 21:35:45 +0000 MIME-Version: 1.0 In-Reply-To: <20151212200342.GA30934@kroah.com> Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Sender: stable-owner@vger.kernel.org List-ID: On 12/12/2015 08:03 PM, Greg KH wrote: > On Sat, Dec 12, 2015 at 11:57:49AM -0800, Greg KH wrote: >> On Sat, Dec 12, 2015 at 02:32:10PM +0000, Filipe Manana wrote: >>> >>> >>> On 12/11/2015 04:54 PM, gregkh@linuxfoundation.org wrote: >>>> >>>> The patch below does not apply to the 4.2-stable tree. >>>> If someone wants it applied there, or to any other stable or longterm >>>> tree, then please email the backport, including the original git commit >>>> id to . >>> >>> Greg, please find attached a version of the patch that applies against >>> 4.2.7. >> >> This patch breaks the build: I did build and tested it (in reply to your next comment about sending untested stuff). See comment inlined below. >> >> fs/btrfs/delayed-ref.c: In function ‘merge_ref’: >> fs/btrfs/delayed-ref.c:223:38: error: ‘struct btrfs_delayed_ref_node’ has no mem >> ber named ‘no_quota’ >> if (next->type != ref->type || next->no_quota != ref->no_quota) I don't get why you're getting that error. struct btrfs_delayed_ref_node has a member named 'no_quota': https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/tree/fs/btrfs/delayed-ref.h?id=refs/tags/v4.2.7#n36 https://git.kernel.org/cgit/linux/kernel/git/stable/linux-stable.git/tree/fs/btrfs/delayed-ref.h?id=refs/tags/v4.2.7#n71 Am I missing something? I think you applied 2 patches in the wrong order. You should apply the following two in this order: 1) "Btrfs: fix regression when running delayed references" - https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=2c3cf7d5f6105bb957df125dfce61d4483b8742d 2) "Btrfs: fix regression running delayed references when using qgroups" - https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=b06c4bf5c874a57254b197f53ddf588e7a24a2bf Please confirm if you applied those 2 in that specific order, because it doesn't seem so. The first one adds code that still uses member "no_quota" while the second one removes the "no_quota" member from that struct. Thanks >> ^ >> fs/btrfs/delayed-ref.c:223:55: error: ‘struct btrfs_delayed_ref_node’ has no mem >> ber named ‘no_quota’ >> if (next->type != ref->type || next->no_quota != ref->no_quota) >> ^ >> fs/btrfs/delayed-ref.c: At top level: >> fs/btrfs/delayed-ref.c:313:13: error: redefinition of ‘merge_ref’ >> static bool merge_ref(struct btrfs_trans_handle *trans, >> ^ >> fs/btrfs/delayed-ref.c:200:13: note: previous definition of ‘merge_ref’ was here >> static bool merge_ref(struct btrfs_trans_handle *trans, >> ^ >> fs/btrfs/delayed-ref.c:380:6: error: redefinition of ‘btrfs_merge_delayed_refs’ >> void btrfs_merge_delayed_refs(struct btrfs_trans_handle *trans, >> ^ >> fs/btrfs/delayed-ref.c:267:6: note: previous definition of ‘btrfs_merge_delayed_refs’ was here >> void btrfs_merge_delayed_refs(struct btrfs_trans_handle *trans, >> ^ >> >> So I'm going to have to drop it, sorry. > > It also breaks on 4.3, so I'm dropping that patch as well. It would > have been nice if you had at least test-built what you sent me :( > > thanks, > > greg k-h >