Message-ID: <fd1ca57c-1ab7-fa4f-12ba-4e8fd5fef6f9@prevas.dk>
Date:   Tue, 31 Jan 2023 11:58:55 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.2
Subject: Re: [PATCH resend] compiler-intel: Remove duplicate RELOC_HIDE()
Content-Language: en-US, da
To:     Arnd Bergmann <arnd@arndb.de>,
        Geert Uytterhoeven <geert+renesas@glider.be>,
        Masahiro Yamada <masahiroy@kernel.org>,
        Andrew Morton <akpm@linux-foundation.org>,
        "H. Peter Anvin" <hpa@zytor.com>
Cc:     linux-kbuild@vger.kernel.org, linux-kernel@vger.kernel.org
References: <35c0fc4ba93b11c4b8a55c2907f736a840cd12a3.1675161286.git.geert+renesas@glider.be>
 <9d2450ce-6ba2-49eb-9f00-bcb1b8590eeb@app.fastmail.com>
From:   Rasmus Villemoes <rasmus.villemoes@prevas.dk>
In-Reply-To: <9d2450ce-6ba2-49eb-9f00-bcb1b8590eeb@app.fastmail.com>
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?TitnZUUvQlYweW9UYWlQMER6SXNMQ1ZyRGZXU0NWVnE2MWY4U3diNjV3WUZl?=
 =?utf-8?B?MHAzcGJCOEd3T3N0OHYzVmFWWmxrWHhaakloMXBpMnJOZHlBVE5NY2dQdFR2?=
 =?utf-8?B?VG9nTk1KRFBRSHpiMnVqV1pPK0lSaWptMmVndzZpSmFyMU1mb0hlRDVaTzdn?=
 =?utf-8?B?dGlsWERXUWlVVCtyTUx5dHJzcDJidGR6dGlUTUNvbVpHekNyZUtVQ085bTg2?=
 =?utf-8?B?OEFQSXAyRDk0YWFBRnV0MHh6Ykh1YU1HRU1wbTI1NlNIM0JlbkZKL2RlUWhI?=
 =?utf-8?B?TTR6Ymk0a3RvUm9wWFlMTXBoWFA3MGNSb0xNaS9iVUNIa041RzdsYzhSUGhD?=
 =?utf-8?B?VUZWWmZKRnBaT1VDOFN2Qkp5eEo2RTNNbW9RQ2QzcHZUVTFlV0JRT0RlSS9V?=
 =?utf-8?B?NnIyZjJHR1VPeE1TT2lBSDFZSWx0Z0RzWk1tV1p6M05MZjh4Ni9VWGk5M0FM?=
 =?utf-8?B?SSs0OGQvUjgxeXdmYUpaTG8yMCtnTldKQko4a25lUXFLS2hTSUxjOWx5N2NL?=
 =?utf-8?B?N0VSRGIxTTl0dTFscHdaTkxoQjU3bXQ3RE1ZcGRNRE1ZV0lhdUJuMVozdXVF?=
 =?utf-8?B?Z0YzaEQ2Vm9zd3NmSXFEUFRLdWJFSHdYZFRsRHNmTWp6UnJZNmRicXFjU1NT?=
 =?utf-8?B?K3pFemZFUndkTDJZZ0hUM3lCVXFCaWhJR0FFREFxdTFZQ1lLMWE1R2Jnb05q?=
 =?utf-8?B?ZmwvSXV6cWVXSU9adzhIbnAyTUdEOWRhWHhzZktBN09Bd2I2UjA0WTIwWUhZ?=
 =?utf-8?B?TFNKaWZRYUR1Q3Rob2JYMk1BSUYycERPam9CamRyVWM4UzVNQlYvMzNvQmpT?=
 =?utf-8?B?WTVvK3h1Mlh4UmhCV2VtQVZ3OTFKclYwc0RYYkhYNzdpUEQ3UXRxV2wvRWtl?=
 =?utf-8?B?NHBOSG1KcVBnNFk1T1pENnh0QnRydzZrN1JqMXZ3dkRjSmFZdjNVTnYwQnhu?=
 =?utf-8?B?NWJ3eFJpdjdqd1g0WXJzdFd6RzZuU2pSZkc4N2FGWFRmNGpkKzdYMlFWd1Ny?=
 =?utf-8?B?cE1NNUYxenZjbXlicTFEVEt2Vms5T3NDTU41RHBUdE5ac01WUWVSWXgvcHVz?=
 =?utf-8?B?VWQweE9iQmt1SUlFMUhoVzNzcjd4V0E3eHhWMWVqK3Q1NzFsZjRUNGZpTEJz?=
 =?utf-8?B?dUw0ODQyaFJYWC9aZVBycStWSzRJRXg0eU9UbDFtVzhudmp5TDVrdVo5WFFl?=
 =?utf-8?B?KzA4WTlDRHdRWjBBa3l0ZnUvU0xTRWJCUnNKVXlPRDR5eG1ya0pYaCtoREJp?=
 =?utf-8?B?YVk4R0N6K29GM3lXVEdsU3p1WW8wcU5GZHRjZWdINzBVZklMNDFreGhXSTdh?=
 =?utf-8?B?NHhCbllnd09BdjRtWThidW9vWXI5OE5qMnpnMlJSSmJHVXlHK0dxQ092MFdK?=
 =?utf-8?B?TnQ4TlArS1Y5MEpiYk1ScEVKQUlvUTVSak92MjQ5UDk0bFc0TldreTVFMElY?=
 =?utf-8?B?YjdXUU0vQzA2OXljeHhlTjU1Q0xkbXF6eVVRd0cvOG14VDdmbENKUVNDZDBq?=
 =?utf-8?B?dWZLUndNU3B0ZC9ZOVFvZnZRKzcxK0lUWHVQN1l5eVRPVEd1VTZFSldCODQv?=
 =?utf-8?B?elNJRVhYT1FHemJnY3A0UVdIdnQ3ZW05ZmZIU2ppTmVrUmlhQjMyaCs4ZG4z?=
 =?utf-8?B?NHVNRWNEV09veDBYVWEvbGdPY2p0UjVIbmpNZlpaemNQY1dHL25vRTZCQVB3?=
 =?utf-8?B?M3RBT2o0aHhPbEhHQkRMb05YZGttS1dwdDM2R0dwbkRkZFZMNGg3ZXozS3VC?=
 =?utf-8?B?UnBndWM2bTlCSHZEbXpZY3UwclBEZkNJSzh3Yjd3OHZNSWE3c28xdWs0OHI0?=
 =?utf-8?B?L0w1ektuY21WV1JQQUxseVRISjlwNFFNb0ZqNUNGL0dNSDNKTWxnTVl1d0ZV?=
 =?utf-8?B?NVpyWjVwc2ZVS2ZZY1FJS2llUnNKNDlRUWNjYWVIeFo2dWpuanNlaFBIRlhE?=
 =?utf-8?B?MGZqRFVuSGpFRWVEV05BcXc5a1ZQSjIzb1RzRXZTejYwd2RsUHljYjVEYnYx?=
 =?utf-8?B?dnF1L2V3ZWNXSE1IMmx3NEgvd2JKNVJDREpyTEx5amRZTlNGalFnanorWGxB?=
 =?utf-8?B?Mm12djAwdllsbzFDVGdyMm9LUFN3LzYrUFJKMU5RamJNclVmWlRoYTJoTHN6?=
 =?utf-8?B?VS9Ob2pwdk5Xa2RnTmRjbEh3UFVGTE9Vei9IWHJiRGVWY1R5UGFEci92eFlW?=
 =?utf-8?B?TXc9PQ==?=
X-OriginatorOrg: prevas.dk
X-MS-Exchange-CrossTenant-Network-Message-Id: ae40d3ed-b133-4137-1173-08db037a265e
X-MS-Exchange-CrossTenant-AuthSource: DU0PR10MB5266.EURPRD10.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Jan 2023 10:58:56.9503
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: d350cf71-778d-4780-88f5-071a4cb1ed61
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: +hNrRxXYsCR5RTH6xG733rkwQmN5z7rbxQBkTbMwo3Dl6lmfTHst3+7TR3u1zvoLc0DSVGUkA28OSaIr1s1ln34bogNttHJXIhSWUDfeaUk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR10MB3652
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 31/01/2023 11.41, Arnd Bergmann wrote:
> On Tue, Jan 31, 2023, at 11:37, Geert Uytterhoeven wrote:
>> The RELOC_HIDE() implementation for the Intel compiler is identical to
>> the generic version in <linux/compiler.h>, so the former can just be
>> removed.
>>
>> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
>> ---
>> Not even compile-tested.
>> ---
>>  include/linux/compiler-intel.h | 5 -----
>>  1 file changed, 5 deletions(-)
> 
> I thought the entire file was gone already. I don't see anyone
> objecting to the earlier patch
> 
> https://lore.kernel.org/all/20221016182349.49308-1-masahiroy@kernel.org/
> 
> but it never made it in. Maybe we can revive that instead?

Yes, please!