Hi all, So this thing turned out to be fairly ugly overall, and I don't think it's worth the effort to polish the series through some giant rebasing. make distcheck still works so I don't think I've fumbled this too badly and will hence just push the patches as-is. I'm still very much interested in feedback, I'll simply apply that as follow-up patches. For those who want a peak at what the i-g-t docs look like now: http://people.freedesktop.org/~danvet/igt/ Everything but the modeset helpers is now neatly documented, with some intro texts where useful. I think for the igt_display/kms libraries we first need to give the apis a bit more polish before documentation makes sense. There's still an awful lot of boilerplate going on in kms tests ... Cheers, Daniel Daniel Vetter (28): lib: rename intel_pci.c to intel_chipset.c lib: consolidate chipset helpers in intel_chipset.[hc] lib/intel_chipset: intel_ prefix for pch global lib/intel_chipset: api docs lib: make rendercopy.h an internal header lib: make media_fill.h an internal header lib/intel_batchbuffer: un-inline buf_height/width lib/intel_batchbuffer: igt_ namespace for the buffer structure lib/intel_batchbuffer: igt_ prefix for rendercopy/mediafill funcs lib/intel_batchbuffer: drop cpu_mapping from igt_buf lib/intel_batchbuffer: api documentation for render copy/media fill lib: Move non-register things out of intel-gpu-tools.h tests|lib: remove assert.h includes lib: add #include "foo.h" lines like in manpages lib: remove uncessary #includes from headers lib: more unecessary header removal lib: unnecessary header removal for drmtest.h, part 1 lib: unnecessary header removal for drmtest.h, part 2 lib: rename intel_gpu_tools.h to intel_io.h lib/intel_io: rename mmio setup functions lib/intel_io: api documentation lib: s/drmtest_dumb_aub/igt_aub_dump_enabled/ lib: extract igt_aux.[hc] lib/igt_aux: api documentation lib: s/igt_env_set/igt_check_boolean_env_var lib: Use @include tag for include files lib/drmtest: api documentation lib/intel_batchbuffer: document header dependencies benchmarks/intel_upload_blit_large.c | 9 +- benchmarks/intel_upload_blit_large_gtt.c | 3 +- benchmarks/intel_upload_blit_large_map.c | 3 +- benchmarks/intel_upload_blit_small.c | 3 +- debugger/debug_rdata.c | 3 +- debugger/eudb.c | 2 +- demos/intel_sprite_on.c | 7 +- .../intel-gpu-tools/intel-gpu-tools-docs.xml | 5 +- lib/Makefile.sources | 9 +- lib/drmtest.c | 243 +++----------- lib/drmtest.h | 80 +++-- lib/gen6_render.h | 8 - lib/gen7_media.h | 2 + lib/gen8_media.h | 2 + lib/igt_aux.c | 373 +++++++++++++++++++++ lib/igt_aux.h | 67 ++++ lib/igt_core.c | 9 +- lib/igt_core.h | 7 + lib/igt_debugfs.c | 3 + lib/igt_debugfs.h | 3 +- lib/igt_kms.c | 14 +- lib/igt_kms.h | 10 +- lib/intel_batchbuffer.c | 93 ++++- lib/intel_batchbuffer.h | 96 +++++- lib/intel_chipset.c | 215 ++++++++++++ lib/intel_chipset.h | 26 ++ lib/intel_drm.c | 200 ----------- lib/intel_gpu_tools.h | 126 ------- lib/intel_io.h | 81 +++++ lib/intel_iosf.c | 61 +++- lib/intel_mmio.c | 133 +++++++- lib/intel_os.c | 171 ++++++++++ lib/intel_pci.c | 123 ------- lib/intel_reg_map.c | 3 +- lib/ioctl_wrappers.c | 6 +- lib/ioctl_wrappers.h | 5 + lib/media_fill.c | 14 - lib/media_fill.h | 49 +-- lib/media_fill_gen7.c | 16 +- lib/media_fill_gen8.c | 16 +- lib/rendercopy.h | 71 +--- lib/rendercopy_gen6.c | 51 ++- lib/rendercopy_gen7.c | 44 ++- lib/rendercopy_gen8.c | 58 ++-- lib/rendercopy_i830.c | 64 ++-- lib/rendercopy_i915.c | 30 +- tests/core_get_client_auth.c | 2 +- tests/core_getclient.c | 3 + tests/core_getstats.c | 5 + tests/ddi_compute_wrpll.c | 3 +- tests/drm_vma_limiter.c | 5 +- tests/drm_vma_limiter_cached.c | 5 +- tests/drm_vma_limiter_cpu.c | 5 +- tests/drm_vma_limiter_gtt.c | 5 +- tests/drv_suspend.c | 11 +- tests/eviction_common.c | 1 + tests/gem_bad_address.c | 5 +- tests/gem_bad_batch.c | 5 +- tests/gem_bad_blit.c | 5 +- tests/gem_bad_length.c | 2 +- tests/gem_basic.c | 2 +- tests/gem_caching.c | 10 +- tests/gem_close_race.c | 3 +- tests/gem_concurrent_blit.c | 10 +- tests/gem_cpu_reloc.c | 10 +- tests/gem_cs_prefetch.c | 10 +- tests/gem_cs_tlb.c | 9 +- tests/gem_ctx_bad_destroy.c | 4 +- tests/gem_ctx_bad_exec.c | 2 +- tests/gem_ctx_basic.c | 24 +- tests/gem_ctx_create.c | 4 +- tests/gem_ctx_exec.c | 8 +- tests/gem_double_irq_loop.c | 5 +- tests/gem_dummy_reloc_loop.c | 5 +- tests/gem_evict_alignment.c | 8 +- tests/gem_evict_everything.c | 3 +- tests/gem_exec_bad_domains.c | 5 +- tests/gem_exec_big.c | 2 +- tests/gem_exec_blt.c | 4 +- tests/gem_exec_faulting_reloc.c | 4 +- tests/gem_exec_lut_handle.c | 2 +- tests/gem_exec_nop.c | 4 +- tests/gem_fd_exhaustion.c | 3 + tests/gem_fence_thrash.c | 34 +- tests/gem_fenced_exec_thrash.c | 8 +- tests/gem_flink.c | 2 +- tests/gem_flink_race.c | 3 +- tests/gem_gtt_cpu_tlb.c | 4 +- tests/gem_gtt_hog.c | 3 +- tests/gem_gtt_speed.c | 2 +- tests/gem_hang.c | 5 +- tests/gem_hangcheck_forcewake.c | 5 +- tests/gem_largeobject.c | 2 +- tests/gem_linear_blits.c | 12 +- tests/gem_lut_handle.c | 2 +- tests/gem_madvise.c | 2 +- tests/gem_media_fill.c | 29 +- tests/gem_mmap.c | 2 +- tests/gem_mmap_gtt.c | 2 +- tests/gem_mmap_offset_exhaustion.c | 2 +- tests/gem_non_secure_batch.c | 5 +- tests/gem_partial_pwrite_pread.c | 12 +- tests/gem_persistent_relocs.c | 12 +- tests/gem_pin.c | 9 +- tests/gem_pipe_control_store_loop.c | 5 +- tests/gem_pread.c | 2 +- tests/gem_pread_after_blit.c | 12 +- tests/gem_pwrite.c | 2 +- tests/gem_pwrite_pread.c | 5 +- tests/gem_readwrite.c | 2 +- tests/gem_reg_read.c | 4 +- tests/gem_reloc_overflow.c | 5 +- tests/gem_reloc_vs_gpu.c | 12 +- tests/gem_render_copy.c | 40 ++- tests/gem_render_linear_blits.c | 31 +- tests/gem_render_tiled_blits.c | 27 +- tests/gem_reset_stats.c | 11 +- tests/gem_ring_sync_copy.c | 21 +- tests/gem_ring_sync_loop.c | 5 +- tests/gem_ringfill.c | 26 +- tests/gem_seqno_wrap.c | 22 +- tests/gem_set_tiling_vs_blt.c | 5 +- tests/gem_set_tiling_vs_gtt.c | 5 +- tests/gem_set_tiling_vs_pwrite.c | 4 +- tests/gem_storedw_batches_loop.c | 7 +- tests/gem_storedw_loop_blt.c | 5 +- tests/gem_storedw_loop_bsd.c | 5 +- tests/gem_storedw_loop_render.c | 5 +- tests/gem_storedw_loop_vebox.c | 5 +- tests/gem_stress.c | 87 +++-- tests/gem_threaded_access_tiled.c | 2 +- tests/gem_tiled_blits.c | 12 +- tests/gem_tiled_fence_blits.c | 10 +- tests/gem_tiled_partial_pwrite_pread.c | 12 +- tests/gem_tiled_pread.c | 5 +- tests/gem_tiled_pread_pwrite.c | 9 +- tests/gem_tiled_swapping.c | 9 +- tests/gem_tiling_max_stride.c | 5 +- tests/gem_unfence_active_buffers.c | 5 +- tests/gem_unref_active_buffers.c | 5 +- tests/gem_vmap_blits.c | 5 +- tests/gem_wait_render_timeout.c | 22 +- tests/gem_write_read_ring_switch.c | 12 +- tests/gen3_mixed_blits.c | 5 +- tests/gen3_render_linear_blits.c | 5 +- tests/gen3_render_mixed_blits.c | 5 +- tests/gen3_render_tiledx_blits.c | 5 +- tests/gen3_render_tiledy_blits.c | 5 +- tests/gen7_forcewake_mt.c | 5 +- tests/igt_fork_helper.c | 1 + tests/igt_simulation.c | 2 + tests/kms_addfb.c | 2 +- tests/kms_cursor_crc.c | 1 + tests/kms_fbc_crc.c | 12 +- tests/kms_flip.c | 17 +- tests/kms_pipe_crc_basic.c | 1 + tests/kms_plane.c | 1 + tests/kms_render.c | 7 +- tests/kms_setmode.c | 27 +- tests/pm_lpsp.c | 2 + tests/pm_pc8.c | 5 +- tests/pm_psr.c | 2 + tests/pm_rc6_residency.c | 2 + tests/pm_rps.c | 8 +- tests/prime_nv_api.c | 3 +- tests/prime_nv_pcopy.c | 5 +- tests/prime_nv_test.c | 5 +- tests/prime_self_import.c | 2 +- tests/prime_udl.c | 8 +- tests/testdisplay.c | 6 +- tools/intel_audio_dump.c | 9 +- tools/intel_backlight.c | 6 +- tools/intel_bios_reader.c | 4 +- tools/intel_dpio_read.c | 3 +- tools/intel_dpio_write.c | 3 +- tools/intel_error_decode.c | 3 +- tools/intel_forcewaked.c | 3 +- tools/intel_framebuffer_dump.c | 2 +- tools/intel_gpu_time.c | 6 +- tools/intel_gpu_top.c | 6 +- tools/intel_gtt.c | 3 +- tools/intel_infoframes.c | 8 +- tools/intel_l3_parity.c | 3 +- tools/intel_lid.c | 5 +- tools/intel_nc_read.c | 3 +- tools/intel_nc_write.c | 3 +- tools/intel_opregion_decode.c | 3 +- tools/intel_panel_fitter.c | 4 +- tools/intel_perf_counters.c | 3 +- tools/intel_punit_read.c | 3 +- tools/intel_punit_write.c | 3 +- tools/intel_reg_checker.c | 5 +- tools/intel_reg_dumper.c | 17 +- tools/intel_reg_read.c | 3 +- tools/intel_reg_snapshot.c | 5 +- tools/intel_reg_write.c | 3 +- tools/intel_stepping.c | 3 +- tools/intel_vga_read.c | 3 +- tools/intel_vga_write.c | 3 +- tools/quick_dump/Makefile.am | 6 +- tools/quick_dump/chipset_macro_wrap.c | 33 ++ tools/quick_dump/intel_chipset.c | 33 -- 202 files changed, 2401 insertions(+), 1398 deletions(-) create mode 100644 lib/igt_aux.c create mode 100644 lib/igt_aux.h create mode 100644 lib/intel_chipset.c delete mode 100644 lib/intel_drm.c delete mode 100644 lib/intel_gpu_tools.h create mode 100644 lib/intel_io.h create mode 100644 lib/intel_os.c delete mode 100644 lib/intel_pci.c delete mode 100644 lib/media_fill.c create mode 100644 tools/quick_dump/chipset_macro_wrap.c delete mode 100644 tools/quick_dump/intel_chipset.c -- 1.8.5.2 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
