From patchwork Mon Mar 4 19:36:47 2019 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arnd Bergmann X-Patchwork-Id: 159568 Delivered-To: patch@linaro.org Received: by 2002:a02:5cc1:0:0:0:0:0 with SMTP id w62csp4136898jad; Mon, 4 Mar 2019 11:37:38 -0800 (PST) X-Google-Smtp-Source: APXvYqyhby+xIHho9rE73gZdg6lP9uqA5UOmwWFCVjl+qsHZhQXJEm2Gu0GdpeqgezSD8ZuxusUO X-Received: by 2002:a17:902:48c8:: with SMTP id u8mr22287221plh.87.1551728258608; Mon, 04 Mar 2019 11:37:38 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1551728258; cv=none; d=google.com; s=arc-20160816; b=YoauqGtxIqVcY2cizVShC6Uea3nI08nlclwD81zFLKUQYz63vXpGYy/CI7iP//c0GX Vv78Y5jNIaIPqNKCDodXf3tmj/DfuIiZkrsTPVR6HUgqzn6/W/JePuWOG0BeIH2H7sxz lW5tr2EsTIMtCpJaQa1W4NQJkcPSCDAAP7oc6yKm5B1u0pdd3XTnl8gZYBBwH3rJb9xc nY0+oWEssluqvhH5639B2tESs6nVBPVVZIYtMLl7a8a8WDmrfodorTvrqddiG7QOzBqB fH6Xz13WPi5Mpn6Q9KN68XSqLMwRnrFTh2i4/jFHMiT6xyUYy3NXR+pAl/8PStheFiqo cShA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :message-id:date:subject:cc:to:from; bh=EAbW/NlZEnMX6cChUz+If3MaXvrQj1WePtQ5Ud8Hvr8=; b=GtTimrfrvNz5uME6+5o8I0Ej8A5CHZTfpCKh1R2IdYR7GMsTU7m5MgUic6LvSvRlHR /4gOFaXqFB6jFTpTHcgLg/7Un1v+ce/HU+p6ozD0z0nFMo5lrb4L2d0snc4fkW4y1XM5 /DoURqP4FPWYTJ5OgkO+VyqZw9x6FppPtuC2YR4PnXy+jPufKZu178bmPchcz+SCbYVh vAWBjtLYinlndeMra021LUiAqZGdgLFoBmU8xGPkoskbkXd7vFXsuTa3ZUk6Ym72qN6/ tzFCCsbwKx7MSdKq6aCxjJRJVM/lHVH0K9Vf8bX/lejGa5gH+6DDgtkDnvB4Q4NhBhLF kmeQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id v11si5543920pgs.172.2019.03.04.11.37.38; Mon, 04 Mar 2019 11:37:38 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726243AbfCDThh (ORCPT + 31 others); Mon, 4 Mar 2019 14:37:37 -0500 Received: from mout.kundenserver.de ([212.227.126.130]:41843 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726038AbfCDThg (ORCPT ); Mon, 4 Mar 2019 14:37:36 -0500 Received: from wuerfel.lan ([109.192.41.194]) by mrelayeu.kundenserver.de (mreue011 [212.227.15.129]) with ESMTPA (Nemesis) id 1MJVY8-1gh14P47YG-00JvBc; Mon, 04 Mar 2019 20:37:26 +0100 From: Arnd Bergmann To: "Darrick J. Wong" , linux-xfs@vger.kernel.org Cc: Christoph Hellwig , Arnd Bergmann , Dave Chinner , Brian Foster , linux-kernel@vger.kernel.org Subject: [PATCH] xfs: mark xfs_dir2_sf_entry_t as __packed again Date: Mon, 4 Mar 2019 20:36:47 +0100 Message-Id: <20190304193723.657089-1-arnd@arndb.de> X-Mailer: git-send-email 2.20.0 MIME-Version: 1.0 X-Provags-ID: V03:K1:wXjoFVzi3CaIX/kd0PSUSW/UMteqMiuiM0hOYrJAchPiuY8ucN2 szrEsz+19lnKPNjoEl9LQgTRfObe0rGn1aFFWfSRR8mbvdZitzBOrcL9+GVS2mRNAIKbHh0 oOH3xgrcveCiwx/TKDOhhqdd2dJa5YhL5859Hs7UxyrkYQFMV8IXNE+vecaYUqEOKANZBcM Qe0WFJix/pAqM3o4ncuug== X-Spam-Flag: NO X-UI-Out-Filterresults: notjunk:1; V03:K0:6x01TYg4aRM=:STmeLI/8ogrJQE2/kLUYp+ 1JeloQY3OSBYYrSeAKDpOwicDLnQO0PVirGa8XKz8oOFpj5VCNuVeOhoYkGWV01uYdHtJaIgL aJ3+DebPhMlQlinKQxs1qmc5AYjCjZFmMCM/KZIdKwSIc6JOyPjkkJWW3Ih5LFLPuea/WAkLt n5iofEPRkvwvi7f7WFcW3/UeB0sniRCook2AEDPpzzdWfx9sqXAmeQvpjSQ0T3wHVuy5WFdVN sPEw4rh6YVBuhnNyz/7Vjs8YZRt37IPG6yDLERxpNG6BjzUWJ2U0M58tTt1mgVA73VrScrfjZ jKxyHkkPd1eIJ9v4TvAyApyMEWwvHSIUg6ZSd4p9Enrc9ngvLI7PINw+9/xoIUsYibWI92RsY 4n9Wd5Ncl4wMG9OAmIjB/3QDJZzr875CEygyIUzQ2wkyav9aaSr0+djLc72qQHoy5H4qf1bEw 10TpsGRcrLwoNHeAMdWdAaHul6rxoam8KfOgXZ3GI9CWf9GgQE+1j0XSokkKPOUKgP/QDTLXq 2kPTk1UkrSLMdzxKDKfOdAKFSasGJuxEfdUc6FCMkddDY8Xd2ArN+QgRZuF68dCTFNP/H7dlR 8PF+afh0p+/S7ZQ2l0CPYO/jARMh17a1G2J2NPg+p2vVgfjgOoYQkaOQ+WPAk7su9iOuwq4PZ hR701LC4Mf17Ht8P63DMnDqFYrFZmN8Mfgt8ghn2RuJQzbTztEcfmkDDuPaG4D+KYpalu3EAn K9Fgtzm/PW4jfLZm844DyaCwjEIEA7U3K+DKLA== Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org For ARM OABI builds, we run into a compile time assertion: inlined from 'init_xfs_fs' at /git/arm-soc/fs/xfs/xfs_super.c:1991:2: fs/xfs/xfs_ondisk.h:119:208: error: call to '__compiletime_assert_119' declared with attribute error: XFS: sizeof(xfs_dir2_sf_entry_t) is wrong, expected 3 While ARM OABI is pretty much dead and fails to build for typical configurations on modern architectures (ARMv6 or higher), and has been declared deprecated in user space since gcc-4.6, the kernel still allows it to used for building the kernel. In commit 8353a649f577 ("xfs: kill xfs_dir2_sf_off_t"), Christoph removed the old __arch_pack annotation that made it possible to build xfs with oddball ABIs. However, OABI not only requrires padding around short structure but still adds padding after this change. There is no harm to unconditionally mark the structure as __packed now, and that will do the right thing here. As of commit aa2dd0ad4d6d ("xfs: remove __arch_pack"), we need to use __packed here as well, instead of the old __arch_pack. Fixes: 8353a649f577 ("xfs: kill xfs_dir2_sf_off_t") Signed-off-by: Arnd Bergmann --- fs/xfs/libxfs/xfs_da_format.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.20.0 diff --git a/fs/xfs/libxfs/xfs_da_format.h b/fs/xfs/libxfs/xfs_da_format.h index ae654e06b2fb..fa8f07451d39 100644 --- a/fs/xfs/libxfs/xfs_da_format.h +++ b/fs/xfs/libxfs/xfs_da_format.h @@ -230,7 +230,7 @@ typedef struct xfs_dir2_sf_entry { * A 64-bit or 32-bit inode number follows here, at a variable offset * after the name. */ -} xfs_dir2_sf_entry_t; +} __packed xfs_dir2_sf_entry_t; static inline int xfs_dir2_sf_hdr_size(int i8count) {