sthib...@osl.iu.edu wrote:
> Author: sthibaul
> Date: 2010-04-04 09:20:28 EDT (Sun, 04 Apr 2010)
> New Revision: 1910
> URL: https://svn.open-mpi.org/trac/hwloc/changeset/1910
>
> Log:
> add --top option to lstopo to show bound processes
>   

Great, thanks!

I think it'd be better if lstopo in non-verbose mode would print those
Misc objects after everything else with same cpuset. Right now we get

Machine (32GB)
  NUMANode #0 (phys=0 8191MB) + Socket #0 + L3 #0 (2048KB)
    L2 #0 (512KB) + L1 #0 (64KB) + Core #0 + PU #0 (phys=0)
    L2 #1 (512KB) + L1 #1 (64KB) + Core #1 + PU #1 (phys=1)
    15887 /home/bgoglin/hwloc/build/utils/.libs/lt-lstopo + L2 #2 (512KB) + L1 
#2 (64KB) + Core #2 + PU #2 (phys=2)

while I would prefer

Machine (32GB)
  NUMANode #0 (phys=0 8191MB) + Socket #0 + L3 #0 (2048KB)
    L2 #0 (512KB) + L1 #0 (64KB) + Core #0 + PU #0 (phys=0)
    L2 #1 (512KB) + L1 #1 (64KB) + Core #1 + PU #1 (phys=1)
    L2 #2 (512KB) + L1 #2 (64KB) + Core #2 + PU #2 (phys=2) + 15887 
/home/bgoglin/hwloc/build/utils/.libs/lt-lstop


In non-verbose mode, I am not sure what's best between

Machine (phys=0 total=33553780KB Supermicro H8QM8)
  NUMANode #0 (phys=0 local=8387956KB total=8387956KB)
    Socket #0 (phys=0)
      L3Cache #0 (2048KB)
        16972 /home/bgoglin/hwloc/build/utils/.libs/lt-lstopo "16972 
/home/bgoglin/hwloc/build/utils/.libs/lt-lstopo"
          L2Cache #2 (512KB)
            L1Cache #2 (64KB)
              Core #2 (phys=2)
                PU #2 (phys=2)

and

Machine (phys=0 total=33553780KB Supermicro H8QM8)
  NUMANode #0 (phys=0 local=8387956KB total=8387956KB)
    Socket #0 (phys=0)
      L3Cache #0 (2048KB)
        L2Cache #2 (512KB)
          L1Cache #2 (64KB)
            Core #2 (phys=2)
              PU #2 (phys=2)
                16972 /home/bgoglin/hwloc/build/utils/.libs/lt-lstopo "16972 
/home/bgoglin/hwloc/build/utils/.libs/lt-lstopo"


So, do we want something like:
1) insert_misc_by_cpuset() to always insert below objects with same cpusets? 
Might be better for some use cases, and worse for others...
2) have a parameter to switch between current behavior and (1) ?
3) add a quirk to lstopo to show Misc objects last? this one would also let us 
replace "+" with another character right before the PID in the non-verbose 
output

/me going back to easter eggs...
Brice


Reply via email to