[procps] Re: important extension to newlib

  • From: Craig Small <csmall@xxxxxxxxxxxx>
  • To: Jim Warner <james.warner@xxxxxxxxxxx>
  • Date: Thu, 12 Aug 2021 21:48:43 +1000

Hi Jim,
  That second set of patches is applied and pushed.  I tried it with the
same set of processes and they look the same no matter if I used the main
process or the thread.

One minor thing. It's more about the top man page and the -p (or -H)
option. If you specify show threads and the process ID you see the process
and its threads, which makes complete sense.
If you specify show threads and a thread ID, you (again) see all the
process threads.

So specifying the thread ID you see (just) that thread. Adding "show
thread" you see all threads for the process the specified thread is in.
I think this is the correct way of doing things, but it is (slightly)
unexpected, from the man page.

           Monitor  only  processes  with  specified  process  IDs.   This
option can be given up to 20 times, or you can provide a comma delimited
list with up to 20 pids.  Co-mingling both approaches is
            permitted.

Should it warn a user if you specify a thread? It's like no -H is select
thread but -H selects the process of the thread.

 - Craig


On Wed, 11 Aug 2021 at 00:35, Jim Warner <james.warner@xxxxxxxxxxx> wrote:

On 8/9/21 2:52 PM, Jim Warner wrote:
Anyway, attached is the fix, along with another patch.

Hi Craig,

Attached are revised replacement patches for the above fix.

I added a commit reference in #0001 and omitted an unnecessary line of
code in #0002.

If this reaches you after you've pushed the originals, a local reset and
forced push would be appreciated.  Otherwise, just toss the original
'newlib-for-2.tgz' tarball.

In either case, sorry for the inconvenience.

Thanks in advance.

Jim


Other related posts: