From: Shuah Khan <shuahkh-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
To: Kees Cook <keescook-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
Cc: AKASHI Takahiro
<takahiro.akashi-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>,
Andy Lutomirski <luto-kltTT9wpgjJwATOyAt5JVQ@public.gmane.org>,
Will Drewry <wad-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>,
Linux API <linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
LKML <linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Shuah Khan <shuahkh-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
Subject: Re: [PATCH v3] selftests/seccomp: build and pass on arm64
Date: Thu, 15 Oct 2015 20:09:27 -0600 [thread overview]
Message-ID: <56205C57.3070406@osg.samsung.com> (raw)
In-Reply-To: <CAGXu5jLsYyXxQNbPKVsdzfSbbgddpJiz2MzgwEdbLd7mhjDmxw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
On 10/15/2015 05:02 PM, Kees Cook wrote:
> On Thu, Oct 15, 2015 at 4:01 PM, Shuah Khan <shuahkh-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org> wrote:
>> On 10/15/2015 05:00 PM, Shuah Khan wrote:
>>> On 10/15/2015 04:07 PM, Kees Cook wrote:
>>>> On Thu, Oct 15, 2015 at 1:06 PM, Shuah Khan <shuahkh-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org> wrote:
>>>>> On 10/15/2015 12:42 PM, Kees Cook wrote:
>>>>>> On Thu, Oct 15, 2015 at 7:07 AM, Shuah Khan <shuahkh-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org> wrote:
>>>>>>> On 10/06/2015 01:30 PM, Kees Cook wrote:
>>>>>>>> Changing arm64 syscalls is done via a specific register set, more like s390
>>>>>>>> than like arm (specific ptrace call) and x86 (part of general registers).
>>>>>>>> Since (restarting) poll doesn't exist on arm64, switch to using nanosleep
>>>>>>>> for testing restart_syscall. And since it looks like the syscall ABI is
>>>>>>>> inconsistent on arm-compat, so we must work around it (and document it) in
>>>>>>>> the test.
>>>>>>>>
>>>>>>>> Signed-off-by: Kees Cook <keescook-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
>>>>>>>> ---
>>>>>>>> v3:
>>>>>>>> - correctly set syscall number on native arm64.
>>>>>>>> v2:
>>>>>>>> - switch to nanosleep from a bad mix of poll and ppoll for testing restart.
>>>>>>>> ---
>>>>>>>
>>>>>>> Is this good to go? Failed to apply to linux-kselftest next.
>>>>>>> If you can rebase and resend. I can get this into 4.4-rc1
>>>>>>
>>>>>> Yes please. :)
>>>>>>
>>>>>
>>>>> ok. Please rebase to linux-kselftest next and resend the patch.
>>>>
>>>> Am I looking at the right tree? linux-kselftest#next doesn't appear to
>>>> have the s390 patch that was included in 4.3.
>>>>
>>>
>>> I see what happened. Your patch is linux-next fixes and that went into
>>> 4.3-rc2. I can get linux-next rebase to 4.3-rc2 and get your patch in.
>>> Thanks for clearing this up.
>>
>> oops rebase linux-kselftest next i.e
>
> Okay, cool. Let me know if I need to do anything more. :)
>
Applied to linux-ksefltest next for 4.4-rc1
thanks,
-- Shuah
--
Shuah Khan
Sr. Linux Kernel Developer
Open Source Innovation Group
Samsung Research America (Silicon Valley)
shuahkh-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org | (970) 217-8978
WARNING: multiple messages have this Message-ID (diff)
From: Shuah Khan <shuahkh@osg.samsung.com>
To: Kees Cook <keescook@chromium.org>
Cc: AKASHI Takahiro <takahiro.akashi@linaro.org>,
Arnd Bergmann <arnd@arndb.de>,
Andy Lutomirski <luto@amacapital.net>,
Will Drewry <wad@chromium.org>,
Linux API <linux-api@vger.kernel.org>,
LKML <linux-kernel@vger.kernel.org>,
Shuah Khan <shuahkh@osg.samsung.com>
Subject: Re: [PATCH v3] selftests/seccomp: build and pass on arm64
Date: Thu, 15 Oct 2015 20:09:27 -0600 [thread overview]
Message-ID: <56205C57.3070406@osg.samsung.com> (raw)
In-Reply-To: <CAGXu5jLsYyXxQNbPKVsdzfSbbgddpJiz2MzgwEdbLd7mhjDmxw@mail.gmail.com>
On 10/15/2015 05:02 PM, Kees Cook wrote:
> On Thu, Oct 15, 2015 at 4:01 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>> On 10/15/2015 05:00 PM, Shuah Khan wrote:
>>> On 10/15/2015 04:07 PM, Kees Cook wrote:
>>>> On Thu, Oct 15, 2015 at 1:06 PM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>>>> On 10/15/2015 12:42 PM, Kees Cook wrote:
>>>>>> On Thu, Oct 15, 2015 at 7:07 AM, Shuah Khan <shuahkh@osg.samsung.com> wrote:
>>>>>>> On 10/06/2015 01:30 PM, Kees Cook wrote:
>>>>>>>> Changing arm64 syscalls is done via a specific register set, more like s390
>>>>>>>> than like arm (specific ptrace call) and x86 (part of general registers).
>>>>>>>> Since (restarting) poll doesn't exist on arm64, switch to using nanosleep
>>>>>>>> for testing restart_syscall. And since it looks like the syscall ABI is
>>>>>>>> inconsistent on arm-compat, so we must work around it (and document it) in
>>>>>>>> the test.
>>>>>>>>
>>>>>>>> Signed-off-by: Kees Cook <keescook@chromium.org>
>>>>>>>> ---
>>>>>>>> v3:
>>>>>>>> - correctly set syscall number on native arm64.
>>>>>>>> v2:
>>>>>>>> - switch to nanosleep from a bad mix of poll and ppoll for testing restart.
>>>>>>>> ---
>>>>>>>
>>>>>>> Is this good to go? Failed to apply to linux-kselftest next.
>>>>>>> If you can rebase and resend. I can get this into 4.4-rc1
>>>>>>
>>>>>> Yes please. :)
>>>>>>
>>>>>
>>>>> ok. Please rebase to linux-kselftest next and resend the patch.
>>>>
>>>> Am I looking at the right tree? linux-kselftest#next doesn't appear to
>>>> have the s390 patch that was included in 4.3.
>>>>
>>>
>>> I see what happened. Your patch is linux-next fixes and that went into
>>> 4.3-rc2. I can get linux-next rebase to 4.3-rc2 and get your patch in.
>>> Thanks for clearing this up.
>>
>> oops rebase linux-kselftest next i.e
>
> Okay, cool. Let me know if I need to do anything more. :)
>
Applied to linux-ksefltest next for 4.4-rc1
thanks,
-- Shuah
--
Shuah Khan
Sr. Linux Kernel Developer
Open Source Innovation Group
Samsung Research America (Silicon Valley)
shuahkh@osg.samsung.com | (970) 217-8978
next prev parent reply other threads:[~2015-10-16 2:09 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-06 19:30 [PATCH v3] selftests/seccomp: build and pass on arm64 Kees Cook
2015-10-06 19:30 ` Kees Cook
2015-10-15 14:07 ` Shuah Khan
[not found] ` <561FB33F.5030906-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
2015-10-15 18:42 ` Kees Cook
2015-10-15 18:42 ` Kees Cook
[not found] ` <CAGXu5jKXeu-kBie+wJODoAjoK1kX+ScFRSn0s3s_wvN+sHT1yw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-10-15 20:06 ` Shuah Khan
2015-10-15 20:06 ` Shuah Khan
[not found] ` <56200761.8050607-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
2015-10-15 22:07 ` Kees Cook
2015-10-15 22:07 ` Kees Cook
[not found] ` <CAGXu5jJMpSTBFqS3Xj0ftMYVwJuUU8MUbggrWW8wN5CGDBsLvA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-10-15 23:00 ` Shuah Khan
2015-10-15 23:00 ` Shuah Khan
[not found] ` <56203023.2060007-JPH+aEBZ4P+UEJcrhfAQsw@public.gmane.org>
2015-10-15 23:01 ` Shuah Khan
2015-10-15 23:01 ` Shuah Khan
2015-10-15 23:02 ` Kees Cook
[not found] ` <CAGXu5jLsYyXxQNbPKVsdzfSbbgddpJiz2MzgwEdbLd7mhjDmxw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-10-16 2:09 ` Shuah Khan [this message]
2015-10-16 2:09 ` Shuah Khan
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=56205C57.3070406@osg.samsung.com \
--to=shuahkh-jph+aebz4p+uejcrhfaqsw@public.gmane.org \
--cc=arnd-r2nGTMty4D4@public.gmane.org \
--cc=keescook-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org \
--cc=linux-api-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=luto-kltTT9wpgjJwATOyAt5JVQ@public.gmane.org \
--cc=takahiro.akashi-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
--cc=wad-F7+t8E8rja9g9hUCZPvPmw@public.gmane.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.