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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id BC8A3E7AD41 for ; Tue, 3 Oct 2023 13:28:55 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236750AbjJCN24 (ORCPT ); Tue, 3 Oct 2023 09:28:56 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60574 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236706AbjJCN2x (ORCPT ); Tue, 3 Oct 2023 09:28:53 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 71003B8; Tue, 3 Oct 2023 06:28:50 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A0DF8C433C9; Tue, 3 Oct 2023 13:28:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1696339730; bh=sehLgD2TutIYB7MxXdgHW4HKeN6+TJNSC72ZiOsgVHA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=r4Zk1cff/6ophzGy0aSfolPXKFijZgIqqv11DAnnz9yibMx+C7wqxLLsAKHFTqpYA MOw1AdubPL6KCRqLlHmqS3nJp4joCxL1/PTuKooki0iD2thdwD5NZdDzOGiZ6nb+JJ 3WxPCZrOSL4OwZ/XIfjngqmuUi9m8VTlc7l+QifRyJVcrXpbmjT+jwQxEdrh9GnwUs T4DWTbb0hueVUCqbNVvrbKt0PeHJIoic0bLeHmaBgrYpyX+SUzWhvmTiAzLdl8yZjO KF/3TEnqRedIPcYZE/IBOyx6MogYfefMuhGmimv9cEqEbJuc7rMmaAcWbIjse7JDQ8 f9BTOil6eOdBw== Date: Tue, 3 Oct 2023 15:28:45 +0200 From: Christian Brauner To: Wedson Almeida Filho Cc: Alexander Viro , linux-fsdevel@vger.kernel.org, linux-kernel@vger.kernel.org, Wedson Almeida Filho Subject: Re: [PATCH 00/29] const xattr tables Message-ID: <20231003-eiszapfen-bruder-d6fba9031133@brauner> References: <20230930050033.41174-1-wedsonaf@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: <20230930050033.41174-1-wedsonaf@gmail.com> Precedence: bulk List-ID: X-Mailing-List: linux-fsdevel@vger.kernel.org On Sat, Sep 30, 2023 at 02:00:04AM -0300, Wedson Almeida Filho wrote: > From: Wedson Almeida Filho > > The 's_xattr' field of 'struct super_block' currently requires a mutable > table of 'struct xattr_handler' entries (although each handler itself is > const). However, no code in vfs actually modifies the tables. > > So this series changes the type of 's_xattr' to allow const tables, and > modifies existing file system to move their tables to .rodata. This is > desirable because these tables contain entries with function pointers in > them; moving them to .rodata makes it considerably less likely to be > modified accidentally or maliciously at runtime. Fine by me and good idea. On vacation this week but I'll wait for individual fs maintainers to ack their portions anyway and then pick up next week.