diff options
author | Eugeni Dodonov <eugeni.dodonov@intel.com> | 2011-09-05 20:04:17 -0300 |
---|---|---|
committer | Eugeni Dodonov <eugeni.dodonov@intel.com> | 2011-09-05 20:04:17 -0300 |
commit | 15cb28218786ae80101ad0f6cc5a21c1ad9e8d3e (patch) | |
tree | 59003e4b713537061bafa7d4a33f202bcee15da4 /tools/intel_gpu_top.c | |
parent | 8ad08546e5d6189d2c778b6175e97b486332d37e (diff) |
intel_gpu_top: access hardware before running profiling command
Without this patch, intel_gpu_top will try to get access to the hardware
after running the profiling command in the background. In case such access
fails, the main process quits, but the background one will continue
running.
So let's move this test up, so it is done before we fork.
Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>
Diffstat (limited to 'tools/intel_gpu_top.c')
-rw-r--r-- | tools/intel_gpu_top.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/tools/intel_gpu_top.c b/tools/intel_gpu_top.c index a2277cc5..2c2ed393 100644 --- a/tools/intel_gpu_top.c +++ b/tools/intel_gpu_top.c @@ -486,6 +486,11 @@ int main(int argc, char **argv) argc -= optind; argv += optind; + pci_dev = intel_get_pci_device(); + devid = pci_dev->device_id; + intel_get_mmio(pci_dev); + init_instdone_definitions(devid); + /* Do we have a command to run? */ if (cmd != NULL) { if (output != stdout) { @@ -514,11 +519,6 @@ int main(int argc, char **argv) } } - pci_dev = intel_get_pci_device(); - devid = pci_dev->device_id; - intel_get_mmio(pci_dev); - init_instdone_definitions(devid); - for (i = 0; i < num_instdone_bits; i++) { top_bits[i].bit = &instdone_bits[i]; top_bits[i].count = 0; |