From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751450AbdASAGQ (ORCPT ); Wed, 18 Jan 2017 19:06:16 -0500 Received: from pandora.armlinux.org.uk ([78.32.30.218]:50700 "EHLO pandora.armlinux.org.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751249AbdASAGP (ORCPT ); Wed, 18 Jan 2017 19:06:15 -0500 Date: Wed, 18 Jan 2017 23:45:28 +0000 From: Russell King - ARM Linux To: Laura Abbott Cc: kernel-hardening@lists.openwall.com, mark.rutland@arm.com, f.fainelli@gmail.com, keescook@chromium.org, pawel.moll@arm.com, ard.biesheuvel@linaro.org, will.deacon@arm.com, kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org, Jinbum Park , vladimir.murzin@arm.com, linux-arm-kernel@lists.infradead.org, andy.gross@linaro.org, arjan@linux.intel.com, mingo@kernel.org, paul.gortmaker@windriver.com, jonathan.austin@arm.com Subject: Re: [kernel-hardening] Re: [PATCH] ARM: mm: add testcases for RODATA Message-ID: <20170118234527.GW27312@n2100.armlinux.org.uk> References: <20170118135310.GA4733@pjb1027-Latitude-E5410> <20170118223629.GQ27312@n2100.armlinux.org.uk> <46bae2bb-4b4f-158b-6a8d-3737e181ad74@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <46bae2bb-4b4f-158b-6a8d-3737e181ad74@redhat.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jan 18, 2017 at 03:38:52PM -0800, Laura Abbott wrote: > On 01/18/2017 02:36 PM, Russell King - ARM Linux wrote: > > I don't see why this needs to be in cacheflush.h - it doesn't seem to > > have anything to do with cache flushing, and placing it in here means > > that if you change the state of CONFIG_DEBUG_RODATA_TEST, most likely > > the entire kernel gets rebuilt. Please put it in a separate header > > file. > > cacheflush.h seems to be where all the set_memory_* functions have > ended up. I was just looking at cleaning that up unless someone > beats me to it. +1 -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net.