From: James Hogan <james.hogan@imgtec.com>
To: Denis Efremov <yefremov.denis@gmail.com>
Cc: <linux-kernel@vger.kernel.org>, <trivial@kernel.org>,
<ldv-project@linuxtesting.org>
Subject: Re: [PATCH 12/21] metag: delay: remove inline marking of EXPORT_SYMBOL functions
Date: Thu, 9 May 2013 12:03:46 +0100 [thread overview]
Message-ID: <518B8292.3030700@imgtec.com> (raw)
In-Reply-To: <1368095819-11011-1-git-send-email-yefremov.denis@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 1200 bytes --]
On 09/05/13 11:36, Denis Efremov wrote:
> EXPORT_SYMBOL and inline directives are contradictory to each other.
> The patch fixes this inconsistency.
>
> Found by Linux Driver Verification project (linuxtesting.org).
>
> Signed-off-by: Denis Efremov <yefremov.denis@gmail.com>
> ---
> arch/metag/lib/delay.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/metag/lib/delay.c b/arch/metag/lib/delay.c
> index 0b308f4..7ff0ca7 100644
> --- a/arch/metag/lib/delay.c
> +++ b/arch/metag/lib/delay.c
> @@ -36,7 +36,7 @@ void __delay(unsigned long loops)
> }
> EXPORT_SYMBOL(__delay);
>
> -inline void __const_udelay(unsigned long xloops)
> +void __const_udelay(unsigned long xloops)
> {
> u64 loops = (u64)xloops * (u64)loops_per_jiffy * HZ;
> __delay(loops >> 32);
>
The inline still has an effect on code generation if other functions in
the same compilation unit use it, so this isn't really a trivial change.
I.e. before this patch __udelay and __ndelay inlined __const_udelay, but
after it they are changed to just multiply & branch.
Therefore I'd like to hear some more justification for these changes.
Cheers
James
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
next prev parent reply other threads:[~2013-05-09 11:04 UTC|newest]
Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-09 7:57 [PATCH 00/21] Exported funtions that are marked inline Denis Efremov
2013-05-09 7:58 ` [PATCH 01/21] [SCSI] libiscsi: remove inline marking of EXPORT_SYMBOL functions Denis Efremov
2013-05-09 7:58 ` [PATCH 02/21] fs: " Denis Efremov
2013-05-09 8:07 ` Al Viro
2013-05-09 8:42 ` Denis
2013-05-09 7:58 ` [PATCH 03/21] x86: " Denis Efremov
2013-05-09 13:50 ` H. Peter Anvin
2013-05-09 7:58 ` [PATCH 04/21] NFC: " Denis Efremov
2013-05-09 7:58 ` [PATCH 05/21] 9p: " Denis Efremov
2013-05-09 9:19 ` [PATCH 06/21] staging: nvec: " Denis Efremov
2013-05-09 9:19 ` [PATCH 07/21] netfilter: nf_nat: " Denis Efremov
2013-05-09 9:47 ` David Laight
2013-05-09 9:19 ` [PATCH 08/21] iwlegacy: " Denis Efremov
2013-05-09 9:19 ` [PATCH 09/21] dw_dmac: " Denis Efremov
[not found] ` <CAHp75VfWzzevr6AQzE25P8FUXd=amzL5-XgkJXvbRDESF=M76A@mail.gmail.com>
2013-05-12 15:13 ` Viresh Kumar
2013-05-30 18:01 ` Vinod Koul
2013-05-09 9:19 ` [PATCH 10/21] mm: filemap: " Denis Efremov
2013-05-09 9:19 ` [PATCH 11/21] ipv4: ip_output: " Denis Efremov
2013-05-11 23:13 ` David Miller
2013-05-09 10:36 ` [PATCH 12/21] metag: delay: " Denis Efremov
2013-05-09 11:03 ` James Hogan [this message]
2013-05-09 10:36 ` [PATCH 13/21] openrisc: " Denis Efremov
2013-05-09 10:45 ` Will Deacon
2013-05-09 10:36 ` [PATCH 14/21] arm64: " Denis Efremov
2013-05-09 10:39 ` Catalin Marinas
2013-05-09 10:36 ` [PATCH 15/21] arch/tile: " Denis Efremov
2013-05-09 18:02 ` Chris Metcalf
2013-05-09 10:36 ` [PATCH 16/21] sparc: " Denis Efremov
2013-05-09 10:36 ` [PATCH 17/21] CRISv32: " Denis Efremov
2013-05-14 16:35 ` Jesper Nilsson
2013-05-09 10:36 ` [PATCH 18/21] m68k/sun3/: " Denis Efremov
2013-05-09 11:09 ` Geert Uytterhoeven
2013-05-09 10:36 ` [PATCH 19/21] MIPS: MSP71xx: " Denis Efremov
2013-05-09 10:36 ` [PATCH 20/21] powerpc/ps3: " Denis Efremov
2013-05-09 11:35 ` Geoff Levand
2013-05-17 16:05 ` Geert Uytterhoeven
2013-05-28 23:14 ` Geoff Levand
2013-05-09 10:36 ` [PATCH 21/21] ARM: " Denis Efremov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=518B8292.3030700@imgtec.com \
--to=james.hogan@imgtec.com \
--cc=ldv-project@linuxtesting.org \
--cc=linux-kernel@vger.kernel.org \
--cc=trivial@kernel.org \
--cc=yefremov.denis@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox