Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: JwQG1NpOAGuIXilfbCB4QMPvsJR8FUjHd4PEdadEEqD0KeOqYnm4jGhUce4V8kpnlGua4St52O
 ixiKG8sK8hNw==
IronPort-SDR: OkdchkXaujI5vp5VYuLXIaNe4jnHt5rKnVixn9wQnIUiwpZC8uQfMhsw5rjTVRrTSNSkVusLqz
 XYqQZYUJYS5A==
Subject: Re: [PATCH] x86/resctrl: Constify kernfs_ops
To:     Rikard Falkeborn <rikard.falkeborn@gmail.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>
Cc:     Fenghua Yu <fenghua.yu@intel.com>,
        "H. Peter Anvin" <hpa@zytor.com>, x86@kernel.org,
        linux-kernel@vger.kernel.org
References: <20201110230228.801785-1-rikard.falkeborn@gmail.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
Message-ID: <3a32fd8a-2b41-3be5-2742-540ada4a21e5@intel.com>
Date:   Wed, 18 Nov 2020 15:35:29 -0800
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Thunderbird/78.4.3
MIME-Version: 1.0
In-Reply-To: <20201110230228.801785-1-rikard.falkeborn@gmail.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Precedence: bulk

On 11/10/2020 3:02 PM, Rikard Falkeborn wrote:
> The only usage of the kf_ops field in the rftype struct is to pass it as
> argument to __kernfs_create_file(), which accepts a pointer to const.
> Make it a pointer to const. This makes it possible to make
> rdtgroup_kf_single_ops and kf_mondata_ops const, which allows the
> compiler to put them in read-only memory.
> 
> Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com>

Thank you very much Rikard.

Acked-by: Reinette Chatre <reinette.chatre@intel.com>

Reinette