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> > --track-fds=3Drandom.<br> <br> Is this really related to descriptor tracking, though?<br></blockquote><div= ><br></div><div>Florian, you're right, I agree it'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'm proposing a new option then, called --mod= ify-fds=3D[no,high]</div><div>no is a default and high is what I've bee= n calling 'new' previously.</div><div>I'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'm not sure if 'random'= 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==--