On Thu, 1 Apr 2021 at 15:58, Tvrtko Ursulin <tvrtko.ursu...@linux.intel.com> wrote: > > > On 01/04/2021 15:49, Matthew Auld wrote: > > On Wed, 31 Mar 2021 at 10:12, Tvrtko Ursulin > > <tvrtko.ursu...@linux.intel.com> wrote: > >> > >> From: Tvrtko Ursulin <tvrtko.ursu...@intel.com> > >> > >> Put a note on how to use JSON output into the man page. > >> > >> Signed-off-by: Tvrtko Ursulin <tvrtko.ursu...@intel.com> > >> References: https://gitlab.freedesktop.org/drm/igt-gpu-tools/-/issues/100 > >> --- > >> man/intel_gpu_top.rst | 5 +++++ > >> 1 file changed, 5 insertions(+) > >> > >> diff --git a/man/intel_gpu_top.rst b/man/intel_gpu_top.rst > >> index f6d74852558b..94fdc6520fd3 100644 > >> --- a/man/intel_gpu_top.rst > >> +++ b/man/intel_gpu_top.rst > >> @@ -81,6 +81,11 @@ Filter types: :: > >> pci pci:[vendor=%04x/name][,device=%04x][,card=%d] > >> vendor is hex number or vendor name > >> > >> +JSON OUTPUT > >> +=========== > >> + > >> +To parse the JSON as output by the tool the consumer should wrap its > >> entirety into square brackets ([ ]). This will make each sample point an > >> JSON array element and will avoid "Multiple root elements" JSON validation > >> error. > >> + > > > > a JSON array element > > > > It could be argued that this should go into LIMITATIONS? > > Hm yes, 50-50 or you lean more towards limitations? Otherwise inertia > says it can also stay as is.
Just keep it as-is. > > Regards, > > Tvrtko > > > Acked-by: Matthew Auld <matthew.a...@intel.com> > > > >> LIMITATIONS > >> =========== > >> > >> -- > >> 2.27.0 > >> > >> _______________________________________________ > >> igt-dev mailing list > >> igt-...@lists.freedesktop.org > >> https://lists.freedesktop.org/mailman/listinfo/igt-dev _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
