Message ID | 20240103135454.376021-1-masahiroy@kernel.org |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel+bounces-15621-ouuuleilei=gmail.com@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:6f82:b0:100:9c79:88ff with SMTP id tb2csp5030978dyb; Wed, 3 Jan 2024 05:55:40 -0800 (PST) X-Google-Smtp-Source: AGHT+IFPar+LIEJt96NFBdfg21PxtHlKG0XfUNg9RJpOXKxUrg7R2f0m9In6tDTW1E3L+Hhajtk7 X-Received: by 2002:a17:907:718d:b0:a23:5f5f:cb94 with SMTP id zv13-20020a170907718d00b00a235f5fcb94mr6856629ejb.47.1704290139848; Wed, 03 Jan 2024 05:55:39 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1704290139; cv=none; d=google.com; s=arc-20160816; b=a05V26fasSfL1PesyC8kIRHjZt8lJgS2indZ0183rZ9v9WOZYQw7tNkZTKYC78T2N5 3esLBq25ybcR5aXMAvdnbOPLkJe/TEnQ8TPE9BEU/oDpWOjA+t+YsQrwxx3UjrKuYf3v Idd+7GAhzgvJkHiOq/F2E/aLJNuh9CTOd1qzn/zbVICU5mt8SP6UIusMOZ+a1SF3HW4i CdCMMGDgItwoKM3BONLPKkgOPgkIcqyS1lO/zKBrt42r+YZvZnVLSFtrD02V/fdTR8rn 2acKuT5bU0EQlXQrZfT1CbgbhHJSxPYvD6kfBq2ortlTv6FmR9C792Yawt/rveFtFVLZ 5U9g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:date:subject:cc:to :from:dkim-signature; bh=0wvvrYsD1eV0jdocFL0PT8MOwBEewbJaGNv4Nu3JQc0=; fh=cr2kAYujDVm2vUVjNQU/xxltWJfGGL3pZvB2q4ufDBk=; b=wJxVgqZLa76sCcPCsCihIuVIjLJsROcO9obL/SuKKus2YLJC/iIlCAV9ftdIvc1Tjq hwYGixdItTFeVMQI2F3VsMlhPEmqd+agi3YhFLMXXeiOMBgXl8uemE0F6oMMIedJMWKS /6xP8C1uL2dRRTNUlwsjVdEpz7XjWSu5Y7XED7nNyKmKLV1IbbyDYohwTMvq6G/NzddH CUWpN3NXE//bYJSkBpQ5+y9UuhcPkxm8T2dU3jMigKPrO31MlpwJElBaTWoEFodohjv3 wBgVpSPEF063e20vumN36A13ZgQP+Rg5tUKFAIqKK18pelZ2DvtR7b2FlG9Ab12dpdC/ GMQQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=SJOWDdtr; spf=pass (google.com: domain of linux-kernel+bounces-15621-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-15621-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from am.mirrors.kernel.org (am.mirrors.kernel.org. [147.75.80.249]) by mx.google.com with ESMTPS id wj23-20020a170907051700b00a27a0bf414bsi3796790ejb.559.2024.01.03.05.55.39 for <ouuuleilei@gmail.com> (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 03 Jan 2024 05:55:39 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-15621-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=SJOWDdtr; spf=pass (google.com: domain of linux-kernel+bounces-15621-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) smtp.mailfrom="linux-kernel+bounces-15621-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by am.mirrors.kernel.org (Postfix) with ESMTPS id 521091F249F5 for <ouuuleilei@gmail.com>; Wed, 3 Jan 2024 13:55:39 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 36C281A5B1; Wed, 3 Jan 2024 13:55:05 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="SJOWDdtr" X-Original-To: linux-kernel@vger.kernel.org Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 96C821A59C; Wed, 3 Jan 2024 13:55:02 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8135BC433C7; Wed, 3 Jan 2024 13:55:01 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1704290102; bh=e+vdgZdNlFkyaCqPGKRk3VegfICWH1EOn+tuydPy1NQ=; h=From:To:Cc:Subject:Date:From; b=SJOWDdtrO8MiO2UrWdF7alkssLRSYC87HIhmoVCdQA2HC6cZ7JwY7uAK4exvO+Cvm 1LFu1V0F63pbEEaHi3nCfcdYLV6zBe6jIj91mabM6lzN0em9qmQN5/a0/wJHFIe6Cb yQ0er1M7R/lUZN4AbdYHrSktGoQnGPqXztNwCSAr5RlYvhMf5+gG/gnXh/7LZ1H27o CPyEBmU0Sajz1SBAgkE1Wbak3su8mqK8j/3BU1u4WNf/ZUstoZrEpZbSKZE6WZec8p VvSwA21E8bz+lLbb95Vu3e922amoXZ9cJzDiGrACXyX7U3waRLZEcjz42N1ocn/tWF 0OFZSKjR6G2fw== From: Masahiro Yamada <masahiroy@kernel.org> To: linux-kbuild@vger.kernel.org Cc: Masahiro Yamada <masahiroy@kernel.org>, linux-kernel@vger.kernel.org Subject: [PATCH] linux/export.h: remove unneeded .balign directive Date: Wed, 3 Jan 2024 22:54:54 +0900 Message-Id: <20240103135454.376021-1-masahiroy@kernel.org> X-Mailer: git-send-email 2.40.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: <linux-kernel.vger.kernel.org> List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org> List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1787077737977761562 X-GMAIL-MSGID: 1787077737977761562 |
Series |
linux/export.h: remove unneeded .balign directive
|
|
Commit Message
Masahiro Yamada
Jan. 3, 2024, 1:54 p.m. UTC
The .export_symbol section is discarded by the linker script, hence
no alignment is needed. Simplify the code.
Signed-off-by: Masahiro Yamada <masahiroy@kernel.org>
---
include/linux/export.h | 10 +++-------
1 file changed, 3 insertions(+), 7 deletions(-)
Comments
Happy new 2024 Masahiro, some small nits. What about adding links to the commits... linux/export: Fix alignment for 64-bit ksymtab entries [ Upstream commit f6847807c22f6944c71c981b630b9fff30801e73 ] linux/export: Ensure natural alignment of kcrctab array [ Upstream commit 753547de0daecbdbd1af3618987ddade325d9aaa ] ^^ AFAICS in linux-stable - v6.6.10-rc1 includes both ^^ Is yours a follow-up and makes sense for linux-stable releases? ...and CC Helge Deller? Best regards, -Sedat- Link: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/log/?h=linux-6.6.y On Wed, Jan 3, 2024 at 2:55 PM Masahiro Yamada <masahiroy@kernel.org> wrote: > > The .export_symbol section is discarded by the linker script, hence > no alignment is needed. Simplify the code. > > Signed-off-by: Masahiro Yamada <masahiroy@kernel.org> > --- > > include/linux/export.h | 10 +++------- > 1 file changed, 3 insertions(+), 7 deletions(-) > > diff --git a/include/linux/export.h b/include/linux/export.h > index 0bbd02fd351d..dff230bb5aca 100644 > --- a/include/linux/export.h > +++ b/include/linux/export.h > @@ -15,13 +15,9 @@ > */ > > #ifdef CONFIG_64BIT > -#define __EXPORT_SYMBOL_REF(sym) \ > - .balign 8 ASM_NL \ > - .quad sym > +#define __EXPORT_SYMBOL_PTR .quad > #else > -#define __EXPORT_SYMBOL_REF(sym) \ > - .balign 4 ASM_NL \ > - .long sym > +#define __EXPORT_SYMBOL_PTR .long > #endif > > #define ___EXPORT_SYMBOL(sym, license, ns) \ > @@ -29,7 +25,7 @@ > __export_symbol_##sym: ASM_NL \ > .asciz license ASM_NL \ > .asciz ns ASM_NL \ > - __EXPORT_SYMBOL_REF(sym) ASM_NL \ > + __EXPORT_SYMBOL_PTR sym ASM_NL \ > .previous > > #if defined(__DISABLE_EXPORTS) > -- > 2.40.1 > >
On Thu, Jan 4, 2024 at 1:05 AM Sedat Dilek <sedat.dilek@gmail.com> wrote: > > Happy new 2024 Masahiro, > > some small nits. > > What about adding links to the commits... > > linux/export: Fix alignment for 64-bit ksymtab entries > [ Upstream commit f6847807c22f6944c71c981b630b9fff30801e73 ] > > linux/export: Ensure natural alignment of kcrctab array > [ Upstream commit 753547de0daecbdbd1af3618987ddade325d9aaa ] > > ^^ AFAICS in linux-stable - v6.6.10-rc1 includes both > ^^ Is yours a follow-up and makes sense for linux-stable releases? No. This patch is a separate clean-up, not a bug fix. No need for back-porting. > > ...and CC Helge Deller? You did it. Thanks. > Best regards, > -Sedat- > > Link: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/log/?h=linux-6.6.y > > On Wed, Jan 3, 2024 at 2:55 PM Masahiro Yamada <masahiroy@kernel.org> wrote: > > > > The .export_symbol section is discarded by the linker script, hence > > no alignment is needed. Simplify the code. > > > > Signed-off-by: Masahiro Yamada <masahiroy@kernel.org> > > --- > > > > include/linux/export.h | 10 +++------- > > 1 file changed, 3 insertions(+), 7 deletions(-) > > > > diff --git a/include/linux/export.h b/include/linux/export.h > > index 0bbd02fd351d..dff230bb5aca 100644 > > --- a/include/linux/export.h > > +++ b/include/linux/export.h > > @@ -15,13 +15,9 @@ > > */ > > > > #ifdef CONFIG_64BIT > > -#define __EXPORT_SYMBOL_REF(sym) \ > > - .balign 8 ASM_NL \ > > - .quad sym > > +#define __EXPORT_SYMBOL_PTR .quad > > #else > > -#define __EXPORT_SYMBOL_REF(sym) \ > > - .balign 4 ASM_NL \ > > - .long sym > > +#define __EXPORT_SYMBOL_PTR .long > > #endif > > > > #define ___EXPORT_SYMBOL(sym, license, ns) \ > > @@ -29,7 +25,7 @@ > > __export_symbol_##sym: ASM_NL \ > > .asciz license ASM_NL \ > > .asciz ns ASM_NL \ > > - __EXPORT_SYMBOL_REF(sym) ASM_NL \ > > + __EXPORT_SYMBOL_PTR sym ASM_NL \ > > .previous > > > > #if defined(__DISABLE_EXPORTS) > > -- > > 2.40.1 > > > > >
On Thu, Jan 4, 2024 at 3:17 PM Masahiro Yamada <masahiroy@kernel.org> wrote: > > On Thu, Jan 4, 2024 at 1:05 AM Sedat Dilek <sedat.dilek@gmail.com> wrote: > > > > Happy new 2024 Masahiro, > > > > some small nits. > > > > What about adding links to the commits... > > > > linux/export: Fix alignment for 64-bit ksymtab entries > > [ Upstream commit f6847807c22f6944c71c981b630b9fff30801e73 ] > > > > linux/export: Ensure natural alignment of kcrctab array > > [ Upstream commit 753547de0daecbdbd1af3618987ddade325d9aaa ] > > > > ^^ AFAICS in linux-stable - v6.6.10-rc1 includes both > > ^^ Is yours a follow-up and makes sense for linux-stable releases? > > > No. > > This patch is a separate clean-up, not a bug fix. > > No need for back-porting. > Thanks for the clarification. But it is an improvement - so no discarding at link-time means reduction of build-time? Amy numbers? > > > > > > > ...and CC Helge Deller? > CC in the patch - makes for me only sense if you agree on adding above commit-ids of Helge's work. I think this is good in sense of follow-able references. Best regards, -Sedat- > > You did it. > > > Thanks. > > > > > > > Best regards, > > -Sedat- > > > > Link: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/log/?h=linux-6.6.y > > > > On Wed, Jan 3, 2024 at 2:55 PM Masahiro Yamada <masahiroy@kernel.org> wrote: > > > > > > The .export_symbol section is discarded by the linker script, hence > > > no alignment is needed. Simplify the code. > > > > > > Signed-off-by: Masahiro Yamada <masahiroy@kernel.org> > > > --- > > > > > > include/linux/export.h | 10 +++------- > > > 1 file changed, 3 insertions(+), 7 deletions(-) > > > > > > diff --git a/include/linux/export.h b/include/linux/export.h > > > index 0bbd02fd351d..dff230bb5aca 100644 > > > --- a/include/linux/export.h > > > +++ b/include/linux/export.h > > > @@ -15,13 +15,9 @@ > > > */ > > > > > > #ifdef CONFIG_64BIT > > > -#define __EXPORT_SYMBOL_REF(sym) \ > > > - .balign 8 ASM_NL \ > > > - .quad sym > > > +#define __EXPORT_SYMBOL_PTR .quad > > > #else > > > -#define __EXPORT_SYMBOL_REF(sym) \ > > > - .balign 4 ASM_NL \ > > > - .long sym > > > +#define __EXPORT_SYMBOL_PTR .long > > > #endif > > > > > > #define ___EXPORT_SYMBOL(sym, license, ns) \ > > > @@ -29,7 +25,7 @@ > > > __export_symbol_##sym: ASM_NL \ > > > .asciz license ASM_NL \ > > > .asciz ns ASM_NL \ > > > - __EXPORT_SYMBOL_REF(sym) ASM_NL \ > > > + __EXPORT_SYMBOL_PTR sym ASM_NL \ > > > .previous > > > > > > #if defined(__DISABLE_EXPORTS) > > > -- > > > 2.40.1 > > > > > > > > > > > -- > Best Regards > Masahiro Yamada
On Fri, Jan 5, 2024 at 5:21 PM Sedat Dilek <sedat.dilek@gmail.com> wrote: > > On Thu, Jan 4, 2024 at 3:17 PM Masahiro Yamada <masahiroy@kernel.org> wrote: > > > > On Thu, Jan 4, 2024 at 1:05 AM Sedat Dilek <sedat.dilek@gmail.com> wrote: > > > > > > Happy new 2024 Masahiro, > > > > > > some small nits. > > > > > > What about adding links to the commits... > > > > > > linux/export: Fix alignment for 64-bit ksymtab entries > > > [ Upstream commit f6847807c22f6944c71c981b630b9fff30801e73 ] > > > > > > linux/export: Ensure natural alignment of kcrctab array > > > [ Upstream commit 753547de0daecbdbd1af3618987ddade325d9aaa ] > > > > > > ^^ AFAICS in linux-stable - v6.6.10-rc1 includes both > > > ^^ Is yours a follow-up and makes sense for linux-stable releases? > > > > > > No. > > > > This patch is a separate clean-up, not a bug fix. > > > > No need for back-porting. > > > > Thanks for the clarification. > > But it is an improvement - so no discarding at link-time means > reduction of build-time? > Amy numbers? The .export_symbol is already discarded. The alignment of discarded code is "don't care". So, this is a code clean-up. I do not think there is a noticeable difference in the build speed. See line 1005 of include/asm-generic/vmlinux.h.lds #define COMMON_DISCARDS \ SANITIZER_DISCARDS \ PATCHABLE_DISCARDS \ *(.discard) \ *(.discard.*) \ *(.export_symbol) \ *(.modinfo) \ /* ld.bfd warns about .gnu.version* even when not emitted */ \ *(.gnu.version*) > > > > > > > > > > > > ...and CC Helge Deller? > > > > CC in the patch - makes for me only sense if you agree on adding above > commit-ids of Helge's work. > I think this is good in sense of follow-able references. > > Best regards, > -Sedat- > > > > > You did it. > > > > > > Thanks. > > > > > > > > > > > > > Best regards, > > > -Sedat- > > > > > > Link: https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git/log/?h=linux-6.6.y > > > > > > On Wed, Jan 3, 2024 at 2:55 PM Masahiro Yamada <masahiroy@kernel.org> wrote: > > > > > > > > The .export_symbol section is discarded by the linker script, hence > > > > no alignment is needed. Simplify the code. > > > > > > > > Signed-off-by: Masahiro Yamada <masahiroy@kernel.org> > > > > --- > > > > > > > > include/linux/export.h | 10 +++------- > > > > 1 file changed, 3 insertions(+), 7 deletions(-) > > > > > > > > diff --git a/include/linux/export.h b/include/linux/export.h > > > > index 0bbd02fd351d..dff230bb5aca 100644 > > > > --- a/include/linux/export.h > > > > +++ b/include/linux/export.h > > > > @@ -15,13 +15,9 @@ > > > > */ > > > > > > > > #ifdef CONFIG_64BIT > > > > -#define __EXPORT_SYMBOL_REF(sym) \ > > > > - .balign 8 ASM_NL \ > > > > - .quad sym > > > > +#define __EXPORT_SYMBOL_PTR .quad > > > > #else > > > > -#define __EXPORT_SYMBOL_REF(sym) \ > > > > - .balign 4 ASM_NL \ > > > > - .long sym > > > > +#define __EXPORT_SYMBOL_PTR .long > > > > #endif > > > > > > > > #define ___EXPORT_SYMBOL(sym, license, ns) \ > > > > @@ -29,7 +25,7 @@ > > > > __export_symbol_##sym: ASM_NL \ > > > > .asciz license ASM_NL \ > > > > .asciz ns ASM_NL \ > > > > - __EXPORT_SYMBOL_REF(sym) ASM_NL \ > > > > + __EXPORT_SYMBOL_PTR sym ASM_NL \ > > > > .previous > > > > > > > > #if defined(__DISABLE_EXPORTS) > > > > -- > > > > 2.40.1 > > > > > > > > > > > > > > > > > -- > > Best Regards > > Masahiro Yamada
On Wed, Jan 03, 2024 at 10:54:54PM +0900, Masahiro Yamada wrote: > The .export_symbol section is discarded by the linker script, hence > no alignment is needed. Simplify the code. > > Signed-off-by: Masahiro Yamada <masahiroy@kernel.org> With this patch in linux-next, all sh4 builds fail. Building sh:defconfig ... failed -------------- Error log: <stdin>:1519:2: warning: #warning syscall clone3 not implemented [-Wcpp] {standard input}: Assembler messages: {standard input}:6: Error: misaligned data {standard input}:7: Error: misaligned data {standard input}:8: Error: misaligned data make[6]: *** [scripts/Makefile.build:243: arch/sh/boards/mach-x3proto/ilsel.o] Error 1 Reverting this patch fixes the problem. Guenter --- # bad: [8cb47d7cd090a690c1785385b2f3d407d4a53ad0] Add linux-next specific files for 20240110 # good: [0dd3ee31125508cd67f7e7172247f05b7fd1753a] Linux 6.7 git bisect start 'HEAD' 'v6.7' # bad: [b14da9dd6f977c6e18a4c81e4b92692fd9b6997f] Merge branch 'main' of git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git git bisect bad b14da9dd6f977c6e18a4c81e4b92692fd9b6997f # bad: [e85ebbed67d0d74e8602d11a496183d8f65e0b2f] Merge branch 'for-next' of https://evilpiepirate.org/git/bcachefs.git git bisect bad e85ebbed67d0d74e8602d11a496183d8f65e0b2f # bad: [65d7e41dbca9e31017cfd36c2ce4cb8408b1f2c6] Merge branch 'perf-tools-next' of git://git.kernel.org/pub/scm/linux/kernel/git/perf/perf-tools-next.git git bisect bad 65d7e41dbca9e31017cfd36c2ce4cb8408b1f2c6 # good: [fb46e22a9e3863e08aef8815df9f17d0f4b9aede] Merge tag 'mm-stable-2024-01-08-15-31' of git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm git bisect good fb46e22a9e3863e08aef8815df9f17d0f4b9aede # good: [fd38dd6abda589a8771e7872e4dea28c99c6a6ef] kselftest/alsa - conf: Stringify the printed errno in sysfs_get() git bisect good fd38dd6abda589a8771e7872e4dea28c99c6a6ef # good: [8a0bd6d47301eefd80fdfeb3a00ec09011546155] Merge branch 'mtd/fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git git bisect good 8a0bd6d47301eefd80fdfeb3a00ec09011546155 # good: [16f533ade706d33e60324ff32e526bda20bccbd9] perf unwind: Use function to add missing maps lock git bisect good 16f533ade706d33e60324ff32e526bda20bccbd9 # bad: [c6113f13957f918762d54080b2e2adbe0d7889e5] linux/export.h: remove unneeded .balign directive git bisect bad c6113f13957f918762d54080b2e2adbe0d7889e5 # good: [d821f8a26efb6789666d70ce7a8f27df6c33c12e] sparc: vdso: use $(addprefix ) instead of $(foreach ) git bisect good d821f8a26efb6789666d70ce7a8f27df6c33c12e # good: [9c65810cfb215f40f14d2c00694911fbc5408761] kbuild: deb-pkg: split debian/copyright from the mkdebian script git bisect good 9c65810cfb215f40f14d2c00694911fbc5408761 # good: [5e73758b43c3defba2578df6d3a53e942fa6b41e] kbuild: deb-pkg: use more debhelper commands in builddeb git bisect good 5e73758b43c3defba2578df6d3a53e942fa6b41e # good: [343f74e7779c482c2fd4492cc6b45c64e18fe321] kbuild: deb-pkg: make debian/rules quiet by default git bisect good 343f74e7779c482c2fd4492cc6b45c64e18fe321 # good: [0832e9bcfb0dc48833bbc8054cdc9b697afa8cd9] kbuild: deb-pkg: build binary-arch in parallel git bisect good 0832e9bcfb0dc48833bbc8054cdc9b697afa8cd9 # good: [53bc4fa98f8dd5ccca9a8e038660bf0d30d9e22d] kbuild: deb-pkg: call more misc debhelper commands git bisect good 53bc4fa98f8dd5ccca9a8e038660bf0d30d9e22d # first bad commit: [c6113f13957f918762d54080b2e2adbe0d7889e5] linux/export.h: remove unneeded .balign directive
On Thu, Jan 11, 2024 at 8:11 AM Guenter Roeck <linux@roeck-us.net> wrote: > > On Wed, Jan 03, 2024 at 10:54:54PM +0900, Masahiro Yamada wrote: > > The .export_symbol section is discarded by the linker script, hence > > no alignment is needed. Simplify the code. > > > > Signed-off-by: Masahiro Yamada <masahiroy@kernel.org> > > With this patch in linux-next, all sh4 builds fail. Thanks. I dropped it. > > Building sh:defconfig ... failed > -------------- > Error log: > <stdin>:1519:2: warning: #warning syscall clone3 not implemented [-Wcpp] > {standard input}: Assembler messages: > {standard input}:6: Error: misaligned data > {standard input}:7: Error: misaligned data > {standard input}:8: Error: misaligned data > make[6]: *** [scripts/Makefile.build:243: arch/sh/boards/mach-x3proto/ilsel.o] Error 1 > > Reverting this patch fixes the problem. > > Guenter > > --- > # bad: [8cb47d7cd090a690c1785385b2f3d407d4a53ad0] Add linux-next specific files for 20240110 > # good: [0dd3ee31125508cd67f7e7172247f05b7fd1753a] Linux 6.7 > git bisect start 'HEAD' 'v6.7' > # bad: [b14da9dd6f977c6e18a4c81e4b92692fd9b6997f] Merge branch 'main' of git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git > git bisect bad b14da9dd6f977c6e18a4c81e4b92692fd9b6997f > # bad: [e85ebbed67d0d74e8602d11a496183d8f65e0b2f] Merge branch 'for-next' of https://evilpiepirate.org/git/bcachefs.git > git bisect bad e85ebbed67d0d74e8602d11a496183d8f65e0b2f > # bad: [65d7e41dbca9e31017cfd36c2ce4cb8408b1f2c6] Merge branch 'perf-tools-next' of git://git.kernel.org/pub/scm/linux/kernel/git/perf/perf-tools-next.git > git bisect bad 65d7e41dbca9e31017cfd36c2ce4cb8408b1f2c6 > # good: [fb46e22a9e3863e08aef8815df9f17d0f4b9aede] Merge tag 'mm-stable-2024-01-08-15-31' of git://git.kernel.org/pub/scm/linux/kernel/git/akpm/mm > git bisect good fb46e22a9e3863e08aef8815df9f17d0f4b9aede > # good: [fd38dd6abda589a8771e7872e4dea28c99c6a6ef] kselftest/alsa - conf: Stringify the printed errno in sysfs_get() > git bisect good fd38dd6abda589a8771e7872e4dea28c99c6a6ef > # good: [8a0bd6d47301eefd80fdfeb3a00ec09011546155] Merge branch 'mtd/fixes' of git://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git > git bisect good 8a0bd6d47301eefd80fdfeb3a00ec09011546155 > # good: [16f533ade706d33e60324ff32e526bda20bccbd9] perf unwind: Use function to add missing maps lock > git bisect good 16f533ade706d33e60324ff32e526bda20bccbd9 > # bad: [c6113f13957f918762d54080b2e2adbe0d7889e5] linux/export.h: remove unneeded .balign directive > git bisect bad c6113f13957f918762d54080b2e2adbe0d7889e5 > # good: [d821f8a26efb6789666d70ce7a8f27df6c33c12e] sparc: vdso: use $(addprefix ) instead of $(foreach ) > git bisect good d821f8a26efb6789666d70ce7a8f27df6c33c12e > # good: [9c65810cfb215f40f14d2c00694911fbc5408761] kbuild: deb-pkg: split debian/copyright from the mkdebian script > git bisect good 9c65810cfb215f40f14d2c00694911fbc5408761 > # good: [5e73758b43c3defba2578df6d3a53e942fa6b41e] kbuild: deb-pkg: use more debhelper commands in builddeb > git bisect good 5e73758b43c3defba2578df6d3a53e942fa6b41e > # good: [343f74e7779c482c2fd4492cc6b45c64e18fe321] kbuild: deb-pkg: make debian/rules quiet by default > git bisect good 343f74e7779c482c2fd4492cc6b45c64e18fe321 > # good: [0832e9bcfb0dc48833bbc8054cdc9b697afa8cd9] kbuild: deb-pkg: build binary-arch in parallel > git bisect good 0832e9bcfb0dc48833bbc8054cdc9b697afa8cd9 > # good: [53bc4fa98f8dd5ccca9a8e038660bf0d30d9e22d] kbuild: deb-pkg: call more misc debhelper commands > git bisect good 53bc4fa98f8dd5ccca9a8e038660bf0d30d9e22d > # first bad commit: [c6113f13957f918762d54080b2e2adbe0d7889e5] linux/export.h: remove unneeded .balign directive
diff --git a/include/linux/export.h b/include/linux/export.h index 0bbd02fd351d..dff230bb5aca 100644 --- a/include/linux/export.h +++ b/include/linux/export.h @@ -15,13 +15,9 @@ */ #ifdef CONFIG_64BIT -#define __EXPORT_SYMBOL_REF(sym) \ - .balign 8 ASM_NL \ - .quad sym +#define __EXPORT_SYMBOL_PTR .quad #else -#define __EXPORT_SYMBOL_REF(sym) \ - .balign 4 ASM_NL \ - .long sym +#define __EXPORT_SYMBOL_PTR .long #endif #define ___EXPORT_SYMBOL(sym, license, ns) \ @@ -29,7 +25,7 @@ __export_symbol_##sym: ASM_NL \ .asciz license ASM_NL \ .asciz ns ASM_NL \ - __EXPORT_SYMBOL_REF(sym) ASM_NL \ + __EXPORT_SYMBOL_PTR sym ASM_NL \ .previous #if defined(__DISABLE_EXPORTS)