Re: New --track-fds=new option expected behaviour

Alexandra Petlanova Hajkova <[email protected]> Tue, 7 Jan 2025 14:04:26 +0100
Newsgroups gmane.comp.debugging.valgrind
Message-ID <CAJVr-ENx_TFpNqX7aJrU_HrhGh7uc9w3dnvoEPQT=AnFB=28cQ@mail.gmail.com>
--===============6742165717808180624==
Content-Type: multipart/alternative; boundary="000000000000da3638062b1d7a60"

--000000000000da3638062b1d7a60
Content-Type: text/plain; charset="UTF-8"

Thank you everyone for the feedback!

>
> > --track-fds=random.
>
> Is this really related to descriptor tracking, though?
>

Florian, you're right, I agree it's better to transform this feature into a
new option.

>
> I would expected a separate option, with a couple of choices:
>
>   min (the POSIX default)
>   max (what you call new)
>   oldest (first no reuse; after running out of numbers, oldest close first)
>   random
>   strict-max
>   strict-oldest
>   strict-random
>
>
> I'm proposing a new option then, called --modify-fds=[no,high]
no is a default and high is what I've been calling 'new' previously.
I'm going to start with these to for the beginning and we might add
additional
options later if needed.

I'm not sure if 'random' is really needed. Florian, do you have a use for
it?

Thank you,
Alexandra

--000000000000da3638062b1d7a60
Content-Type: text/html; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

<div dir=3D"ltr"><div dir=3D"ltr"><br></div><div>Thank you everyone for the=
 feedback!</div><div><div class=3D"gmail_quote gmail_quote_container"><bloc=
kquote class=3D"gmail_quote" style=3D"margin:0px 0px 0px 0.8ex;border-left:=
1px solid rgb(204,204,204);padding-left:1ex"><br>
&gt; --track-fds=3Drandom.<br>
<br>
Is this really related to descriptor tracking, though?<br></blockquote><div=
><br></div><div>Florian, you&#39;re right, I agree it&#39;s better to trans=
form this feature into a new option. <br></div><blockquote class=3D"gmail_q=
uote" style=3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,2=
04);padding-left:1ex">
<br>
I would expected a separate option, with a couple of choices:<br>
<br>
=C2=A0 min (the POSIX default)<br>
=C2=A0 max (what you call new)<br>
=C2=A0 oldest (first no reuse; after running out of numbers, oldest close f=
irst)<br>
=C2=A0 random<br>
=C2=A0 strict-max<br>
=C2=A0 strict-oldest<br>
=C2=A0 strict-random<br>
<br><br></blockquote><div>I&#39;m proposing a new option then, called --mod=
ify-fds=3D[no,high]</div><div>no is a default and high is what I&#39;ve bee=
n calling &#39;new&#39; previously.</div><div>I&#39;m going to start with t=
hese to for the beginning and we might add additional</div><div>options lat=
er if needed.</div><div><br></div><div>I&#39;m not sure if &#39;random&#39;=
 is really needed. Florian, do you have a use for it?</div><div><br></div><=
div>Thank you,</div><div>Alexandra <br></div></div></div></div>

--000000000000da3638062b1d7a60--



--===============6742165717808180624==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline


--===============6742165717808180624==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

_______________________________________________
Valgrind-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/valgrind-users

--===============6742165717808180624==--