Re: [hwc PATCH v1] drm_hwcomposer: reorder source layers according to zorder

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Sep 21, 2017 at 05:02:21PM -0700, Zach Reizner wrote:
> On Thu, Sep 21, 2017 at 4:53 PM, Adrian Salido <salidoa@xxxxxxxxxx> wrote:
> > Precomp layers may be added to the back at different points which may
> > cause elements to be unsorted. Make sure that these are sorted after
> > provisioning planes to ensure right composition based on zorder.
> >
> > Signed-off-by: Adrian Salido <salidoa@xxxxxxxxxx>
> Reviewed-by: Zach Reizner <zachr@xxxxxxxxxxxx>

Applied.

Sean

> > ---
> >  drmdisplaycomposition.cpp | 3 +++
> >  1 file changed, 3 insertions(+)
> >
> > diff --git a/drmdisplaycomposition.cpp b/drmdisplaycomposition.cpp
> > index 293160bfd012..0f8084b39706 100644
> > --- a/drmdisplaycomposition.cpp
> > +++ b/drmdisplaycomposition.cpp
> > @@ -379,6 +379,9 @@ int DrmDisplayComposition::Plan(SquashState *squash,
> >      if (!i.plane())
> >        continue;
> >
> > +    // make sure that source layers are ordered based on zorder
> > +    std::sort(i.source_layers().begin(), i.source_layers().end());
> > +
> >      std::vector<DrmPlane *> *container;
> >      if (i.plane()->type() == DRM_PLANE_TYPE_PRIMARY)
> >        container = primary_planes;
> > --
> > 2.14.1.821.g8fa685d3b7-goog
> >
> > _______________________________________________
> > dri-devel mailing list
> > dri-devel@xxxxxxxxxxxxxxxxxxxxx
> > https://lists.freedesktop.org/mailman/listinfo/dri-devel
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/dri-devel

-- 
Sean Paul, Software Engineer, Google / Chromium OS
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux