From patchwork Thu Jan 31 18:09:09 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Luis Chamberlain X-Patchwork-Id: 10791103 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id AEFAA184E for ; Thu, 31 Jan 2019 18:09:33 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id A6C53311AD for ; Thu, 31 Jan 2019 18:09:33 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 9B491311B0; Thu, 31 Jan 2019 18:09:33 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-8.0 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 34429311AD for ; Thu, 31 Jan 2019 18:09:32 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727114AbfAaSJb (ORCPT ); Thu, 31 Jan 2019 13:09:31 -0500 Received: from mail.kernel.org ([198.145.29.99]:43260 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726164AbfAaSJb (ORCPT ); Thu, 31 Jan 2019 13:09:31 -0500 Received: from garbanzo.lan (c-73-71-40-85.hsd1.ca.comcast.net [73.71.40.85]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 016C1218AF; Thu, 31 Jan 2019 18:09:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1548958170; bh=mMI2+m2WsJ+xNR81c7xLhVT2iKLBosew7jBfgOhb/4s=; h=From:To:Cc:Subject:Date:From; b=d85zHm1vqdZzgWr/5mRxmoAjwahRGxvluDD+GQv/D+Z8jpAy23br9QOEj0vNbEGd/ Rx7VQuqr+ZENgKJc6kKEPAkOFHqxpOjw67Gr2ExsV82Dj0u78QvG/h1u39W/V3w1pb pDiC4dZObwIwNlDum533xkNfLOirNxg2ZRw0B3lc= From: Luis Chamberlain To: linux-xfs@vger.kernel.org, gregkh@linuxfoundation.org, Alexander.Levin@microsoft.com Cc: amir73il@gmail.com, hch@infradead.org, Luis Chamberlain Subject: [PATCH 00/10] xfs: stable fixes for v4.19.y Date: Thu, 31 Jan 2019 10:09:09 -0800 Message-Id: <20190131180919.2500-1-mcgrof@kernel.org> X-Mailer: git-send-email 2.18.0 Sender: linux-xfs-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-xfs@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP I've volunteered to help with stable XFS maintenance at LSFMM. This is the initial set of work as a result of this. For stable v4.20.y I have not identified any candidate fixes. I'll keep monitoring development for potential fixes. For v4.19.y I have identified 10 candidate fixes I'd like to propose and if there are no issues found with them by folks, I'd like to ask for these to be merged on the next v4.19.y stable push. They are available in git form on my linux-stable linux-xfs-4.19.y branch [0] and consist of the patches posted in this series. As part of this effort I've established and will be tracking a baseline for fstests for XFS for v4.20.y and v4.19.y on oscheck [1], an OS wrapper for fstests check, so long as they are maintained as longterm kernels by the stable kernel team. Since an issue creeped up with 64k blocks recently [2] I've decided to extend my test coverage to also include 64k block sizes on XFS. For this I'm using a powerpc ppc64le system provided by Brazil's OpenPOWER@UNICAMP [3], a university effort. This is refleced by a new section to test on oscheck "xfs_bigblock". One patch in this series fixes that issue on v4.19.y. I'm aware of Brian Foster's patch "xfs: eof trim writeback mapping as soon as it is cached" which I do confirm fixes fstests generic/464, as soon as that gets merged on Linus' tree we can push it through. Perhaps the most questionable fixes are two fixes extracted from a series of 7 by David Chinner [4]. I'm farily certain these two patches are independent, however further feedback / confirmation from others is greatly appreciated: xfs: fix transient reference count error in xfs_buf_resubmit_failed_buffers xfs: delalloc -> unwritten COW fork allocation can go wrong As promised at LSFMM, to help with even further being careful about stable XFS patches I would have at least another pair of eyeballs review this new stable effort, as such I volunteered Amir Goldstein's eyeballs for this series and he also concurs that the set of fixes are sensible and independent -- they don't have any missing dependencies. Worth noting is that since VFS iomap fixes are trickling through the XFS tree, I asked Christoph to take a look at those independently, and he's willing to do so. If we want to eventually bundle those together with XFS stable fixes we can, its just they will require further review from Christoph. Reviews, questions, or rants are greatly appreciated. [0] https://git.kernel.org/pub/scm/linux/kernel/git/mcgrof/linux-stable.git/log/?h=linux-xfs-4.19.y [1] https://gitlab.com/mcgrof/oscheck.git [2] https://bugzilla.kernel.org/show_bug.cgi?id=201577 [3] http://openpower.ic.unicamp.br/ [4] https://patchwork.kernel.org/cover/10689445/ Brian Foster (1): xfs: fix shared extent data corruption due to missing cow reservation Carlos Maiolino (1): xfs: Fix xqmstats offsets in /proc/fs/xfs/xqmstat Christoph Hellwig (1): xfs: cancel COW blocks before swapext Christophe JAILLET (1): xfs: Fix error code in 'xfs_ioc_getbmap()' Darrick J. Wong (1): xfs: fix PAGE_MASK usage in xfs_free_file_space Dave Chinner (3): xfs: fix overflow in xfs_attr3_leaf_verify xfs: fix transient reference count error in xfs_buf_resubmit_failed_buffers xfs: delalloc -> unwritten COW fork allocation can go wrong Eric Sandeen (1): xfs: fix inverted return from xfs_btree_sblock_verify_crc Ye Yin (1): fs/xfs: fix f_ffree value for statfs when project quota is set fs/xfs/libxfs/xfs_attr_leaf.c | 11 +++++++++-- fs/xfs/libxfs/xfs_bmap.c | 5 ++++- fs/xfs/libxfs/xfs_btree.c | 2 +- fs/xfs/xfs_bmap_util.c | 10 ++++++++-- fs/xfs/xfs_buf_item.c | 28 +++++++++++++++++++++------- fs/xfs/xfs_ioctl.c | 2 +- fs/xfs/xfs_qm_bhv.c | 2 +- fs/xfs/xfs_reflink.c | 1 + fs/xfs/xfs_stats.c | 2 +- 9 files changed, 47 insertions(+), 16 deletions(-)