On 08/21, Chris Wilson wrote: > Quoting Rodrigo Siqueira (2018-08-21 15:29:03) > > On 08/21, Chris Wilson wrote: > > > Quoting Rodrigo Siqueira (2018-08-21 00:54:45) > > > > The kms_flip test does not support drivers without VBlank which exclude > > > > some virtual drivers. This patch adds a function that checks if a module > > > > has a VBlank or not; if a module has VBlank than kms_flip will execute > > > > all the VBlank tests, otherwise, VBlank tests will be skipped. > > > > > > > > Signed-off-by: Rodrigo Siqueira <rodrigosiqueiramelo@xxxxxxxxx> > > > > --- > > > > lib/igt_aux.c | 14 ++++++++++++++ > > > > lib/igt_aux.h | 2 ++ > > > > tests/kms_flip.c | 26 ++++++++++++++++++++++++-- > > > > 3 files changed, 40 insertions(+), 2 deletions(-) > > > > > > > > diff --git a/lib/igt_aux.c b/lib/igt_aux.c > > > > index 1250d5c5..da5be4bb 100644 > > > > --- a/lib/igt_aux.c > > > > +++ b/lib/igt_aux.c > > > > @@ -564,6 +564,20 @@ bool igt_aub_dump_enabled(void) > > > > return dump_aub; > > > > } > > > > > > > > +bool igt_there_is_vblank(int drm_fd) > > > > > > I would suggest breaking out in the kms namespace, > > > > > > bool kms_has_vblank(int fd); > > Hi, > > > > Do you mean to move this function to igt_kms, right? If so, how about > > kmstest_has_vblank. > > Yes, start igt_kms.c to avoid letting igt_aux.c keep growing. (Its > mostly for unstructured miscellany). Personally, I think since this is > talking to the kernel and not our own intermediary, kms_has_vblank() > is more appropriate than kmstest_foo. I got it now! Thanks! :) I will work on the second version > -Chris -- Rodrigo Siqueira http://siqueira.tech Graduate Student Department of Computer Science University of São Paulo _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
