linux-parisc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Sam James <sam@gentoo.org>
To: Helge Deller <deller@gmx.de>
Cc: Sam James <sam@gentoo.org>,
	"linux-parisc@vger.kernel.org" <linux-parisc@vger.kernel.org>,
	joey.gouly@arm.com
Subject: Re: prctl call wrongly succeeds on HPPA?
Date: Fri, 10 Nov 2023 20:12:55 +0000	[thread overview]
Message-ID: <87zfzl8ix3.fsf@gentoo.org> (raw)
In-Reply-To: <0fec2446-a523-486f-9df7-4c6af9e2e0e3@gmx.de>


Helge Deller <deller@gmx.de> writes:

> On 11/3/23 13:53, Sam James wrote:
>> Sam James <sam@gentoo.org> writes:
>>> I recently hit an issue with systemd-254 which tries to use the new
>>> prctl(PR_SET_MDWE) for systemd's MemoryDenyWriteExecute functionality.
>
> Is this still a problem?

Yes. When I get time, I will play with Dave's changes to allow using
non-exeuctable stacks, but for now, it is broken until I can test these
(thanks dave for working on that, and helge for the kernel side).

>
>>> On HPPA, we still need executable stacks, so this option doesn't work
>>> and leads to a segfault on boot.
>
> For kernel we don't need it any longer.
> But there might be dependencies on glibc version and/or combination.
> So, I've currently lost overview if we still need executable stacks...
>

I don't remember which kernel version either.. I think it was last year
that you finished off all the DSO bits.

I had to configure binutils with --enable-default-execstack=no for it to
work in addition to Dave's GCC patches. But I did not test systemd yet...

(sorry, I know this is equally vague.)

>>> Should this call be succeeeding on HPPA, or should we reject it for
>>> now until we have things wired up?
>>>
>>> Reported to systemd at https://github.com/systemd/systemd/issues/29775.
>>
>> Lennart has made clear (and I don't think I disagree) that he considers
>> this squarely a kernel bug.
>
> I've read the various bug reports and looked at the kernel commits regarding, e.g.
>
> commit b507808ebce23561d4ff8c2aa1fb949fe402bc61
> Author: Joey Gouly <joey.gouly@arm.com>
> Date:   Thu Jan 19 16:03:43 2023 +0000
>
>     mm: implement memory-deny-write-execute as a prctl
>
> but what is prctl(PR_SET_MDWE, PR_MDWE*, 0, 0)... expected to return on parisc?
> EINVAL? ENOTSUP?

Maybe we can ask Joey or the ARM people what they expect the semantics
to be.

>
> Helge


  parent reply	other threads:[~2023-11-10 20:14 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-31  4:32 prctl call wrongly succeeds on HPPA? Sam James
2023-11-03 12:53 ` Sam James
2023-11-10 20:01   ` Helge Deller
2023-11-10 20:12     ` John David Anglin
2023-11-10 20:38       ` Helge Deller
2023-11-10 21:08         ` John David Anglin
2023-11-10 21:32           ` Sam James
2023-11-10 22:00             ` John David Anglin
2023-11-10 22:16               ` Sam James
2023-11-10 23:02                 ` John David Anglin
2023-11-17 14:55                   ` Helge Deller
2023-11-17 15:41                     ` John David Anglin
2023-11-18  5:52                       ` Sam James
2023-11-19 12:11                     ` Sam James
2023-11-10 20:12     ` Sam James [this message]
2023-11-10 20:25       ` Helge Deller
2023-11-10 20:31         ` Helge Deller

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=87zfzl8ix3.fsf@gentoo.org \
    --to=sam@gentoo.org \
    --cc=deller@gmx.de \
    --cc=joey.gouly@arm.com \
    --cc=linux-parisc@vger.kernel.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).