From:   Tom Saeger <tom.saeger@oracle.com>
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc:     Sasha Levin <sashal@kernel.org>,
        Naresh Kamboju <naresh.kamboju@linaro.org>,
        John Paul Adrian Glaubitz <glaubitz@physik.fu-berlin.de>,
        Rich Felker <dalias@libc.org>,
        Masahiro Yamada <masahiroy@kernel.org>,
        Ard Biesheuvel <ardb@kernel.org>,
        Nathan Chancellor <nathan@kernel.org>,
        Dennis Gilmore <dennis@ausil.us>,
        Palmer Dabbelt <palmer@rivosinc.com>,
        Arnd Bergmann <arnd@arndb.de>,
        Andrew Morton <akpm@linux-foundation.org>,
        Tom Saeger <tom.saeger@oracle.com>, stable@vger.kernel.org,
        linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org,
        linux-riscv@lists.infradead.org, linuxppc-dev@lists.ozlabs.org,
        linux-s390@vger.kernel.org, linux-sh@vger.kernel.org
Subject: [PATCH 5.4 v2 2/6] arch: fix broken BuildID for arm64 and riscv
Date:   Fri, 10 Feb 2023 13:20:23 -0700
Message-Id: <20230210-tsaeger-upstream-linux-stable-5-4-v2-2-a56d1e0f5e98@oracle.com>
In-Reply-To: <20230210-tsaeger-upstream-linux-stable-5-4-v2-0-a56d1e0f5e98@oracle.com>
References: <20230210-tsaeger-upstream-linux-stable-5-4-v2-0-a56d1e0f5e98@oracle.com>
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ZHJTdmdlWmQ3UlZwS215c3ljSE9QUmZwUk9yNGRGbkROYmd0bFAreXBLbWY5?=
 =?utf-8?B?bVljUXN1WjdWOUR1TC9qNGhucEVLLzFpMU5XNHRhem9YL0haZVFEQWV0VGF4?=
 =?utf-8?B?S1BuYmJyVzN5N2tKQ3JDUGxyU2Yyci9JTXVETUNvNU5Kenp0bnRqUXg5NEY1?=
 =?utf-8?B?bEx6WWtVYjdHQkhBU2xSWklWZGtqNjhGbSsrcTZCbTRwL0hDKzNGRFVUSUhP?=
 =?utf-8?B?R0FqaWRsRHpqMU9EOThURFhVQ3JPM0w1TlZXZEI1ZjYycVVmSnllamsycFBE?=
 =?utf-8?B?OER5ZzJ3TmlETmJHTjVHRzJVekFKdmpxbkU3b2szQmtRZWRZRVl4WWpjMWM0?=
 =?utf-8?B?Vk5mdGFOWGk5UHhBYmtwa2tlUDFpc3lYUmJLTW5HWDVYa1lpZDZ4bjZvanhi?=
 =?utf-8?B?VHlYVmhBRlpQZ1pPdlNIYkI3cVliSjAxbkhaRWtSNHRyNmthZXdYVE96N0l2?=
 =?utf-8?B?NjgwQzZFUm14cUV3dXFHS3h4RFc1NExNanM2elVXUk5CSWxOTFNaUFY2eGlj?=
 =?utf-8?B?SnlCaVhXVkxUTmtMQ21rQk81R1hCazZHT3o5T3pkSExVMWtOdXRBaEFxeVpD?=
 =?utf-8?B?ckhxWkk3d3lBTklTMFZqaUoySmZoTG9lMTA1bGpXSThtdUhNWWZMdWlOdUJF?=
 =?utf-8?B?VVBFSFBaMWFqc2lwc3IvVUliTUdoNVZTUTRqWVA4MGJDY1VVcHczaW0wM21Y?=
 =?utf-8?B?VzF3Y2dIcU1uMTEweFh5K0dqVVEvcndiYzR0bHhrWjBxaVJYSkdoeUdtQjdn?=
 =?utf-8?B?WFJHR2F2MXhrN1YzK3lhdXY5dWdPejNvLytidVpyaG9QWGUycmpvK3ArTFZv?=
 =?utf-8?B?ZFNoZXVpQWdqanU2U1JVNGFQWlViNEJzdWF3a25oVkJTL2NUQk12c0crZjBF?=
 =?utf-8?B?TVRxTVp6bFhHWnhCK2RmZnNQSzQxNTBhZ3MzZXk3ekM1SFNaNWNVQmNtV2xq?=
 =?utf-8?B?MjF3UTJsZGMzMXEvTHpyeExQdFJoN2F3ZGpNYlM2QzNnU1hvOUhldWhjL0NR?=
 =?utf-8?B?QU1kNFo3Sk0rSU5FSVVRMFkzZExVbGlEK29RMDhwZHJNNWhQNXFySmM5MGEz?=
 =?utf-8?B?UGR0NXhlcFZvZ25sMU5ucVVJTXJ6WkNLSzBZcnZ1S2V0K2tNcW5zaU1VcFdZ?=
 =?utf-8?B?OGZDY044dEt3UDhWNWE0TEZodjRidDJ2WWxFMUcyZmVFUzhYTlpjS080aFlw?=
 =?utf-8?B?YWlJellQWFBXOUtRTjc2aE1NUWU3MzF4U241dVdxcCtONmV3NWlybS9OOFQ4?=
 =?utf-8?B?M1hDb0x6K1AxdWMyVTI5R1gwdER6Y3RsUzU1VWh4R3lPMXJqUmdwR1FlY05D?=
 =?utf-8?B?bDV2TUduSlZjUHBUVHNOUVNUOUpYdjBKRk1YL2NQVkU4L0hQQVFYME1wUG5n?=
 =?utf-8?B?eC84dTlGOU1WT3ZzNW5Pd1AxMkdKVTdjdHVmeFNmZWc5YWhvMlp3TWJEc3Fl?=
 =?utf-8?B?Sk5obnhSMmZJQThTNEF1d1hlTzY0NnIyVnllS0dJQmRkdjBLOHdqMVB6cVJ2?=
 =?utf-8?B?NjZ6LzFqbVVudUQwanByR2M5Z3M4cUFaM2xLMytxM3UvQ3Q2R3FoWjhNcThW?=
 =?utf-8?B?Vkk3ZGcxM3JJQklqdm5wZTZhL1FhUVBOTDV5eE5DeDRtd0tzUHRGUWxRYWEy?=
 =?utf-8?B?c0llT3EyYi9hMVpWSlMwdWhJb2tLcHM0RlRURjNtK21WNXZsbXIvTzc3WTJa?=
 =?utf-8?B?NzR3YkN5WldtMHBySXZ4L0tHUDl5cC8xMWQyK0xGRzFaVWNXNGpIUnJRdHdS?=
 =?utf-8?B?NmhkL0psanNRZUVnWHdoSGxpV3FkaWNCUUNqeDB4empYL0tQU0NSREc2WEli?=
 =?utf-8?B?YUhUSlJqN3A0SytTZGNRTlhQYnY5Y0pnSnF1cGF2WGlIa2d6VzZFejlRZGoz?=
 =?utf-8?B?emtXUzF4VVFWMHJMTk83dS91WDhhL2h2YmYzZGxKQWlRWmwybm84bHVRRmdQ?=
 =?utf-8?B?aGJ4NzJwVmdkd3dDL0w1dk5yMWtMaytCMTBqbk5jOWt4L3lGbkpyYzZpQS9F?=
 =?utf-8?B?K0QwT3dtallUZW5LVXJ6YW90RUFtU3FtV2p5UFo1QW1tQXU3VUd1WkU2c1pI?=
 =?utf-8?B?TlVPb2dzREh6YVprWTZuNGRGTXlVeVNPSVFzd3JHcXBlaFZaT3RWenprS0xL?=
 =?utf-8?B?RTE0bEZMWk96TWthdEsvN0lMV21CYzZiWDd4ZWRSZ3dlR1VEYlJMVUgrNGxK?=
 =?utf-8?Q?kO9PEPzaY72FdopwNGEYbZI=3D?=
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-ExternalHop-MessageData-0: =?utf-8?B?b3gwaWFST2xyTXU4UlMzbzN5aFJScXIzQTFiTkhPNlBIeENyWnUrQml3dzRv?=
 =?utf-8?B?TzdBcS91SmVBeDExRlZ0SzM2WGUyb29aa3ZWRmc3T1JzSzQ4WmFYZXMyZlE1?=
 =?utf-8?B?cUpwWEhTb2FJd05ZeEFoMUhkVitwbFVqL29Wa3JvU0RjK29ML3p4c0dQUFg0?=
 =?utf-8?B?blZTVXY2aEk2MmN6WHJtS3krdis3R3ZqT0VLTHdvQkRUQ2tHMHVta21MQzAw?=
 =?utf-8?B?L3gxL1FMSFhvS1lYcVByNmJLL2FGdkFjc0dqZDBlRFBJOERuZ0ltaEtVVnpo?=
 =?utf-8?B?eFIvcHB4OVpxd0JhQWVwNEVTUDRjMElQKzdjVU9ITmI4Y2t2UWtub1dPd2hZ?=
 =?utf-8?B?K2RXU0Rhc1pEbDM5THYrUnZnUDk4eTErUHdFS1N4Z1JESElwUEZhZndhbzJG?=
 =?utf-8?B?cWN3M1Zqam9sTnJXWGkrS20xUVd6c3NkK3lIeTExMTdLaldFMVdYQW9jZ1cv?=
 =?utf-8?B?ZXYyQWVYNWZVSjVydk1TeDAySkw1S3dGeUd6MUtOUFF2U0tQRWl1VUpqa3Zm?=
 =?utf-8?B?dTRLUzBsdnJWb0hPbTY5dk9Hemc0VlVPT09ubWV4ekxTSE1aY1ExYXQ3a2pD?=
 =?utf-8?B?TkQ4eldDb21TSlg0ankrc3kxYUo4L0s0VEFlQTJCcTdVTXBuU0k4YU1SWmFx?=
 =?utf-8?B?UWRZY1dPSkQ2WFA1Zkl1QWdFRXJjdlpHLzh4UlpSS1VJWHZ4Njg5dFROSlFS?=
 =?utf-8?B?VC9XRUFGb0R6ZTdidllFR0FkR3p6YjJTVnJqalYrclRlZzdwcTFRbWlJemxI?=
 =?utf-8?B?M0ZzMEYwbm5WOVloV0ZtQlh6UUY5WUpkYnJ1L2ZlMXc4UkJla1I1T3pRbndu?=
 =?utf-8?B?dmRSdFNlMUhKR1ZpUjkzTnI0Yk9qQjE3bXVaQ3dQM1gzaU1LTThycXdOamM0?=
 =?utf-8?B?cW5hbUMwaFNDbGg4eW9vYjBNdDlPZzlzdXRCcG5TZnFPZVBRU0xJQzlod3pD?=
 =?utf-8?B?Z0tuMnYrK1luZHJPby9BSnpVTmVETFZlSjhnN21WSjgweW1QajIvV2ZLT08v?=
 =?utf-8?B?RWprcDdQQ0t0UWxESXdKM2FpeGZXMG0xM2d1WGtQRHlRSjdqZG9WeldVRFFN?=
 =?utf-8?B?dTAyUWZsMGVwV2xtNXp1UDNVendRKzV6d283bTMzdlRUOWV4c00zVjR6TUJs?=
 =?utf-8?B?UWhxZnVkMWE4TGdFR0F6N0hzeS9XVTVkLzF3TjBaeXpGTUZ6T1VObDZxY1pF?=
 =?utf-8?B?WWppS0pNS3ZyMjhvR1lReTVVTXBKY2tncEJjdVNDeFNzU3FaKzVuL0tCSjZ5?=
 =?utf-8?B?aXlucnRiRE8zZFM1Zk5IKzZMcjIrUStZbE04blV4Mm5rQ1gzWDc4UEFKTkdK?=
 =?utf-8?B?bGQ3cXlLeW9YRnhjU0M1YXJCK1dvWksvN3phT1l6V2pSSkxaRE8zQWx4NUxH?=
 =?utf-8?B?UWlGejJPbG05K09LLy9GUEYrbjByRmhhOVA0YmRwbkdpS1J4cGpOZk1jQ1lX?=
 =?utf-8?B?azM1UXM2UEt5R1dQdk9aSFNVcmtwUFJkYXp1QmlKY2RIN3VLWVgyZzJ4ZHRY?=
 =?utf-8?B?cnZEaGo2RlZhY2xsTTUxVW5Tb29hL2w4Y05KYzNnTUJ6NWVnTlF1THBzY3c4?=
 =?utf-8?Q?V2CV+Fu8QM3eJN/YAfKOgQXv9w91bNnjtBGgiVLbbarJWi?=
X-OriginatorOrg: oracle.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 52d07fb7-ddc2-4805-0cb6-08db0ba44745
X-MS-Exchange-CrossTenant-AuthSource: BY5PR10MB3794.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 10 Feb 2023 20:20:40.3337
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 4e2c6054-71cb-48f1-bd6c-3a9705aca71b
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: UAjtaVZI8Za08TwmF6Ht0kSGyZQWL714modtBainm9TwXKIDyxxUugDIT5cPfKNpixW3jcr5b57K1SX0cRISMw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO6PR10MB5652
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.219,Aquarius:18.0.930,Hydra:6.0.562,FMLib:17.11.170.22
 definitions=2023-02-10_15,2023-02-09_03,2023-02-09_01
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 bulkscore=0 suspectscore=0 adultscore=0
 malwarescore=0 mlxscore=0 phishscore=0 spamscore=0 mlxlogscore=999
 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2212070000
 definitions=main-2302100173
X-Proofpoint-ORIG-GUID: EyXU24meCM88HAVzdZNV_09UdtD1bkia
X-Proofpoint-GUID: EyXU24meCM88HAVzdZNV_09UdtD1bkia
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

From: Masahiro Yamada <masahiroy@kernel.org>

commit 99cb0d917ffa1ab628bb67364ca9b162c07699b1 upstream.

Dennis Gilmore reports that the BuildID is missing in the arm64 vmlinux
since commit 994b7ac1697b ("arm64: remove special treatment for the
link order of head.o").

The issue is that the type of .notes section, which contains the BuildID,
changed from NOTES to PROGBITS.

Ard Biesheuvel figured out that whichever object gets linked first gets
to decide the type of a section. The PROGBITS type is the result of the
compiler emitting .note.GNU-stack as PROGBITS rather than NOTE.

While Ard provided a fix for arm64, I want to fix this globally because
the same issue is happening on riscv since commit 2348e6bf4421 ("riscv:
remove special treatment for the link order of head.o"). This problem
will happen in general for other architectures if they start to drop
unneeded entries from scripts/head-object-list.txt.

Discard .note.GNU-stack in include/asm-generic/vmlinux.lds.h.

Link: https://lore.kernel.org/lkml/CAABkxwuQoz1CTbyb57n0ZX65eSYiTonFCU8-LCQc=74D=xE=rA@mail.gmail.com/
Fixes: 994b7ac1697b ("arm64: remove special treatment for the link order of head.o")
Fixes: 2348e6bf4421 ("riscv: remove special treatment for the link order of head.o")
Reported-by: Dennis Gilmore <dennis@ausil.us>
Suggested-by: Ard Biesheuvel <ardb@kernel.org>
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
Acked-by: Palmer Dabbelt <palmer@rivosinc.com>
Signed-off-by: Tom Saeger <tom.saeger@oracle.com>
---
 include/asm-generic/vmlinux.lds.h | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
index 2d45d98773e2..a68535f36d13 100644
--- a/include/asm-generic/vmlinux.lds.h
+++ b/include/asm-generic/vmlinux.lds.h
@@ -825,7 +825,12 @@
 #define TRACEDATA
 #endif
 
+/*
+ * Discard .note.GNU-stack, which is emitted as PROGBITS by the compiler.
+ * Otherwise, the type of .notes section would become PROGBITS instead of NOTES.
+ */
 #define NOTES								\
+	/DISCARD/ : { *(.note.GNU-stack) }				\
 	.notes : AT(ADDR(.notes) - LOAD_OFFSET) {			\
 		__start_notes = .;					\
 		KEEP(*(.note.*))					\

-- 
2.39.1