On 24 April 2015 at 16:27, Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxxxxxxxx> wrote: > From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > > Measures DRM_IOCTL_MODE_SETCRTC and DRM_IOCTL_MODE_SETPLANE as proxy for > drm_atomic_helper_update_plane if I got it right. > > Discovered some slow cursor updates (1.6ms) so needed something to test > different kernel configs etc. > > Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > --- > benchmarks/.gitignore | 1 + > benchmarks/Makefile.sources | 3 +- > benchmarks/kms_atomic.c | 193 ++++++++++++++++++++++++++++++++++++++++++++ > lib/igt_kms.c | 8 +- > lib/igt_kms.h | 5 ++ > 5 files changed, 205 insertions(+), 5 deletions(-) > create mode 100644 benchmarks/kms_atomic.c > > diff --git a/benchmarks/.gitignore b/benchmarks/.gitignore > index 09e5bd8..9a0b8af 100644 > --- a/benchmarks/.gitignore > +++ b/benchmarks/.gitignore > @@ -3,4 +3,5 @@ intel_upload_blit_large > intel_upload_blit_large_gtt > intel_upload_blit_large_map > intel_upload_blit_small > +kms_atomic > # Please keep sorted alphabetically > diff --git a/benchmarks/Makefile.sources b/benchmarks/Makefile.sources > index 60bdae2..25557d7 100644 > --- a/benchmarks/Makefile.sources > +++ b/benchmarks/Makefile.sources > @@ -3,4 +3,5 @@ bin_PROGRAMS = \ > intel_upload_blit_large_gtt \ > intel_upload_blit_large_map \ > intel_upload_blit_small \ > - gem_userptr_benchmark > + gem_userptr_benchmark \ > + kms_atomic > diff --git a/benchmarks/kms_atomic.c b/benchmarks/kms_atomic.c > new file mode 100644 > index 0000000..c758f66 > --- /dev/null > +++ b/benchmarks/kms_atomic.c > @@ -0,0 +1,193 @@ > +/* > + * Copyright © 2015 Intel Corporation > + * > + * Permission is hereby granted, free of charge, to any person obtaining a > + * copy of this software and associated documentation files (the "Software"), > + * to deal in the Software without restriction, including without limitation > + * the rights to use, copy, modify, merge, publish, distribute, sublicense, > + * and/or sell copies of the Software, and to permit persons to whom the > + * Software is furnished to do so, subject to the following conditions: > + * > + * The above copyright notice and this permission notice (including the next > + * paragraph) shall be included in all copies or substantial portions of the > + * Software. > + * > + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR > + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, > + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL > + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER > + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING > + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS > + * IN THE SOFTWARE. > + * > + */ > + > +#include <math.h> > +#include <sys/time.h> > + > +#include "drmtest.h" > +#include "igt_debugfs.h" > +#include "igt_kms.h" > +#include "igt_core.h" > +#include "intel_chipset.h" Although this isn't a test case, it might still be nice to add a short description for it using the IGT_TEST_DESCRIPTION macro. > + > +typedef struct { > + int gfx_fd; > + igt_display_t display; > + struct igt_fb fb; > + struct igt_fb fb_modeset; > +} data_t; > + > + > +static void prepare_crtc(data_t *data, igt_output_t *output, enum pipe pipe, > + igt_plane_t *plane) > +{ > + drmModeModeInfo *mode; > + igt_display_t *display = &data->display; > + int fb_id, fb_modeset_id; > + unsigned int w, h; > + uint64_t tiling = LOCAL_DRM_FORMAT_MOD_NONE; > + uint32_t pixel_format = DRM_FORMAT_XRGB8888; > + enum igt_commit_style commit = COMMIT_LEGACY; > + igt_plane_t *primary; > + > + igt_output_set_pipe(output, pipe); > + > + mode = igt_output_get_mode(output); > + > + w = mode->hdisplay; > + h = mode->vdisplay; > + > + fb_modeset_id = igt_create_fb(data->gfx_fd, > + w, h, > + pixel_format, > + tiling, > + &data->fb_modeset); > + igt_assert(fb_modeset_id); > + > + /* > + * With igt_display_commit2 and COMMIT_UNIVERSAL, we call just the > + * setplane without a modeset. So, to be able to call > + * igt_display_commit and ultimately setcrtc to do the first modeset, > + * we create an fb covering the crtc and call commit > + */ > + > + primary = igt_output_get_plane(output, IGT_PLANE_PRIMARY); > + igt_plane_set_fb(primary, &data->fb_modeset); > + igt_display_commit(display); > + > + fb_id = igt_create_fb(data->gfx_fd, > + w, h, > + pixel_format, > + tiling, > + &data->fb); > + igt_assert(fb_id); > + > + igt_plane_set_fb(plane, &data->fb); > + > + if (plane->is_primary || plane->is_cursor) { > + igt_require(data->display.has_universal_planes); > + commit = COMMIT_UNIVERSAL; > + } > + > + igt_display_commit2(display, commit); > +} > + > +static void cleanup_crtc(data_t *data, igt_output_t *output, igt_plane_t *plane) > +{ > + igt_display_t *display = &data->display; > + > + igt_remove_fb(data->gfx_fd, &data->fb); > + igt_remove_fb(data->gfx_fd, &data->fb_modeset); > + > + /* XXX: see the note in prepare_crtc() */ > + if (!plane->is_primary) { > + igt_plane_t *primary; > + > + primary = igt_output_get_plane(output, IGT_PLANE_PRIMARY); > + igt_plane_set_fb(primary, NULL); > + } > + > + igt_plane_set_fb(plane, NULL); > + igt_output_set_pipe(output, PIPE_ANY); > + > + igt_display_commit(display); > +} > + > +static double elapsed(const struct timeval *start, > + const struct timeval *end, > + int loop) > +{ > + return (1e6*(end->tv_sec - start->tv_sec) + (end->tv_usec - start->tv_usec))/loop; > +} > + > +static void test_commit_speed(data_t *data, enum igt_plane plane_type) > +{ > + igt_display_t *display = &data->display; > + igt_output_t *output; > + enum pipe pipe; > + enum igt_commit_style commit = COMMIT_LEGACY; > + unsigned int i; > + const unsigned int loops = 1000; > + struct timeval start, end; > + > + if (plane_type == IGT_PLANE_PRIMARY || plane_type == IGT_PLANE_CURSOR) { > + igt_require(data->display.has_universal_planes); > + commit = COMMIT_UNIVERSAL; > + } > + > + for_each_connected_output(display, output) { > + for_each_pipe(display, pipe) { > + igt_plane_t *plane; > + > + igt_output_set_pipe(output, pipe); > + plane = igt_output_get_plane(output, plane_type); > + > + prepare_crtc(data, output, pipe, plane); > + > + igt_display_commit2(display, commit); > + > + gettimeofday(&start, NULL); > + for (i = loops; i > 0; i--) { > + plane->position_changed = true; > + igt_plane_commit(plane, output, commit, false); > + } > + gettimeofday(&end, NULL); > + igt_info("Pipe: %u Plane: %u Time: %7.3fµs\n", > + pipe, plane->index, > + elapsed(&start, &end, loops)); > + > + kmstest_restore_vt_mode(); > + kmstest_set_vt_graphics_mode(); > + > + cleanup_crtc(data, output, plane); > + } > + } > +} > + > +igt_main > +{ > + data_t data = {}; > + > + igt_skip_on_simulation(); > + > + igt_fixture { > + data.gfx_fd = drm_open_any_master(); > + > + kmstest_set_vt_graphics_mode(); > + > + igt_display_init(&data.display, data.gfx_fd); > + } > + > + igt_subtest_f("setcrtc-speed") { > + test_commit_speed(&data, IGT_PLANE_PRIMARY); > + } > + > + igt_subtest_f("setplane-speed") { > + test_commit_speed(&data, IGT_PLANE_2); > + } > + > + igt_fixture { > + igt_display_fini(&data.display); > + } > +} > diff --git a/lib/igt_kms.c b/lib/igt_kms.c > index 14abae8..151b157 100644 > --- a/lib/igt_kms.c > +++ b/lib/igt_kms.c > @@ -1488,10 +1488,10 @@ static int igt_primary_plane_commit_legacy(igt_plane_t *primary, > * Commit position and fb changes to a plane. The value of @s will determine > * which API is used to do the programming. > */ > -static int igt_plane_commit(igt_plane_t *plane, > - igt_output_t *output, > - enum igt_commit_style s, > - bool fail_on_error) Since igt_plane_commit is now public, could you add some API documentation for it here? > +int igt_plane_commit(igt_plane_t *plane, > + igt_output_t *output, > + enum igt_commit_style s, > + bool fail_on_error) > { > if (plane->is_cursor && s == COMMIT_LEGACY) { > return igt_cursor_commit_legacy(plane, output, fail_on_error); > diff --git a/lib/igt_kms.h b/lib/igt_kms.h > index 565df14..58f7a32 100644 > --- a/lib/igt_kms.h > +++ b/lib/igt_kms.h > @@ -255,6 +255,11 @@ void igt_plane_set_rotation(igt_plane_t *plane, igt_rotation_t rotation); > > void igt_wait_for_vblank(int drm_fd, enum pipe pipe); > > +int igt_plane_commit(igt_plane_t *plane, > + igt_output_t *output, > + enum igt_commit_style s, > + bool fail_on_error); > + > #define for_each_connected_output(display, output) \ > for (int i__ = 0; i__ < (display)->n_outputs; i__++) \ > if ((output = &(display)->outputs[i__]), output->valid) > -- > 2.3.5 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx