Messages in this thread |  | | Subject | Re: [PATCH V2 1/3] perf parse-regs: Split parse_regs | From | Ravi Bangoria <> | Date | Wed, 15 May 2019 12:19:58 +0530 |
| |
On 5/15/19 1:49 AM, kan.liang@linux.intel.com wrote: > From: Kan Liang <kan.liang@linux.intel.com> > > The available registers for --int-regs and --user-regs may be different, > e.g. XMM registers. > > Split parse_regs into two dedicated functions for --int-regs and > --user-regs respectively. > > Modify the warning message. "--user-regs=?" should be applied to show > the available registers for --user-regs. > > Signed-off-by: Kan Liang <kan.liang@linux.intel.com> > ---
For patch 1 and 2, Tested-by: Ravi Bangoria <ravi.bangoria@linux.ibm.com>
Minor neat. Should we update document as well? May be something like:
tools/perf/Documentation/perf-record.txt
--user-regs:: Similar to -I, but capture user registers at sample time. To list the available user registers use --user-regs=\?.
Ravi
|  |