From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.8 required=3.0 tests=DKIM_INVALID,DKIM_SIGNED, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 305F9C4CEC7 for ; Sun, 15 Sep 2019 18:05:54 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D9EF62053B for ; Sun, 15 Sep 2019 18:05:53 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=zedlx.com header.i=@zedlx.com header.b="6i6J3C+K" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728786AbfIOSFx (ORCPT ); Sun, 15 Sep 2019 14:05:53 -0400 Received: from server53-3.web-hosting.com ([198.54.126.113]:56477 "EHLO server53-3.web-hosting.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726931AbfIOSFw (ORCPT ); Sun, 15 Sep 2019 14:05:52 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=zedlx.com; s=default; h=MIME-Version:Content-Type:In-Reply-To:References:Subject:Cc:To: From:Message-ID:Date:Sender:Reply-To:Content-Transfer-Encoding:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=+oahKDcaVAV2WRpM8C8qonowqebnWNU+gJQhA6c0/rw=; b=6i6J3C+KESL3hhQ1UnzCxSU+qh WXTmXXRTOnl1n897BsrX4hxGpLU/soTyp36DEpJn7Qp8lUDBWhF+RIQEyfimp1Bn868HodF9zGoyc j+oTun9AFq4t0ZFxRkPNjczFvYEjthAAHfhtbUwOyRv8vXDbhTrIe9dh9RJjEiZxjfDLKUYepW+CS ljGla+lz77ApqEFIc2crFIRLQnVeUICVT9ZX2X+vOGeNeBv+AB/U5ol8zOO7v1KFkokcRc+tmDU2U NgiiIW3SkrKSiKudEVrotTUgQjhmwuNryB8u4a3MXgiC8APSXFw9vOXixiYT6jog9f+XaywH+ldlB lTu+dFpw==; Received: from [::1] (port=42772 helo=server53.web-hosting.com) by server53.web-hosting.com with esmtpsa (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128) (Exim 4.92) (envelope-from ) id 1i9YuB-001TlB-G6; Sun, 15 Sep 2019 14:05:52 -0400 Received: from [95.178.242.92] ([95.178.242.92]) by server53.web-hosting.com (Horde Framework) with HTTPS; Sun, 15 Sep 2019 14:05:47 -0400 Date: Sun, 15 Sep 2019 14:05:47 -0400 Message-ID: <20190915140547.Horde.DRsf7IY4-nawgP5QW2UiAFT@server53.web-hosting.com> From: General Zed To: Zygo Blaxell Cc: Chris Murphy , "Austin S. Hemmelgarn" , Btrfs BTRFS Subject: Re: Feature requests: online backup - defrag - change RAID level References: <20190912173440.Horde.WmxNqLlw7nsFNa-Ux9TTgbz@server53.web-hosting.com> <20190912185726.Horde.HMciH9Z16kV4fK10AfUeRA8@server53.web-hosting.com> <20190912235427.GE22121@hungrycats.org> <20190912202604.Horde.2Cvnicewbvpdb39q5eBASP7@server53.web-hosting.com> <20190913031242.GF22121@hungrycats.org> <20190913025832.Horde.Bwn_M-5buBYcgGbqhc_wDkU@server53.web-hosting.com> <20190913052520.Horde.TXpSDI4drVhkIzGxF7ZVMA8@server53.web-hosting.com> <20190914005931.GI22121@hungrycats.org> <20190913212849.Horde.PHJTyaXyvRA0Reaq2YtVdvS@server53.web-hosting.com> <20190914042859.GK22121@hungrycats.org> In-Reply-To: <20190914042859.GK22121@hungrycats.org> User-Agent: Horde Application Framework 5 Content-Type: text/plain; charset=utf-8; format=flowed; DelSp=Yes MIME-Version: 1.0 Content-Disposition: inline X-OutGoing-Spam-Status: No, score=-1.0 X-AntiAbuse: This header was added to track abuse, please include it with any abuse report X-AntiAbuse: Primary Hostname - server53.web-hosting.com X-AntiAbuse: Original Domain - vger.kernel.org X-AntiAbuse: Originator/Caller UID/GID - [47 12] / [47 12] X-AntiAbuse: Sender Address Domain - zedlx.com X-Get-Message-Sender-Via: server53.web-hosting.com: authenticated_id: zedlryqc/from_h X-Authenticated-Sender: server53.web-hosting.com: general-zed@zedlx.com X-Source: X-Source-Args: X-Source-Dir: X-From-Rewrite: unmodified, already matched Sender: linux-btrfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-btrfs@vger.kernel.org Quoting Zygo Blaxell : > On Fri, Sep 13, 2019 at 09:28:49PM -0400, General Zed wrote: >> >> Quoting Zygo Blaxell : >> >> > On Fri, Sep 13, 2019 at 05:25:20AM -0400, General Zed wrote: >> > > >> > > Quoting General Zed : >> > > >> > > > Quoting Zygo Blaxell : >> > > > >> > > > > On Thu, Sep 12, 2019 at 08:26:04PM -0400, General Zed wrote: >> > > > > > >> > > > > > Quoting Zygo Blaxell : >> > > > > > >> > > > > > > On Thu, Sep 12, 2019 at 06:57:26PM -0400, General Zed wrote: >> > > > > > > > >> > > > > > > > At worst, it just has to completely write-out "all >> > > > > > > metadata", all the way up >> > > > > > > > to the super. It needs to be done just once, because >> > > what's the point of >> > > > > > > > writing it 10 times over? Then, the super is updated as >> > > > > > > the final commit. >> > > > > > > >> > > > > > > This is kind of a silly discussion. The biggest extent >> possible on >> > > > > > > btrfs is 128MB, and the incremental gains of forcing 128MB >> > > extents to >> > > > > > > be consecutive are negligible. If you're defragging a 10GB >> > > file, you're >> > > > > > > just going to end up doing 80 separate defrag operations. >> > > > > > >> > > > > > Ok, then the max extent is 128 MB, that's fine. Someone here >> > > > > > previously said >> > > > > > that it is 2 GB, so he has disinformed me (in order to further >> > > his false >> > > > > > argument). >> > > > > >> > > > > If the 128MB limit is removed, you then hit the block group >> size limit, >> > > > > which is some number of GB from 1 to 10 depending on number of disks >> > > > > available and raid profile selection (the striping raid profiles cap >> > > > > block group sizes at 10 disks, and single/raid1 profiles >> always use 1GB >> > > > > block groups regardless of disk count). So 2GB is _also_ a >> valid extent >> > > > > size limit, just not the first limit that is relevant for defrag. >> > > > > >> > > > > A lot of people get confused by 'filefrag -v' output, which >> coalesces >> > > > > physically adjacent but distinct extents. So if you use that tool, >> > > > > it can _seem_ like there is a 2.5GB extent in a file, but >> it is really >> > > > > 20 distinct 128MB extents that start and end at adjacent addresses. >> > > > > You can see the true structure in 'btrfs ins dump-tree' output. >> > > > > >> > > > > That also brings up another reason why 10GB defrags are >> absurd on btrfs: >> > > > > extent addresses are virtual. There's no guarantee that a pair >> > > of extents >> > > > > that meet at a block group boundary are physically >> adjacent, and after >> > > > > operations like RAID array reorganization or free space >> defragmentation, >> > > > > they are typically quite far apart physically. >> > > > > >> > > > > > I didn't ever said that I would force extents larger than 128 MB. >> > > > > > >> > > > > > If you are defragging a 10 GB file, you'll likely have to do it >> > > > > > in 10 steps, >> > > > > > because the defrag is usually allowed to only use a limited >> > > amount of disk >> > > > > > space while in operation. That has nothing to do with the >> extent size. >> > > > > >> > > > > Defrag is literally manipulating the extent size. >> Fragments and extents >> > > > > are the same thing in btrfs. >> > > > > >> > > > > Currently a 10GB defragment will work in 80 steps, but doesn't >> > > necessarily >> > > > > commit metadata updates after each step, so more than 128MB >> of temporary >> > > > > space may be used (especially if your disks are fast and empty, >> > > > > and you start just after the end of the previous commit interval). >> > > > > There are some opportunities to coalsce metadata updates, >> occupying up >> > > > > to a (arbitrary) limit of 512MB of RAM (or when memory >> pressure forces >> > > > > a flush, whichever comes first), but exploiting those opportunities >> > > > > requires more space for uncommitted data. >> > > > > >> > > > > If the filesystem starts to get low on space during a defrag, it can >> > > > > inject commits to force metadata updates to happen more often, which >> > > > > reduces the amount of temporary space needed (we can't delete >> > > the original >> > > > > fragmented extents until their replacement extent is committed); >> > > however, >> > > > > if the filesystem is so low on space that you're worried >> about running >> > > > > out during a defrag, then you probably don't have big >> enough contiguous >> > > > > free areas to relocate data into anyway, i.e. the defrag is just >> > > going to >> > > > > push data from one fragmented location to a different fragmented >> > > location, >> > > > > or bail out with "sorry, can't defrag that." >> > > > >> > > > Nope. >> > > > >> > > > Each defrag "cycle" consists of two parts: >> > > > 1) move-out part >> > > > 2) move-in part >> > > > >> > > > The move-out part select one contiguous area of the disk. Almost any >> > > > area will do, but some smart choices are better. It then moves-out all >> > > > data from that contiguous area into whatever holes there are >> left empty >> > > > on the disk. The biggest problem is actually updating the metadata, >> > > > since the updates are not localized. >> > > > Anyway, this part can even be skipped. >> > > > >> > > > The move-in part now populates the completely free contiguous >> area with >> > > > defragmented data. >> > > > >> > > > In the case that the move-out part needs to be skipped because the >> > > > defrag estimates that the update to metatada will be too big (like in >> > > > the pathological case of a disk with 156 GB of metadata), it can >> > > > sucessfully defrag by performing only the move-in part. In that case, >> > > > the move-in area is not free of data and "defragmented" data won't be >> > > > fully defragmented. Also, there should be at least 20% free disk space >> > > > in this case in order to avoid defrag turning pathological. >> > > > >> > > > But, these are all some pathological cases. They should be >> considered in >> > > > some other discussion. >> > > >> > > I know how to do this pathological case. Figured it out! >> > > >> > > Yeah, always ask General Zed, he knows the best!!! >> > > >> > > The move-in phase is not a problem, because this phase >> generally affects a >> > > low number of files. >> > > >> > > So, let's consider the move-out phase. The main concern here is that the >> > > move-out area may contain so many different files and fragments that the >> > > move-out forces a practically undoable metadata update. >> > > >> > > So, the way to do it is to select files for move-out, one by >> one (or even >> > > more granular, by fragments of files), while keeping track of >> the size of >> > > the necessary metadata update. When the metadata update exceeds >> a certain >> > > amount (let's say 128 MB, an amount that can easily fit into RAM), the >> > > move-out is performed with only currently selected files (file >> fragments). >> > > (The move-out often doesn't affect a whole file since only a >> part of each >> > > file lies within the move-out area). >> > >> > This move-out phase sounds like a reinvention of btrfs balance. Balance >> > already does something similar, and python-btrfs gives you a script to >> > target block groups with high free space fragmentation for balancing. >> > It moves extents (and their references) away from their block group. >> > You get GB-sized (or multi-GB-sized) contiguous free space areas into >> > which you can then allocate big extents. >> >> Perhaps btrfs balance needs to perform something similar, but I can assure >> you that a balance cannot replace the defrag. > > Correct, balance is only half of the solution. > > The balance is required for two things on btrfs: "move-out" phase of > free space defragmentation, and to ensure at least one unallocated block > group exists on the filesystem in case metadata expansion is required. > > A btrfs can operate without defrag for...well, forever, defrag is not > necessary at all. I have dozens of multi-year-old btrfs filesystems of > assorted sizes that have never run defrag even once. > > By contrast, running out of unallocated space is a significant problem > that should be corrected with the same urgency as RAID entering degraded > mode. I generally recommend running 'btrfs balance start -dlimit=1' about > once per day to force one block group to always be empty. > > Filesystems that don't maintain unallocated space can run into problems > if metadata runs out of space. These problems can be inconvenient to > recover from. > >> The point and the purpose of "move out" is to create a clean contiguous free >> space area, so that defragmented files can be written into it. > >> >> > > Now the defrag has to decide: whether to continue with another >> round of the >> > > move-out to get a cleaner move-in area (by repeating the same procedure >> > > above), or should it continue with a move-in into a partialy >> dirty area. I >> > > can't tell you what's better right now, as this can be >> determined only by >> > > experiments. >> > > >> > > Lastly, the move-in phase is performed (can be done whether the >> move-in area >> > > is dirty or completely clean). Again, the same trick can be >> used: files can >> > > be selected one by one until the calculated metadata update >> exceeds 128 MB. >> > > However, it is more likely that the size of move-in area will >> be exhausted >> > > before this happens. >> > > >> > > This algorithm will work even if you have only 3% free disk space left. >> > >> > I was thinking more like "you have less than 1GB free on a 1TB filesystem >> > and you want to defrag 128MB things", i.e. <0.1% free space. If you don't >> > have all the metadata block group free space you need allocated already >> > by that point, you can run out of metadata space and the filesystem goes >> > read-only. Happens quite often to people. They don't like it very much. >> >> The defrag should abort whenever it detects such adverse conditions as 0.1% >> free disk space. In fact, it should probably abort as soon as it detects >> less than 3% free disk space. This is normal and expected. If the user has a >> partition with less than 3% free disk space, he/she should not defrag it >> until he/she frees some space, perhaps by deleting unnecessary data or by >> moving out some data to other partitions. > > 3% of 45TB is 1.35TB...seems a little harsh. Recall no extent can be > larger than 128MB, so we're talking about enough space for ten thousand > of defrag's worst-case output extents. A limit based on absolute numbers > might make more sense, though the only way to really know what the limit is > on any given filesystem is to try to reach it. Nah. The free space minimum limit must, unfortunately, be based on absolute percentages. There is no better way. The problem is that, in order for defrag to work, it has to (partially) consolidate some of the free space, in order to produce a contiguous free area which will be the destination for defrag data. In order to be able to produce this contiguous free space area, it is of utmost importance that there is sufficient free space left on the partition. Otherwise, this free space consolidation operation will take too much time (too much disk I/O). There is no good way around it the common cases of free space fragmentation. If you reduce the free space minimum limit below 3%, you are likely to spend 2x more I/O in consolidating free space than what is needed to actually defrag the data. I mean, the defrag will still work, but I think that the slowdown is unacceptable. I mean, the user should just free some space! The filesystems should not be left with less than 10% free space, that's simply bad management from the user's part, and the user should accept the consequences.