OSDN Git Service

android-x86/external-IA-Hardware-Composer.git
6 years agoRemove the duplication of the function call
Kishore Kadiyala [Wed, 17 Jan 2018 09:02:06 +0000 (14:32 +0530)]
Remove the duplication of the function call

Process State is already initialized at beginning of the process,
and again calling the same is duplication.

Jira: None
Test: GPA launching 3rd party App shouldn't cause any SIGSEGV
      & also HWC tests should PASS.
Signed-off-by: Kishore Kadiyala <kishore.kadiyala@intel.com>
6 years agoSet scaling mode through environment variable.
Wang,Fei [Thu, 18 Jan 2018 22:25:57 +0000 (14:25 -0800)]
Set scaling mode through environment variable.

Jira: None.
Test: Build passes on Linux and Android.

Signed-off-by: Wang,Fei <fei.w.wang@intel.com>
6 years agoMark NeedsOffScreenComposition const.
Kalyan Kondapally [Tue, 16 Jan 2018 08:36:01 +0000 (00:36 -0800)]
Mark NeedsOffScreenComposition const.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoCheck source and display frame rects separately.
Kalyan Kondapally [Tue, 16 Jan 2018 08:12:58 +0000 (00:12 -0800)]
Check source and display frame rects separately.

Source rect changes are interesting when using display
scalar, if not display frame should be enough.

Jira: None.
Test: No new regressions when using 3D Composition for Mouse.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoRemove separate damage tracking in NativeSurface.
Kalyan Kondapally [Tue, 16 Jan 2018 00:05:33 +0000 (16:05 -0800)]
Remove separate damage tracking in NativeSurface.

We do this tracking in hwclayer now. Instead of seprately
tracking another damage in NativeSurface, we just let it
use the newly calculated one. We make sure any new damage
is propogated to all offscreen surfaces. Layers with a
transform are forced to use their display frame rect as
their damage rect.

Jira: None.
Test: Sliders still update correctly. Moving mouse when rotated
      in 90 degrees doesn't leave any garbage.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoTake Surface Damage into account before reseting kLayerContentChanged.
Kalyan Kondapally [Mon, 15 Jan 2018 08:35:44 +0000 (00:35 -0800)]
Take Surface Damage into account before reseting kLayerContentChanged.

If SurfaceDamage is not empty we need to render the damaged
content in this frame. We didn't check for this earlier.

Jira: None.
Test: No partial rendered frames are shown when launching apps from
      launcher.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoMinimize flickers when layer is replaced with another.
Kalyan Kondapally [Sun, 14 Jan 2018 21:05:40 +0000 (13:05 -0800)]
Minimize flickers when layer is replaced with another.

Let's make sure Surface Damage of current layer includes the
display frame of the replaced layer.

Jira: None.
Test: Flicker is minimized when views having VKB are launched.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoEnsure Media surface state is updated correctly.
Kalyan Kondapally [Sun, 14 Jan 2018 20:29:46 +0000 (12:29 -0800)]
Ensure Media surface state is updated correctly.

GL Backend reset's clear surface state correctly, which we
relied in DisplayQueue to determine if we need to refresh
the surface in case of partial update. VA backend never
did that resulting in display frame never getting updated
after set once. Now, we ensure we refresh surfaces in case
we need partial clear.

Jira: None.
Test: Video layer scales correctly when changing between full
      screen and non-fullscreen mode.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoSkip Revalidation in case layer buffer has changed.
Kalyan Kondapally [Sun, 14 Jan 2018 19:03:24 +0000 (11:03 -0800)]
Skip Revalidation in case layer buffer has changed.

Revalidation has no value in case the layer is going
through 3D Composition pass, let's skip it in this
case.

Jira: None.
Test: No new test commit failures.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoMove all display transform checks to PlaneState.
Kalyan Kondapally [Sun, 14 Jan 2018 11:23:23 +0000 (03:23 -0800)]
Move all display transform checks to PlaneState.

This should given us ability to check if rotation can be
supported by individual layers or offscreen surface of an
plane. This should ensure we are able to validate the
support propertly before trying to use the display capabilities
or fall back to handle the rotation with 3D Compositor.

Jira: None.
Test: 180 rotation works as before.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoUse plane for 180 rotation.
Kalyan Kondapally [Thu, 11 Jan 2018 10:10:47 +0000 (02:10 -0800)]
Use plane for 180 rotation.

We can support this directly using Display Planes.
Ideally, we need to check first if this is supported or not
before deciding to use between 3D or Display for rotation.

Jira: None.
Test: All layers are rotated correctly.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoEnsure we update Video layer when effects state change.
Kalyan Kondapally [Wed, 10 Jan 2018 21:30:50 +0000 (13:30 -0800)]
Ensure we update Video layer when effects state change.

Jira: None.
Test: Any change to Video effects support is taken into
      account immediately.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAdd deinterlace on VA module.
Wang,Fei [Tue, 9 Jan 2018 07:24:50 +0000 (15:24 +0800)]
Add deinterlace on VA module.

Jira: None.
Test: Build passes on Linux and Android.

Signed-off-by: Wang,Fei <fei.w.wang@intel.com>
6 years agoAdd a threshold before we use SurfaceDamage.
Kalyan Kondapally [Tue, 9 Jan 2018 10:08:03 +0000 (02:08 -0800)]
Add a threshold before we use SurfaceDamage.

Layers having displayframewidth and height less than 500, seem
to report wrong surface damage for some reason. This patch adds
a threshold of 1000 before we take advantage of SurfaceDamage.

Jira: None.
Test: Drop down menu's still work fine.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoDon't assume plane always needs rendering.
Kalyan Kondapally [Tue, 9 Jan 2018 09:19:27 +0000 (01:19 -0800)]
Don't assume plane always needs rendering.

We should check if the plane surface is being re-cycled and set
our rendering state accordingly instead of assuming it to be
always true.

Jira: None.
Test: Cursor visibility change doesn't force us to render the full
      content of all planes.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoDont refresh surfaces if not needed.
Kalyan Kondapally [Tue, 9 Jan 2018 10:11:07 +0000 (02:11 -0800)]
Dont refresh surfaces if not needed.

We should update the surfaces only if something has really
changed in the plane state. However, we ended up setting
the full damage whenever RefreshSurfacesIfNeeded was called.
This patch fixes it.

Jira: None.
Test: No rendering artifacts seen with Linux apps.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoApply video effect immediately for current frame
Xiaosong Wei [Mon, 8 Jan 2018 08:08:22 +0000 (16:08 +0800)]
Apply video effect immediately for current frame

The current video effect settings should be applied immediately
for the current frame

Jira: None
Test: video effect works as expected

Change-Id: I1abe484c78f3bc616587bcbd32760bf7c1e896c9
Tracked-On:
Signed-off-by: Xiaosong Wei <xiaosong.wei@intel.com>
6 years agoFix width/height setting in vaCreateSurfaces.
Lin Johnson [Mon, 8 Jan 2018 10:00:11 +0000 (18:00 +0800)]
Fix width/height setting in vaCreateSurfaces.

Tests:video playback sucessful
Jira:None
Signed-off-by: Lin Johnson <johnson.lin@intel.com>
6 years agoFix flicker when partial clears are used.
Kalyan Kondapally [Mon, 8 Jan 2018 10:41:13 +0000 (02:41 -0800)]
Fix flicker when partial clears are used.

We updated this state only for current offscreen surface but
not for all surfaces. This fixes flicker seen with some
apps when we clear only the damaged rect.

Jira: None.
Test: No flicker seen when partial clears are used.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoReset Validationstate when adding layers.
Kalyan Kondapally [Mon, 8 Jan 2018 05:45:31 +0000 (21:45 -0800)]
Reset Validationstate when adding layers.

After removing layers, we might have decided to re-check
the plane for direct scanout. However, we can be adding layers
later, let's reset this state when we do so.

Jira: None.
Test: Bottom layers don't disappear when navigating back to
      home screen. We dont see the following warning
      when SURFACE_PLANE_LAYER_MAP_TRACING is 1:
      "Plane has more than one layer associated when its type is
        kScanout. This needs to be fixed."

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoMake sure we check SurfaceDamage and rect when clear_surface is false.
Kalyan Kondapally [Mon, 8 Jan 2018 04:48:32 +0000 (20:48 -0800)]
Make sure we check SurfaceDamage and rect when clear_surface is false.

We where checking if remove_index != -1, instead we should check
if clear_surface is true.This flag is set only for planes which
have been reset.

Jira: None.
Test: No regressions on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoDont reset Damage more than we should do.
Kalyan Kondapally [Mon, 8 Jan 2018 04:14:02 +0000 (20:14 -0800)]
Dont reset Damage more than we should do.

Jira: None.
Test: Popups showup fine on Android.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAdd helper to reset rect.
Kalyan Kondapally [Mon, 8 Jan 2018 01:07:31 +0000 (17:07 -0800)]
Add helper to reset rect.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoClip regions to Damage Rect.
Kalyan Kondapally [Sun, 7 Jan 2018 02:18:07 +0000 (18:18 -0800)]
Clip regions to Damage Rect.

While calculating the regions in disjoint_layers, lets clip
them to damage rect as this is the region which really needs
to be rendered again.

Jira: None.
Test: Dialog boxes, sliders all update correctly.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoRemove redundant clear surface checks.
Kalyan Kondapally [Sun, 7 Jan 2018 02:09:45 +0000 (18:09 -0800)]
Remove redundant clear surface checks.

Jira: None.
Test: Build passes on Android.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAvoid re-compositing planes in case removed layer was scanned out.
Kalyan Kondapally [Sun, 7 Jan 2018 01:33:58 +0000 (17:33 -0800)]
Avoid re-compositing planes in case removed layer was scanned out.

Jira: None.
Test: Popups and switching views doesn't show any rendering issues.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoMake sure we set correct plane transform when display is rotated 180.
Kalyan Kondapally [Sun, 7 Jan 2018 00:34:07 +0000 (16:34 -0800)]
Make sure we set correct plane transform when display is rotated 180.

Jira: None.
Test: Layer content is rotated 180 when
      PHYSICAL_DISPLAY_ROTATION="0:2" is set in hwc_display.ini file.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoTake Damage rect into account in disjoint_layers.
Kalyan Kondapally [Sat, 6 Jan 2018 23:24:02 +0000 (15:24 -0800)]
Take Damage rect into account in disjoint_layers.

We want to only render layers which are impacted by the damaged
region.

Jira: None.
Test: No new rendering artifacts on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoReset Composition region as needed.
Kalyan Kondapally [Sat, 6 Jan 2018 21:24:47 +0000 (13:24 -0800)]
Reset Composition region as needed.

We need to do this every time we clear the surface and surface
damage has changed.

Jira: None.
Test: Popups are rendered correctly when moving between different
      views

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoRemove include.
Kalyan Kondapally [Thu, 4 Jan 2018 16:00:29 +0000 (08:00 -0800)]
Remove include.

This is not present anymore.

Jira: None.
Test: Tests build on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAlways reset current damage to be same as surface_damage.
Kalyan Kondapally [Thu, 4 Jan 2018 04:51:25 +0000 (20:51 -0800)]
Always reset current damage to be same as surface_damage.

Jira: None.
Test: No new graphics regressions on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoFix MosaicDisplay.
Kalyan Kondapally [Wed, 3 Jan 2018 15:55:05 +0000 (07:55 -0800)]
Fix MosaicDisplay.

We now stricly check our state before doing any updates.
In Mosaic Case, same layer is used for all displays by just
traslating the source crop and display frame. Let's make sure
we don't invalidate the state till all displays are done.

Jira: None.
Test: Mosaic display feature works with 2 displays connected.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoPrefer display with highest refresh rate.
Kalyan Kondapally [Wed, 3 Jan 2018 15:21:00 +0000 (07:21 -0800)]
Prefer display with highest refresh rate.

In case of Mosaic, we alway wait on the last connected
display update to finish before requesting a new update.
Instead let's use the fence from display with higest refresh
rate.

Jira: None.
Test: Mosaic mode works as now.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoTrack render state correctly.
Kalyan Kondapally [Wed, 3 Jan 2018 13:57:30 +0000 (05:57 -0800)]
Track render state correctly.

We need to render layers in case plane has more than one layer
/needs gpu composition.

Jira: None.
Test: Content is correct after rendering idle frame on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoPostpone re-validation when only removing layers.
Kalyan Kondapally [Wed, 3 Jan 2018 13:48:25 +0000 (05:48 -0800)]
Postpone re-validation when only removing layers.

In this case we shouldn't be showing any new content but only
removing old layers. We can avoid full revalidation.

Jira: None.
Test: We dont trigget a full revalidation in case cursor layer
      becomes invisible.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoFix indentation.
Kalyan Kondapally [Wed, 3 Jan 2018 13:46:39 +0000 (05:46 -0800)]
Fix indentation.

Jira: None.
Test: Build passes on Android.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAdd more traces.
Kalyan Kondapally [Wed, 3 Jan 2018 13:43:47 +0000 (05:43 -0800)]
Add more traces.

Jira: None.
Test: Build passes on Android.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoFix cursor layer check in DrmPlane.
Kalyan Kondapally [Wed, 3 Jan 2018 12:24:15 +0000 (04:24 -0800)]
Fix cursor layer check in DrmPlane.

We might be using Overlay to show cursor. Let's check
if the layer is a cursor layer instead of assuming we will
use only cursor plane for cursor layer.

Jira: None.
Test: Cursor layer size is correct ir-respective of the plane
      in use.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoDon't ignore alpha when using overlays.
Kalyan Kondapally [Wed, 3 Jan 2018 12:22:37 +0000 (04:22 -0800)]
Don't ignore alpha when using overlays.

The preffered format order always had x* components as first.
We want to prefer these only in case of primary.

Jira: None.
Test: Right format is used with overlays on APL.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoLet's make sure we use single plane when in idle mode.
Kalyan Kondapally [Wed, 3 Jan 2018 12:20:59 +0000 (04:20 -0800)]
Let's make sure we use single plane when in idle mode.

We ignored this state and continued to use multiple planes.

Jira: None.
Test: Only single plane is used when we are rendering an idle
      frame.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoEnable Damage support.
Kalyan Kondapally [Tue, 2 Jan 2018 15:36:23 +0000 (07:36 -0800)]
Enable Damage support.

We now keep track of both surface damage of buffer and any
damage incurred due to geometry changes i.e. Transform change,
alpha value change etc. We rely on this and not just buffer
damage. With this we are able to now do partial updates, doing
full, partial or no clear as needed.

Jira: None.
Test: Sliders work fine, Mouse GPU composition doesn't show any
      artifacts.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoEnsure we reset composition region cache when we add new layers.
Kalyan Kondapally [Tue, 2 Jan 2018 14:19:45 +0000 (06:19 -0800)]
Ensure we reset composition region cache when we add new layers.

Jira: None.
Test: Check that we re-calculate the region when ever new layers
      are added.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoEnsure we clear surface when display rect has changed.
Kalyan Kondapally [Tue, 2 Jan 2018 11:28:09 +0000 (03:28 -0800)]
Ensure we clear surface when display rect has changed.

Jira: None.
Test: Cursor doesn't render garbage when being gpu composited.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoEnsure we swap surface as needed.
Kalyan Kondapally [Tue, 2 Jan 2018 09:22:11 +0000 (01:22 -0800)]
Ensure we swap surface as needed.

We should do this when we add new layers.

Jira: None.
Test: No new regressions on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoFix Logical Display.
Kalyan Kondapally [Tue, 2 Jan 2018 08:07:30 +0000 (00:07 -0800)]
Fix Logical Display.

We assume cursor layer is always the top layer. Let's make sure
this is valid in case of logicaldisplaymanager.

Jira: None.
Test: No crashes when using logical displays.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAvoid redundant test commit.
Kalyan Kondapally [Mon, 1 Jan 2018 00:01:56 +0000 (16:01 -0800)]
Avoid redundant test commit.

We now track if validatelayers did a test commit with current
combination of layers. In this case we can avoid the same in
ReValidatePlanes.

Jira: None.
Test: No commit failures seen on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoFix commit failure when rotating the screen.
Kalyan Kondapally [Sun, 31 Dec 2017 13:31:09 +0000 (05:31 -0800)]
Fix commit failure when rotating the screen.

Let's ensure we do a test commit when layers are added, even
if they are for last plane. DisplayQueue assumed PlaneManager
handled this even though it didn't. Let's track this
information and pass to DisplayQueue.

Jira: None.
Test: No commit failures while rotating the screen to 90 degress.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoMinor cleanup.
Kalyan Kondapally [Sun, 31 Dec 2017 12:02:59 +0000 (04:02 -0800)]
Minor cleanup.

Only call SetLayerComposition from RenderState. This is when
we know that the layer is actually going through GPU composition.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAdd support to track when surfaces need to be refreshed.
Kalyan Kondapally [Sun, 31 Dec 2017 08:28:22 +0000 (00:28 -0800)]
Add support to track when surfaces need to be refreshed.

We can avoid redundant validation of NativeSurface in case
nothing has actually changed. Let's keep a track of this
without DisplayPlaneState.

Jira: None.
Test: No new regressions on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoRemove UsePlaneScalar from OverlayLayer.
Kalyan Kondapally [Sun, 31 Dec 2017 08:05:34 +0000 (00:05 -0800)]
Remove UsePlaneScalar from OverlayLayer.

Same can be queried from DisplayPlaneState.

Jira: None.
Test: Build passes on Android.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoCombine all incremental plane checks to one.
Kalyan Kondapally [Sun, 31 Dec 2017 01:20:27 +0000 (17:20 -0800)]
Combine all incremental plane checks to one.

We currently do checks whenever a layer needs to be revalidated
and also whenever a particular plane needs revalidation. Let's
combine all these checks into one instead of splitting them up.

Jira: None.
Test: Layer scaling works as now and dont see any commit failures.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoLet's ensure we test layers as rects change.
Kalyan Kondapally [Sat, 30 Dec 2017 23:19:58 +0000 (15:19 -0800)]
Let's ensure we test layers as rects change.

As we start to enable scaling, let's make sure we do a test
commit every time display frame or source rect change
compared to previous frame.

Jira: None.
Test: Video works as now.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoTrack source correctly in displayplanestate.
Kalyan Kondapally [Sat, 30 Dec 2017 22:58:12 +0000 (14:58 -0800)]
Track source correctly in displayplanestate.

We currently use scalars only for media but in future this
could be used for any layer. Let's make sure we track this
correctly and internally decide if source_rect should remain
the union of rects of all layers or equal to display frame.
This decision depends if display scalar flag is enabled
or disabled in displayplanestate.

Jira: None.
Test: Video still uses scaling as now.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoEnsure we do a test commit in case cursor checks fail.
Kalyan Kondapally [Fri, 29 Dec 2017 19:31:42 +0000 (11:31 -0800)]
Ensure we do a test commit in case cursor checks fail.

Jira: None.
Test: No Atomic commit failures seen when rotating screen.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoRemove redundant inputs to validate layers.
Kalyan Kondapally [Fri, 29 Dec 2017 14:47:38 +0000 (06:47 -0800)]
Remove redundant inputs to validate layers.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoCache cursor test results.
Kalyan Kondapally [Fri, 29 Dec 2017 14:33:01 +0000 (06:33 -0800)]
Cache cursor test results.

Adds basic caching to save last test results for cursor layer
with a given plane. This should avoid redundant test checks
for cursor when other layers have been added or removed.

Jira: None.
Test: No test commit is requested if same transform and plane is
      used for cursor.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAvoid surfacedamage/rects calculations in case layers are removed.
Kalyan Kondapally [Fri, 29 Dec 2017 03:58:07 +0000 (19:58 -0800)]
Avoid surfacedamage/rects calculations in case layers are removed.

This is redundant as we will be clearing surface and also all
display rects would have been re-calculated when layers are
reset.

Jira: None.
Test: No new regressions on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAdd RestoreDefaultVideoColor implementation
Xiaosong Wei [Mon, 25 Dec 2017 04:44:51 +0000 (12:44 +0800)]
Add RestoreDefaultVideoColor implementation

Restore the default values for VPP and then remove the effect applied
before.

Jira: None
Test: Set/Restore color works as expected

Change-Id: Ied474e6e37124b35a80e96c5f2b9c403bb38c3a7
Tracked-On:
Signed-off-by: Xiaosong Wei <xiaosong.wei@intel.com>
6 years agoAvoid extra test commit checks.
Kalyan Kondapally [Thu, 28 Dec 2017 00:56:59 +0000 (16:56 -0800)]
Avoid extra test commit checks.

We don't need to do this if we have already done while
validating the layers. Let's keep track of it and do
final validation if still needed.

Jira: None.
Test: No commit fails on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAvoid full validation when adding new layers.
Kalyan Kondapally [Tue, 26 Dec 2017 09:49:04 +0000 (01:49 -0800)]
Avoid full validation when adding new layers.

We can avoid re-validating whole commit when a layer is added
like we do for cursor. This is useful as complexity increases
with various combinations i.e. down/upscaling.

Also added tracing support to track down any duplicate layers
being added to a plane or in case we miss to add any layer.

Jira: None.
Test: No new regressions on Android.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAvoid full validation when layers are removed.
Kalyan Kondapally [Mon, 25 Dec 2017 16:08:42 +0000 (08:08 -0800)]
Avoid full validation when layers are removed.

We now reset layers for the plane and check if this commit
passes before doing a full validation. One case which is
yet to be handled is when we have free planes and determining
if we can distrbiute layers to these planes.

Jira: None.
Test: No crashes when navigating between different views on
      Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoRevert "Clip Viewport to DisplayFrame bounds when not using Scalar."
Kalyan Kondapally [Sun, 24 Dec 2017 17:08:42 +0000 (09:08 -0800)]
Revert "Clip Viewport to DisplayFrame bounds when not using Scalar."

Fixes bugs related to popups being displayed in wrong position.

This reverts commit 644a9828859793da2b39689caa004985cfc5d47e.

6 years agoRevert "Always use source crop to set viewport."
Kalyan Kondapally [Sun, 24 Dec 2017 17:07:56 +0000 (09:07 -0800)]
Revert "Always use source crop to set viewport."

This reverts commit e937df968d2077a98a8d0842e53021aa5df224a4.

6 years agoRemove redundant checks in overlaylayer.
Kalyan Kondapally [Sat, 23 Dec 2017 10:36:46 +0000 (02:36 -0800)]
Remove redundant checks in overlaylayer.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoDon't track Primary plane separately.
Kalyan Kondapally [Sat, 23 Dec 2017 08:13:46 +0000 (00:13 -0800)]
Don't track Primary plane separately.

This should simplify logic when validating layers. Also,
these changes fix case where we added cursor layer to an
overlay and also to the cursor plane separately.

Jira: None.
Test: No new regressions on Android.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoCache DisplayPlaneState.
Kalyan Kondapally [Sat, 23 Dec 2017 03:36:12 +0000 (19:36 -0800)]
Cache DisplayPlaneState.

This patch adds support to cache relevant private data, instead
of having to manually track it for every frame. This simplifies
lot of things like keeping track of any new surfaces when commit
has failed for that frame, display rect changes etc. Also, this
avoids redundant copies of plane state every frame.

Jira: None.
Test: No flicker seen while moving between different views.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoRework DisplayPlaneState API.
Kalyan Kondapally [Sat, 23 Dec 2017 02:22:36 +0000 (18:22 -0800)]
Rework DisplayPlaneState API.

This is to make sure they align with rest of the classes in
code base.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAdd package config file
Harish Krupo [Tue, 5 Dec 2017 12:45:08 +0000 (18:15 +0530)]
Add package config file

Add package config file to export the necessary flags.

Signed-off-by: Harish Krupo <harish.krupo.kps@intel.com>
6 years agoMakefile: Remove unneeded LDADDs
Harish Krupo [Sun, 26 Nov 2017 00:11:43 +0000 (05:41 +0530)]
Makefile: Remove unneeded LDADDs

The LDADD for libhwcomposer_{common,wsi} is not needed as it is
already linked into libhwcomposer.so in the linker flags as whole
archives. Adding this causes "multiple definition" errors during
linking.

Signed-off-by: Harish Krupo <harish.krupo.kps@intel.com>
6 years agoifdef guard for GBM_BO_USE_CAMERA_{WRITE,READ}
Harish Krupo [Mon, 20 Nov 2017 13:26:02 +0000 (18:56 +0530)]
ifdef guard for GBM_BO_USE_CAMERA_{WRITE,READ}

This fixes compilation error when compiled using --enable-gbm
as gbm doesn't have these definitions.

Signed-off-by: Harish Krupo <harish.krupo.kps@intel.com>
6 years agotravis: Add compilation check against libgbm (mesa)
Harish Krupo [Thu, 30 Nov 2017 17:10:23 +0000 (22:40 +0530)]
travis: Add compilation check against libgbm (mesa)

Signed-off-by: Harish Krupo <harish.krupo.kps@intel.com>
6 years agoFix build with Debug logs.
Kalyan Kondapally [Fri, 22 Dec 2017 08:31:27 +0000 (00:31 -0800)]
Fix build with Debug logs.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoForce full re-validation in case last commit failed.
Kalyan Kondapally [Fri, 22 Dec 2017 08:16:53 +0000 (00:16 -0800)]
Force full re-validation in case last commit failed.

In this case their are too many combinations to take care of.
Instead, let's just force a full validation.

Jira: None.
Test: No new regressions on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAdd surface age to NativeSurface.
Kalyan Kondapally [Fri, 22 Dec 2017 08:41:00 +0000 (00:41 -0800)]
Add surface age to NativeSurface.

We need to track how many frames before a surface can be
re-cycled. Otherwise, we will end up updating the surface
while it's still on-screen. Once the age is zero it can be
re-cycled for other planes. If age is 2 or 1 it means we need
to wait that many frames before the surface is free to be
re-used for another plane.

Jira: None.
Test: No flicker seen when navigating between views.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoDont try to use cursor plane always for non cursor layers.
Kalyan Kondapally [Fri, 22 Dec 2017 03:03:27 +0000 (19:03 -0800)]
Dont try to use cursor plane always for non cursor layers.

We should use this only when cursor is reported as an universal
plane.

Jira: None.
Test: Check the cursor plane is not used for other layers during
      test commit.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoUse consumer_usage instead of usage from gralloc side.
Kalyan Kondapally [Fri, 22 Dec 2017 02:27:22 +0000 (18:27 -0800)]
Use consumer_usage instead of usage from gralloc side.

Minigbm structure has changed and we should be using this.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAdd DRM_FORMAT_ARGB8888 checks in drmplane.
Kalyan Kondapally [Fri, 22 Dec 2017 01:35:52 +0000 (17:35 -0800)]
Add DRM_FORMAT_ARGB8888 checks in drmplane.

We support this for Cursor Plane. Also, add verbose
to print when we are using pixel_format = 0 and layer
is Video.

Jira: None.
Test: Check that we are allocating buffers of this format for
      Cursor Plane and not using 0.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoEnsure we track layer usage correctly.
Kalyan Kondapally [Fri, 22 Dec 2017 00:25:25 +0000 (16:25 -0800)]
Ensure we track layer usage correctly.

As we now can interchange between different usage and buffer's
cached, let's make sure we track this state correctly.

Jira: None.
Test: Video plays correctly in multi mode.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoPop Out of scope Cached Buffers as part of PreparePurgedResources.
Kalyan Kondapally [Thu, 21 Dec 2017 20:59:00 +0000 (12:59 -0800)]
Pop Out of scope Cached Buffers as part of PreparePurgedResources.

This would have all buffer destruction in one place and also
make sure we have last 4 frame buffer cache while current is
being populated.

Jira: None.
Test: Build passes on Android. Also, gemhandle failures are
      minimized.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoEnsure we clear surfaces when last commit fails.
Kalyan Kondapally [Thu, 21 Dec 2017 10:55:33 +0000 (02:55 -0800)]
Ensure we clear surfaces when last commit fails.

This should make sure we don't see garbage in these cases.

Jira: None.
Test: Surfaces are cleared when commit fails on Android.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAdd more tests in hwcservice_test
Xiaosong Wei [Thu, 21 Dec 2017 09:16:41 +0000 (17:16 +0800)]
Add more tests in hwcservice_test

Add more tests for setting/restoring video colors

Jira: None
Test: tests work as expected

Signed-off-by: Xiaosong Wei <xiaosong.wei@intel.com>
6 years agoNo need to delete fb if we already have one.
Lin Johnson [Wed, 20 Dec 2017 09:11:56 +0000 (17:11 +0800)]
No need to delete fb if we already have one.

Jira: None
Tests: Build passes on Android.

Signed-off-by: Lin Johnson <johnson.lin@intel.com>
6 years agoRemove unused cursor related code.
Kalyan Kondapally [Thu, 21 Dec 2017 10:01:59 +0000 (02:01 -0800)]
Remove unused cursor related code.

We added code to create cursor buffers but are not really
using it. Remove this code, same time add error logs in case
we end up creating rgb buffers to store YUV data.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoRemove redundant code.
Kalyan Kondapally [Thu, 21 Dec 2017 09:06:00 +0000 (01:06 -0800)]
Remove redundant code.

We now check for preffered format from plane before
creating the buffer. We can remove these checks.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoFix error log.
Kalyan Kondapally [Thu, 21 Dec 2017 07:12:22 +0000 (23:12 -0800)]
Fix error log.

We where reporting about resources even though the draw
failed.

Jira: None.
Test: Build passes on Android and Linux.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAdd sharp adjustment on VA module.
Wang,Fei [Thu, 21 Dec 2017 05:46:30 +0000 (13:46 +0800)]
Add sharp adjustment on VA module.

Jira: None.
Test: Build passes on Linux and Android.

Signed-off-by: Wang,Fei <fei.w.wang@intel.com>
6 years agoMinimize locking while purging resources.
Kalyan Kondapally [Thu, 21 Dec 2017 00:43:18 +0000 (16:43 -0800)]
Minimize locking while purging resources.

We destroy all resources in thread handling present call
but postpone the destruction of gl, media and Native
resources to be done in compositor thread. To handle
synchronization we lock every purge call. Instead, of
this what we do now is store the request for destorying
these resources in same thread, at end of present call
we copy these requests which are then read from
compositor thread. This way we just need to have the locks
only during this copy.

Jira: None.
Test: No new regressions on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoPrint more data when GEM handle fails to close.
Kalyan Kondapally [Thu, 21 Dec 2017 00:13:30 +0000 (16:13 -0800)]
Print more data when GEM handle fails to close.

This is useful to help understand which buffers have issues.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoNo need to reset composition results in case of idle frame.
Kalyan Kondapally [Thu, 21 Dec 2017 00:01:48 +0000 (16:01 -0800)]
No need to reset composition results in case of idle frame.

If we are here than the composition should be exactly same as
previous frame.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoEnsure we mark on-screen & in-flight surface as in use always.
Kalyan Kondapally [Wed, 20 Dec 2017 23:58:30 +0000 (15:58 -0800)]
Ensure we mark on-screen & in-flight surface as in use always.

We recycle surfaces when validating new commits. We set only
the current onscreen surface as in use. However, we might have
a surface in flight to be shown during current vblank. Ensure
we mark both these surfaces in use. This was highlighted now
as we seem to fail test commits/fb creations with AB24 format
for some reason in certain cases.

Jira: None.
Test: No garbage seen when moving between different views on
      Android.
Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoTrigger a full validation in case fb creation fails.
Kalyan Kondapally [Wed, 20 Dec 2017 14:50:59 +0000 (06:50 -0800)]
Trigger a full validation in case fb creation fails.

In this case we should fallback to gpu composition and be able
to create a fb which we support. We cannot determine this without
full validation of layers. Let's force that in this case.

Jira: None.
Test: No atomic commit failues seen on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoRemove gem_handle_ as part of handle.
Kalyan Kondapally [Wed, 20 Dec 2017 12:23:40 +0000 (04:23 -0800)]
Remove gem_handle_ as part of handle.

This is already part of meta data.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoMake Initialize private in drmbuffer.
Kalyan Kondapally [Wed, 20 Dec 2017 12:15:47 +0000 (04:15 -0800)]
Make Initialize private in drmbuffer.

This is not expected to be used by anyone else.

Jira: None.
Test: Build passes on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoNo need to clear surface when rect is updated.
Kalyan Kondapally [Wed, 20 Dec 2017 10:14:33 +0000 (02:14 -0800)]
No need to clear surface when rect is updated.

As we now keep proper track of damage, display, src
rects we can avoid this completely.

Jira: None.
Test: No regressions on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoPartial revert of SufaceDamage.
Kalyan Kondapally [Wed, 20 Dec 2017 09:13:46 +0000 (01:13 -0800)]
Partial revert of SufaceDamage.

Sliders, Progress bar where not getting updated correctly.
For some reason, the damage area is always constant in these
cases. For now, we only consider when surface damage is zero,
otherwise fall back to display frame rect always.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoAlways use source crop to set viewport.
Kalyan Kondapally [Wed, 20 Dec 2017 01:56:34 +0000 (17:56 -0800)]
Always use source crop to set viewport.

Jira: None.
Test: Popups are positioned correctly.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoRemove vasurface.
Kalyan Kondapally [Wed, 20 Dec 2017 01:41:34 +0000 (17:41 -0800)]
Remove vasurface.

This is just a dummy class doing nothing. Instead just use
NativeSurface.

Jira: None.
Test: Build compiles on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoUnify resource tracking for Media and 3D buffers.
Kalyan Kondapally [Wed, 20 Dec 2017 00:39:51 +0000 (16:39 -0800)]
Unify resource tracking for Media and 3D buffers.

We want to make sure we re-use caching, resource management
for all backends. We didn't take this into use for va backend.
This patchs adds the needed support for this.

Jira: None.
Test: Video continues to work as now.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoEnsure we release all surfaces.
Kalyan Kondapally [Tue, 19 Dec 2017 09:54:29 +0000 (01:54 -0800)]
Ensure we release all surfaces.

When doing full validation, we don't mark current on-screen
surface as not in use. This is not tracked after the validation
and state is never set again for this surface to be re-cycled.
Let's make sure we track these surfaces and recycle them.

Jira: None.
Test: No new regressions on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>
6 years agoNo need to handle VideoEffect state in PlaneManager.
Kalyan Kondapally [Tue, 19 Dec 2017 08:03:56 +0000 (00:03 -0800)]
No need to handle VideoEffect state in PlaneManager.

Now, we have an helper function in Queue which can handle
this, let's use it.

Jira: None.
Test: Video plays as now on Android.

Signed-off-by: Kalyan Kondapally <kalyan.kondapally@intel.com>