From: Stefano Brivio <sbrivio@redhat.com>
To: "Lawrence, Richard E" <rlawrence@tamu.edu>
Cc: David Gibson <david@gibson.dropbear.id.au>,
"passt-dev@passt.top" <passt-dev@passt.top>
Subject: Re: [PATCH v2] feat: Pass open files to child in pasta mode
Date: Wed, 21 Oct 2026 02:48:28 +0200 (CEST) [thread overview]
Message-ID: <20261021024828.7bfcf35e@elisabeth> (raw)
In-Reply-To: <DM3PR11MB8713C7C7AFD7A74B4C68F232DADA2@DM3PR11MB8713.namprd11.prod.outlook.com>
Richard,
On Fri, 14 Aug 2026 18:12:56 +0000
"Lawrence, Richard E" <rlawrence@tamu.edu> wrote:
> Howdy Stefano, David,
>
> I don't understand the plan. Are you waiting on me? Am I waiting on you?
>
> I could implement any of these ideas, but I don't wish to implement **all** of them. Can we reach a consensus?
Sorry for the extremely long delay here. I think at this point there are
no moving parts in isolation.c and surroundings, and we could go back to
one of those ideas.
None of the reworks that occurred meanwhile would seem to make ideas
based on partial close_ranges() or reading out procfs entries (as your
v2 does) any easier.
My two proposals (wrapper around open() maintaining an array of files
we shouldn't close, or having all open files in a specific struct /
array can check easily) weren't made any easier either.
At this point I would be inclined to suggest something like your
ealier v3 adding a specific configuration option.
To somewhat retain as much as possible of the original security
benefits, we could explicitly limit that to the case where pasta(1)
spawns another command, because the only use case where we have any
evidence of this need is starting Bubblewrap from pasta's command line.
I'm sorry this wait hasn't been particularly productive but at some
point it looked like we could introduce a easy way to defer closing
files so that the command spawned by pasta could simply inherit them
unconditionally. That doesn't seem to be the case though.
Let me know if you need any help in reviving your earlier v3, or if you
need any support with anything related.
--
Stefano
prev parent reply other threads:[~2026-10-21 0:48 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-31 13:26 Richard Lawrence
2026-08-03 4:44 ` David Gibson
2026-08-03 20:34 ` Lawrence, Richard E
2026-08-06 3:20 ` David Gibson
2026-08-06 21:54 ` Stefano Brivio
2026-08-14 18:12 ` Lawrence, Richard E
2026-08-14 21:43 ` Stefano Brivio
2026-08-15 13:31 ` David Gibson
2026-10-21 0:48 ` Stefano Brivio [this message]
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=20261021024828.7bfcf35e@elisabeth \
--to=sbrivio@redhat.com \
--cc=david@gibson.dropbear.id.au \
--cc=passt-dev@passt.top \
--cc=rlawrence@tamu.edu \
/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.
Code repositories for project(s) associated with this public inbox
https://passt.top/passt
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 IMAP folder(s).