summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2011-10-18tools/decode: don't forget to print the name of the last ringDaniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-10-17Allow intel_dump_decode to read data from stdin.Eugeni Dodonov
Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>
2011-10-17tools/intel_dump_decode: make devid parsing more usefulDaniel Vetter
We use hex for these! Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-10-17tools/decode: retain the ring nameDaniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-10-16tests: add gem_hangcheck_forcewakeDaniel Vetter
Provokes the forcewake warning when the hangcheck runs and no one waits for the gpu (and hence holds the dev->struct_mutex). Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-10-15tests/gem_fence_trash: some retuningDaniel Vetter
We actually want to test lru behaviour, so do a bit of work with the fence before yielding to the next thread (we use twice as many fences as there are, so yielding always is pretty bad, no matter how clever our fence stealing). Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-10-15tests/gem_*_blits: reduce buffer count to not trash swapDaniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-10-14Allow to output statistics to stdout.Eugeni Dodonov
This allows intel_gpu_top to run both in statistics-collecting mode (collecting the per-ring statistics in gnuplot-friendly format) and ncurses top-like mode at the same time. It also allows to output the statistics directly to stdout, by using "-o -", so the results can be parsed directly via a popen() parsing. If you are using intel_gpu_top as previously (without any command-line arguments), it should change nothing for you. If you were using its logging facilities (e.g., the '-o file'), note that the logging will keep running, but the detailed top-like interface will be on the screen at the same time. Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>
2011-10-11bios_reader: Allow forcing the device ID with DEVICE=0xNNNNAdam Jackson
As a pleasant side effect you can now feed it an opregion dump and it'll decode without crashing. Signed-off-by: Adam Jackson <ajax@redhat.com>
2011-10-11intel_reg_dumper: use none instead of NULL if no port is assigned to TRANS_DPJesse Barnes
Makes the output a little more readable.
2011-10-10Treat PPT like CPT as we do in the kernelJesse Barnes
Set the pch type to CPT if we detect a PPT as well.
2011-09-28tests: disable storedw tests on !renderDaniel Vetter
According to docs, this should work with ppgtt and just convert to noops otherwise. In reality, it hangs the machine. So just disable these for the moment. Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-28tools/decode: add missing shift to MI_SEM_MBOX decoderDaniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-16testdisplay: return 0 when successfulPaulo Zanoni
We use the "ret" variable to check function return values, and these values are not necessarily 0 on the successful cases, so we need to reset "ret" after we finish using it. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-15getstats: remove useless assertionPaulo Zanoni
getstats.c:48:2: warning: comparison of unsigned expression >= 0 is always true Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-15drm_open_any: explain why we are abort()ingPaulo Zanoni
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> [danvet: pimped error msg] Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-15testdisplay: remove wrong return statementPaulo Zanoni
It seems that the kernel patches required by the TEST_PLANES feature are still not upstream, so I didn't test this patch. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-15testdisplay: don't enter the main loop if dump_info or test_all_modesPaulo Zanoni
If calls inside update_display fail, the function returns and we don't quit the program if dump_info or test_all_modes. So we enter the main loop and keep waiting for user input, even on cases where we are not supposed to require user input. To fix this, we move the check to outside the update_display function. As a side effect, we also do cleanup instead of just exit(0) and we return 1 in case update_display failed. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-15testdisplay: test all modes if no option is providedPaulo Zanoni
Previously, when called without any arguments, the application required user input to finish. However, testdisplay is ran by "make check", and it is not a good idea to run a program that requires user input in "make check". So we change the default behavior to something that doesn't require user input. You can still access the previous behavior using the "-m" argument. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-15Add missing files to .gitignorePaulo Zanoni
Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-15testdisplay: fix compiler warningsPaulo Zanoni
testdisplay.c:117:1: warning: comparison between signed and unsigned integer expressions testdisplay.c:125:1: warning: comparison between signed and unsigned integer expressions testdisplay.c:145:1: warning: comparison between signed and unsigned integer expressions testdisplay.c:1060:16: warning: comparison between signed and unsigned integer expressions Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-15tests/gem_mmap_gtt: also test gtt pwrite pathsDaniel Vetter
This needs a properly pre-faulted dst bo. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-14tests: build all tests by defaultDaniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-14tests: separate testcases in gem_bad_lengthDaniel Vetter
Creating a zero-length bo should fail, so check that. Still run the minimal batchbuffer, but without the zero-length reloc. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-14tests: disable gem_bad_blitDaniel Vetter
Hangs gen3 and simply writes garbage into the unmappable part of gtt on gen4+, which might cause issues later on. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-13tests/gem_bad_length: dont throw garbage at the gpuDaniel Vetter
It doesn't like it. Really. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-13intel: we don't do distcheck for the debuggerBen Widawsky
2011-09-13intel_gpu_dump: really kill itDaniel Vetter
This is getting a bit hilarious ... Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-13intel_gpu_dump: also kill manpageDaniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-13tools: remove intel_gpu_dumpDaniel Vetter
Superseded by the kernel's i915_error_state dumping facility. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-13tools: beef up intel_dump_decodeDaniel Vetter
- now also handles ascii dumps like i915_ringbuffer_data - getoptified - totally lazy binary detector Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-13tests: exit(77) to properly mark tests as SKIPDaniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-12tests: mark gen3-only tests as suchDaniel Vetter
... instead of hanging the gpu. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-12tests: add basic test for missed irqsDaniel Vetter
Didn't catch any known bug, but can't hurt. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-12tests: check pipe_control coherencyDaniel Vetter
v2: proper support for gen6+ Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-12tools: add intel_genDaniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-12tests: basic ring<->cpu and ring<->ring testsDaniel Vetter
Using a dummy reloc that doesn't matter to trick the kernel into synchroizing the rings. v2: properly apply MI_NOOP workaround to MI_FLUSH_DW and switch to MI_COND_BATCH_BUFFER_END as a dummy command on the render ring to avoid PIPE_CONTROL errata. v3: somebody clever decided that in C, you cound from 1, i.e. I915_EXEC_RENDER == 1. It works now ... Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-09tests: split up gem_storedw_loop into ring specific variationsDaniel Vetter
Also start to shortly explain testcases with an easily-greppable header like this: /* * Testcase: * * [Possible further explanation.] * */ Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-09tests/gem_storedw: disable on pre-gen6Daniel Vetter
Doesn't work. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-09lib/batch: hackish interface to run batches on specific ringsDaniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-09test/gem_storedw_*: tune some moreDaniel Vetter
MI_*/PIPE_CONTROL writes need to be in DOMAIN_INSTRUCTION, because that is what mesa uses and I plan to use this to work around a gen6 ppgtt issue. Also testing with intentionally b0rked GFX_MODE on my snb shows that we need to increase the loop counter a bit to reliably hit the tlb invalidation problem. Test still completes within a few seconds. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-09tools: decode some more MI_ opsDaniel Vetter
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-09tools: decode MI_SEMAPHORE_MBOXDaniel Vetter
Juggle the length check up to avoid duplication. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-06tools/intel_gpu_dump: add hackish --devid parsingDaniel Vetter
For bugreports and whatnotelse. Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-06gem_vmap_blits: fix compilation if I915_PARAM_HAS_VMAP is not definedPaulo Zanoni
Previously, "make check" failed because the main() function was not defined. Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com> Tested-By: Eugeni Dodonov <eugeni.dodonov@intel.com> Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
2011-09-05intel_gpu_top: access hardware before running profiling commandEugeni Dodonov
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>
2011-09-05intel_gpu_tools: describe -e option in usage screenEugeni Dodonov
This patch adds a short description of '-e' option to the usage screen. Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>
2011-09-05intel_gpu_top: adopt to kernel coding styleEugeni Dodonov
This patch fixes adopts the code to the kernel coding_style.
2011-09-05intel_gpu_top: support profiling user-specified commandsEugeni Dodonov
This patch adds support for running intel_gpu_top to profile specific commands. The required command will be carried out in separate process, and main intel_gpu_top will leave when the child process will exit. Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>
2011-09-05intel_gpu_top: support non-interactive modeEugeni Dodonov
This patch adds support for non-interactive mode, invoked by running with '-o output' switch. In this case, no interactive output is being performed, but the execution statistics are being saved into the output file. The output file is generated in both human and gnuplot-readable format. Unlike interactive mode, where non-supported pipes and non-active registers are skipped, the content of such pipes and registers is recorded into the log file to simplify parsing and standardize the list of columns. Also, unlike interactive mode, the registers are not sorted according to the usage - this way, their variation over time can be analysed offline. Signed-off-by: Eugeni Dodonov <eugeni.dodonov@intel.com>