Chromium Code Reviews
DescriptionReland of zone/drm: Only reuse ScanoutBuffers with compatible modifiers (patchset #1 id:1 of https://codereview.chromium.org/2948513003/ )
Reason for revert:
Let's revert the revert and land the fix:
https://chromium-review.googlesource.com/c/540817/?polygerrit=1
Original issue's description:
> Revert of ozone/drm: Only reuse ScanoutBuffers with compatible modifiers (patchset #4 id:60001 of https://codereview.chromium.org/2919533003/ )
>
> Reason for revert:
> This breaks nyan (see crbug.com/734074) so let's revert to hopefully have a green PFQ run today.
>
> Original issue's description:
> > ozone/drm: Only reuse ScanoutBuffers with compatible modifiers
> >
> > Now that we have modifiers, we need to make sure that not just the
> > format, but also the format modifer is compatible with the
> > HardwareDisplayController where we want to use the buffer.
> >
> > BUG=b/38217940
> > TEST=Plug in external monitor that supports same resolution as internal
> > panel with mirror mode enabled
> >
> > Review-Url: https://codereview.chromium.org/2919533003
> > Cr-Commit-Position: refs/heads/master@{#479813}
> > Committed: https://chromium.googlesource.com/chromium/src/+/9947afe90294ff5ad902e2cc02f8064506f7f0a4
>
> TBR=dcastagna@chromium.org,reveman@chromium.org,dnicoara@chromium.org,hoegsberg@google.com,hoegsberg@chromium.org
> # Not skipping CQ checks because original CL landed more than 1 days ago.
> BUG=b/38217940
>
> Review-Url: https://codereview.chromium.org/2948513003
> Cr-Commit-Position: refs/heads/master@{#480683}
> Committed: https://chromium.googlesource.com/chromium/src/+/d4a230f60cc702283992d180a1cd77109d8459bd
TBR=dcastagna@chromium.org,reveman@chromium.org,dnicoara@chromium.org,hoegsberg@chromium.org,marcheu@chromium.org,marcheu@google.com
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=b/38217940
Patch Set 1 #Messages
Total messages: 5 (2 generated)
|