From 15cb28218786ae80101ad0f6cc5a21c1ad9e8d3e Mon Sep 17 00:00:00 2001 From: Eugeni Dodonov Date: Mon, 5 Sep 2011 20:04:17 -0300 Subject: [PATCH] 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 --- tools/intel_gpu_top.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/tools/intel_gpu_top.c b/tools/intel_gpu_top.c index a2277cc..2c2ed39 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; -- 2.7.4