From kde-panel-devel Tue Jul 29 16:15:10 2014 From: "Aleix Pol Gonzalez" Date: Tue, 29 Jul 2014 16:15:10 +0000 To: kde-panel-devel Subject: Re: Review Request 119524: Force OpenGLES if on Qt 5.4 Message-Id: <20140729161510.15443.98569 () probe ! kde ! org> X-MARC-Message: https://marc.info/?l=kde-panel-devel&m=140665054312041 MIME-Version: 1 Content-Type: multipart/mixed; boundary="--===============5476040592720306955==" --===============5476040592720306955== Content-Type: multipart/alternative; boundary="===============6430345317542314139==" --===============6430345317542314139== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/ ----------------------------------------------------------- (Updated July 29, 2014, 4:15 p.m.) Status ------ This change has been marked as submitted. Review request for Plasma. Repository: plasma-workspace Description ------- When trying to reduce memory usage in plasma shell, we realized that one of the reasons we got such big memory footprint was actually the intel driver (on intel systems, that is). After some investigation, David found out we were going through some memory-consuming path [1]. A way to workaround it is by using only OpenGLES. Everything still works here after this patch, so it seems to be a good change to get in. This change needs Qt 5.4, to get the new API that lets us enforce a QSurfaceFormat, hence having it ifdef'd. I'm quite unaware of problems we might find. Knowing we probably want to work in different embedded devices, suggests that OpenGLES on all platforms sounds like a safe bet. [1] http://comments.gmane.org/gmane.comp.video.mesa3d.devel/60848 Diffs ----- shell/main.cpp e34578d Diff: https://git.reviewboard.kde.org/r/119524/diff/ Testing ------- I'm using it now, without visible problems. Now we see no trace of i915 in the massif reports. File Attachments ---------------- before https://git.reviewboard.kde.org/media/uploaded/files/2014/07/28/5a6beb4f-24c0-4cee-a7a2-038385e35119__plasma-massif-opengl.png after https://git.reviewboard.kde.org/media/uploaded/files/2014/07/28/fc301ff5-cb1d-4654-a57f-82990220c8e3__plasma-massif-opengles.png Thanks, Aleix Pol Gonzalez --===============6430345317542314139== MIME-Version: 1.0 Content-Type: text/html; charset="utf-8" Content-Transfer-Encoding: 7bit

This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/119524/

This change has been marked as submitted.

Review request for Plasma.

By Aleix Pol Gonzalez.

Updated July 29, 2014, 4:15 p.m.

Repository: plasma-workspace

Description

When trying to reduce memory usage in plasma shell, we realized that one of the reasons we got such big memory footprint was actually the intel driver (on intel systems, that is). After some investigation, David found out we were going through some memory-consuming path [1]. A way to workaround it is by using only OpenGLES. Everything still works here after this patch, so it seems to be a good change to get in.

This change needs Qt 5.4, to get the new API that lets us enforce a QSurfaceFormat, hence having it ifdef'd.
I'm quite unaware of problems we might find. Knowing we probably want to work in different embedded devices, suggests that OpenGLES on all platforms sounds like a safe bet.

[1] http://comments.gmane.org/gmane.comp.video.mesa3d.devel/60848

Testing

I'm using it now, without visible problems. Now we see no trace of i915 in the massif reports.

Diffs

shell/main.cpp (e34578d)

File Attachments

--===============6430345317542314139==-- --===============5476040592720306955== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel --===============5476040592720306955==--