From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from static.68.134.40.188.clients.your-server.de ([188.40.134.68]:55536 "EHLO mail02.iobjects.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752255AbbIPOAq (ORCPT ); Wed, 16 Sep 2015 10:00:46 -0400 Subject: Re: [PATCH V2] Btrfs: keep dropped roots in cache until transaction commit To: Josef Bacik , linux-btrfs@vger.kernel.org, Omar Sandoval References: <1442326024-2536-1-git-send-email-jbacik@fb.com> <55F83E34.3090900@googlemail.com> <55F86CB1.5020503@googlemail.com> <55F86E63.8050300@fb.com> <55F92F20.2040603@googlemail.com> <55F97399.7000407@fb.com> From: =?UTF-8?Q?Holger_Hoffst=c3=a4tte?= Message-ID: <55F9760B.1000401@googlemail.com> Date: Wed, 16 Sep 2015 16:00:43 +0200 MIME-Version: 1.0 In-Reply-To: <55F97399.7000407@fb.com> Content-Type: text/plain; charset=utf-8 Sender: linux-btrfs-owner@vger.kernel.org List-ID: On 09/16/15 15:50, Josef Bacik wrote: > On 09/16/2015 04:58 AM, Holger Hoffstätte wrote: >> On 09/15/15 21:15, Josef Bacik wrote: >>> On 09/15/2015 03:08 PM, Holger Hoffstätte wrote: >>>> On 09/15/15 17:50, Holger Hoffstätte wrote: >>>>> This V2 does indeed seem to fix the issues I reported with snapshot >>>>> deletion & concurrent sync. I've now created/filled/deleted countless >>>>> snapshots while issuing sync(s) in parallel, and the problem that I >>>>> saw fairly frequently with V1 no longer seems to occur here. >>>> >>>> Well..I may have spoken too soon: >>> >>> Huh this doesn't seem related to my stuff. Can I have your script >>> you are running to reproduce this? Thanks, >>> >>> Josef >> >> It seemed to be related to Omar's free-space-tree - I saw the same bug >> a bit later on a different device without snapshot deletion, just >> deleting two large files. Sorry for the false positive. >> > > Omar, do you have time to look into this? Were they highly fragmented > files or just giant contiguous files? Thanks, These were very contiguous (2* ~1.3 GB, 1-3 extents each). I already sent Omar a long klog dump with more background info. The biggest issue is that after this bug the volume is hosed since the fst sets an incompat flag, and after mounting rw just stumbles again trying to replay. I'll keep trying to isolate exactly how and when this happens. -h