Re: [PATCH v2] struct: Fix handing of percpu symbols (when symbolic argument only)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



----- Original Message -----
> Hi Dave,       

> 
> Sorry about the delay. I completely forgot about this one!
> Please note I have only addressed the case when the argument to the
> 'struct' command is not an address:
> 

And lastly, the symbol:cpuspec argument to the struct command is meant
to be the address of the structure, not a pointer to the address:

  crash> help struct

  NAME
    struct - structure contents

  SYNOPSIS
    struct struct_name[.member[,member]][-o][-l offset][-rfuxdp]
           [address | symbol][:cpuspec] [count | -c count]

  ... [ cut ] ...

           symbol  symbolic reference to the address of a structure.
         :cpuspec  CPU specification for a per-cpu address or symbol:
                     :             CPU of the currently selected task.
                     :a[ll]        all CPUs.
                     :#[-#][,...]  CPU list(s), e.g. "1,3,5", "1-3",
                                 or "1,3,5-7,10".
  ...

So I'm worried that if you wanted to see a structure member that actually
is a pointer, then the patch is going to dereference it and print bogus
data.
 
Dave

--
Crash-utility mailing list
Crash-utility@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/crash-utility



[Index of Archives]     [Fedora Development]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]

 

Powered by Linux