kevin: dragging around play store chews through memory |
|||||
Issue descriptionI'm on root filesystem: 9538.0.0 (Official Build) dev-channel kevin test ...and with kernel: 039e11cfc8fc (m/master, cros/chromeos-4.4) CHROMIUM: arm64: dts: rockchip: Change DDR part for bob rev4+ I do the following: 1. Login to the device 2. Open the "Play Store" 3. Drag the window around a bit Within 15 seconds the system grinds to a halt. If you happen to open up "top" you can see the "free" memory in the system drop down amazingly quickly. === I'll try to bisect. No idea if this is the kernel or root filesystem. Note that I still reproduce when I go back to before the new binder patches, AKA: 6dee22a703a1 CHROMIUM: bluetooth: enlarge supervision timeout to 2 secs
,
May 12 2017
,
May 12 2017
Adding intel, this is one of their backports
,
May 12 2017
This works for me to "fix" the problem: $ git diff d2cc64e3f3be..fbcb3ab08e68 > allofit.patch $ patch -p1 -R < allofit.patch
,
May 12 2017
If this is blocking anyone else's development (like it was mine), you can just grab: https://chromium-review.googlesource.com/505152 TEST-ONLY: Revert all those pesky DRM backports ;-)
,
May 12 2017
I'm guessing this will be fixed with: commit 1e8985a835fdffc99f823c25ace54833656c505a Author: Tomi Valkeinen <tomi.valkeinen@ti.com> Date: Tue May 31 15:03:18 2016 +0300 drm: fix fb refcount issue with atomic modesetting After commit 027b3f8ba9277410c3191d72d1ed2c6146d8a668 ("drm/modes: stop handling framebuffer special") extra fb refs are left around when doing atomic modesetting. The problem is that the new drm_property_change_valid_get() does not return anything in the '**ref' parameter, which causes drm_property_change_valid_put() to do nothing. For some reason this doesn't cause problems with legacy API. Also, previously the code only set the 'ref' variable for fbs, with this patch the 'ref' is set for all objects. Fixes: 027b3f8ba927 ("drm/modes: stop handling framebuffer special") Signed-off-by: Tomi Valkeinen <tomi.valkeinen@ti.com> Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch> Signed-off-by: Dave Airlie <airlied@redhat.com> It's been backported here: https://chromium-review.googlesource.com/#/c/459990/, but hasn't made it through the CQ yet. I still have to test this hypothesis (building is slow), should have something soon.
,
May 12 2017
@6: yup, that fixes it. Thanks.
,
May 13 2017
https://chromium-review.googlesource.com/#/c/459990/ has landed, marking fixed.
,
May 16 2017
Verified with version 60.0.3101.0/9557.0.0 dev-channel kevin |
|||||
►
Sign in to add a comment |
|||||
Comment 1 by diand...@chromium.org
, May 12 2017Owner: seanpaul@chromium.org