All of lore.kernel.org
 help / color / mirror / Atom feed
From: Baruch Siach <baruch@tkos.co.il>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 1/1] package/strace: disable mpers support
Date: Tue, 08 Sep 2020 10:46:58 +0300	[thread overview]
Message-ID: <87363szp59.fsf@tarshish> (raw)
In-Reply-To: <20200908073601.GV14354@scaer>

Hi Yann

On Tue, Sep 08 2020, Yann E. MORIN wrote:
> On 2020-09-08 08:54 +0200, Thomas Petazzoni spake thusly:
>> On Tue, 8 Sep 2020 00:56:55 +0200
>> "Yann E. MORIN" <yann.morin.1998@free.fr> wrote:
> [--SNIP--]
>> > However, some systems may very well run a 64-bit kernel with a 32-bit
>> > userland. In that case, strace should be able to interpret the syscalls
>> > made by the process and the replies sent by the kernel, and those are
>> > done in the bitness of the process, not of the kernel.
>> > 
>> > So, we always want an strace that is of the same bitness as the
>> > userland. And so I indeed think we do not need this ill-named
>> > multi-personality support of strace.
>> > 
>> > Note that some people may however run such multi-bitness systems. But in
>> > that case, they had to build two userlands, a 32-bit one and a 64-bit
>> > one, and somehow aggregate the two after the fact. In that case, they
>> > would bear the responsibility to carry an strace for each bitness.
>> 
>> So, is this an Acked-by for this patch ? :-)
>
> It would, but I'd still like to have some feedback from Baruch, who had
> initial added of the mpers option, so may have further explanations to
> provide.

I only touched mpers (commit fc0d31caeedba) to fix build failure I
(re)introduced in a previous commit (10883729419). strace enables mpers
by default. I am fine with removing mpers support entirely.

baruch

-- 
                                                     ~. .~   Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
   - baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -

  reply	other threads:[~2020-09-08  7:46 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-09-07 20:24 [Buildroot] [PATCH 1/1] package/strace: disable mpers support Heiko Thiery
2020-09-07 22:56 ` Yann E. MORIN
2020-09-08  6:54   ` Thomas Petazzoni
2020-09-08  7:36     ` Yann E. MORIN
2020-09-08  7:46       ` Baruch Siach [this message]
2020-09-08  7:51 ` Yann E. MORIN
2020-09-11 21:33 ` Peter Korsgaard

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=87363szp59.fsf@tarshish \
    --to=baruch@tkos.co.il \
    --cc=buildroot@busybox.net \
    /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.