[PATCH weston 6/6] compositor-headless: Add test mode

Bryce Harrington bryce at osg.samsung.com
Wed Jan 28 18:42:07 PST 2015


On Tue, Jan 27, 2015 at 12:38:46PM +0000, Daniel Stone wrote:
> Hi,
> 
> On 19 December 2014 at 18:45, Derek Foreman <derekf at osg.samsung.com> wrote:
> >  static int
> >  headless_output_repaint(struct weston_output *output_base,
> >                        pixman_region32_t *damage)
> >  {
> >         struct headless_output *output = (struct headless_output *) output_base;
> >         struct weston_compositor *ec = output->base.compositor;
> > +       struct wl_event_loop *loop;
> >
> >         ec->renderer->repaint_output(&output->base, damage);
> >
> >         pixman_region32_subtract(&ec->primary_plane.damage,
> >                                  &ec->primary_plane.damage, damage);
> >
> > -       wl_event_source_timer_update(output->finish_frame_timer, 16);
> > +       if (ec->test_mode) {
> > +               loop = wl_display_get_event_loop(ec->wl_display);
> > +               wl_event_loop_add_idle(loop, finish_frame_handler_idle, output_base);
> > +       } else
> > +               wl_event_source_timer_update(output->finish_frame_timer, 16);
> >
> >         return 0;
> >  }
> 
> Hmm, these make me a bit nervous, and like we should perhaps be doing
> this at a higher level - e.g. in the core repaint/finish_frame, rather
> than per-backend. That would mean we could do automated testing of,
> e.g., the DRM backend, when your hardware supports CRC summing, as at
> least Intel and Exynos do.
> 
> Was this something you looked at and discarded? If there's good reason
> to avoid it then fair enough, but if not, it'd be good to check out so
> we can get more equal testing.

Let's go ahead and defer this patchset to 1.8 with the other testsuite
work that's planned.  Maybe this can get some additional discussion
and/or a v2 of the set.

Bryce



More information about the wayland-devel mailing list