Open Bug 936650 Opened 11 years ago Updated 2 years ago

stretch_copybit : Unsupported Pixel format 1

Categories

(Core :: Graphics, defect)

x86
macOS
defect

Tracking

()

People

(Reporter: gal, Unassigned)

Details

Attachments

(1 file)

This is running 1.2 on helix. E/copybit ( 775): stretch_copybit : Unsupported Pixel format 1 E/msm7627a.hwcomposer( 775): drawLayerUsingCopybit: copybit stretch failed E/HWComposer( 775): H/W Composition failed E/copybit ( 775): stretch_copybit : Unsupported Pixel format 1 E/msm7627a.hwcomposer( 775): drawLayerUsingCopybit: copybit stretch failed E/HWComposer( 775): H/W Composition failed E/copybit ( 775): stretch_copybit : Unsupported Pixel format 1 E/msm7627a.hwcomposer( 775): drawLayerUsingCopybit: copybit stretch failed E/HWComposer( 775): H/W Composition failed E/copybit ( 775): stretch_copybit : Unsupported Pixel format 1 E/msm7627a.hwcomposer( 775): drawLayerUsingCopybit: copybit stretch failed E/HWComposer( 775): H/W Composition failed E/copybit ( 775): stretch_copybit : Unsupported Pixel format 1 E/msm7627a.hwcomposer( 775): drawLayerUsingCopybit: copybit stretch failed E/HWComposer( 775): H/W Composition failed E/copybit ( 775): stretch_copybit : Unsupported Pixel format 1 E/msm7627a.hwcomposer( 775): drawLayerUsingCopybit: copybit stretch failed E/HWComposer( 775): H/W Composition failed E/copybit ( 775): stretch_copybit : Unsupported Pixel format 1 E/msm7627a.hwcomposer( 775): drawLayerUsingCopybit: copybit stretch failed E/HWComposer( 775): H/W Composition failed
Instead of falling back onto the GPU we actually end up not composting the frame, which is bad.
HAL_PIXEL_FORMAT_RGBA_8888 = 1, if (ctx->mAlpha < 255) { switch (src->format) { // we don't support plane alpha with RGBA formats case HAL_PIXEL_FORMAT_RGBA_8888: case HAL_PIXEL_FORMAT_BGRA_8888: case HAL_PIXEL_FORMAT_RGBA_5551: case HAL_PIXEL_FORMAT_RGBA_4444: ALOGE ("%s : Unsupported Pixel format %d", __FUNCTION__, src->format); return -EINVAL; } } So ctx->mAlpha is < 255.
Its initialized to MDP_ALPHA_NOP, so that looks ok. 55 #define MDP_ALPHA_NOP 0xff There are two places that set mAlphas, but both set it to 255 in the original silicon vendor source. I assume the OEM does something bad here in their drivers. copybit->set_parameter(copybit, COPYBIT_PLANE_ALPHA, 255);
Ok, we definitely don't check for set() to fail.
Assignee: nobody → gal
That code is for JB only. That having said we shouldn't have the define there.
I'm assuming this is wait and see, no action on our side.

The bug assignee didn't login in Bugzilla in the last 7 months, so the assignee is being reset.

Assignee: gal → nobody
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: