From: Alexander Graf <agraf@suse.de>
To: "Matwey V. Kornilov" <matwey.kornilov@gmail.com>,
Jonathan Corbet <corbet@lwn.net>
Cc: Peter Foley <pefoley2@pefoley.com>,
LKML <linux-kernel@vger.kernel.org>,
"linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
Jiri Kosina <jkosina@suse.cz>,
Randy Dunlap <rdunlap@infradead.org>,
"afaerber@suse.de" <afaerber@suse.de>
Subject: Re: [PATCH] Documentation: Restrict TSC test code to x86
Date: Sat, 01 Nov 2014 22:16:44 +0100 [thread overview]
Message-ID: <54554DBC.3050607@suse.de> (raw)
In-Reply-To: <5454CC07.1070605@gmail.com>
On 01.11.14 13:03, Matwey V. Kornilov wrote:
> 28.10.2014 16:12, Alexander Graf пишет:
>>
>>
>>
>>> Am 28.10.2014 um 13:47 schrieb Jonathan Corbet <corbet@lwn.net>:
>>>
>>> On Mon, 27 Oct 2014 20:07:51 -0400
>>> Peter Foley <pefoley2@pefoley.com> wrote:
>>>
>>>>> The prctl test code in Documentation/ tries to show how to
>>>>> use a call that only makes sense on x86. Restrict it there
>>>>> so that other platforms don't try to call asm("rdtsc").
>>>>>
>>>>> Signed-off-by: Alexander Graf <agraf@suse.de>
>>>>
>>>> Acked-by: Peter Foley <pefoley2@pefoley.com>
>>>
>>> Snagged into the docs tree, thanks.
>>
>> Awesome, please make sure this makes it into 3.18 - the build is broken on non-x86 archs there ;).
>>
>> Alex
>>
>
> Hi,
>
> Sorry for criticism, but the patch is not complete.
> CONFIG_X86 deals with target architecture, at the same time the problem
> deals with the host architecture.
>
> Imagine, that I run arm, aarch64 or something else and do
> cross-compiling kernel for x86 on it. Then CONFIG_X86 will evaluate to
> 'y' and make will try to compile the apps with the host-compiler, which
> is not x86 one.
Good point. Any ideas how I can easily limit this to x86 hosts?
Alex
next prev parent reply other threads:[~2014-11-01 21:16 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-28 0:03 [PATCH] Documentation: Restrict TSC test code to x86 Alexander Graf
2014-10-28 0:07 ` Peter Foley
2014-10-28 12:47 ` Jonathan Corbet
2014-10-28 13:12 ` Alexander Graf
2014-11-01 12:03 ` Matwey V. Kornilov
2014-11-01 21:16 ` Alexander Graf [this message]
2014-11-02 7:46 ` Matwey V. Kornilov
2014-11-02 8:16 ` Alexander Graf
2014-11-02 10:28 ` Matwey V. Kornilov
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=54554DBC.3050607@suse.de \
--to=agraf@suse.de \
--cc=afaerber@suse.de \
--cc=corbet@lwn.net \
--cc=jkosina@suse.cz \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=matwey.kornilov@gmail.com \
--cc=pefoley2@pefoley.com \
--cc=rdunlap@infradead.org \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.