[kio-extras] [Bug 420173] New: thumbnail.so eats tons of memory with large GeoTiff

2020-04-16 Thread Sandro Mani
https://bugs.kde.org/show_bug.cgi?id=420173

Bug ID: 420173
   Summary: thumbnail.so eats tons of memory with large GeoTiff
   Product: kio-extras
   Version: 19.12
  Platform: Other
OS: Linux
Status: REPORTED
  Severity: normal
  Priority: NOR
 Component: Thumbnails and previews
  Assignee: plasma-b...@kde.org
  Reporter: manisan...@gmail.com
  Target Milestone: ---

Created attachment 127598
  --> https://bugs.kde.org/attachment.cgi?id=127598&action=edit
Patch

SUMMARY
Put any huge image in a folder (im my case a 111320 x 66792 GeoTiff), launch
dolphin and browse to that folder or its parent, and watch thumbnail.so eat up
memory.

The cause is that QImageReader in ImageCreator::create (imagecreator.cpp) will
read the entire raw image, in this needing I suppose like 27GB of memory.

The easiest remedy would be to just limit the size of the images
ImageCreator::create will actually process, like to max 10240 in either
dimension. Proposed trivial patch attached.

Note: QImageReader::setScaledSize reportedly works for some formats to directly
read a downscaled image, but it does not seem to be the case with the qtiff
driver.


SOFTWARE/OS VERSIONS
kio-extras-19.12.3
Fedora Rawhide
Qt 5.14.2

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 384045] Task switcher does not render with compositing enabled

2017-08-28 Thread Sandro Mani
https://bugs.kde.org/show_bug.cgi?id=384045

--- Comment #4 from Sandro Mani  ---
For reference: https://bugreports.qt.io/browse/QTBUG-62837

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kcoreaddons] [Bug 384037] DesktopFileParser::convert: incorrect service file name?

2017-08-27 Thread Sandro Mani
https://bugs.kde.org/show_bug.cgi?id=384037

--- Comment #3 from Sandro Mani  ---
For clarity, I'm not sure at this point what possible consequences the failure
to load these service files has. But I don't think it's a packaging issue,
since the kwin.spec [1] does not do anything particular to those files, and
also since google returns many of results for that error message, originating
from a variety of distros.

Again for clarity, the issue for this bug report is simply  whether service
file names should be i.e.

kwineffect.desktop

or

kwin-effect.desktop.

In the latter case, the code in the DesktopFileParser::convert should possibly
be

QString fileName = service.toLower().replace(QLatin1Char('/'),
QLatin1String(""))+QStringLiteral(".desktop");

Sorry for the confusion, I had initially misdiagnosed the results of my
debugging session.

[1] https://src.fedoraproject.org/rpms/kwin/blob/master/f/kwin.spec

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 384045] Task switcher does not render with compositing enabled

2017-08-26 Thread Sandro Mani
https://bugs.kde.org/show_bug.cgi?id=384045

--- Comment #2 from Sandro Mani  ---
Is it know upstream?

-- 
You are receiving this mail because:
You are watching all bug changes.

[kwin] [Bug 384045] New: Task switcher does not render with compositing enabled

2017-08-26 Thread Sandro Mani
https://bugs.kde.org/show_bug.cgi?id=384045

Bug ID: 384045
   Summary: Task switcher does not render with compositing enabled
   Product: kwin
   Version: 5.10.4
  Platform: Other
OS: Linux
Status: UNCONFIRMED
  Severity: normal
  Priority: NOR
 Component: tabbox
  Assignee: kwin-bugs-n...@kde.org
  Reporter: manisan...@gmail.com
  Target Milestone: ---

For the last couple of months I've had the issue that the ALT+Tab switcher
fails to render with compositing enabled, see [1]. I've finally invested some
time to attempt to debug this, but have not come very far.

What I found out:

- When triggering the task-switcher, kwin reports
QGLXContext: Failed to create dummy context

- Looking at the qt-sources, I found that QT_OPENGL_NO_SANITY_CHECK disables
the sanity checking that triggers the dummy context creation.

- So trying with QT_OPENGL_NO_SANITY_CHECK=1 kwin_x11, triggering the
task-switcher I get loads of
   QOpenGLShader::compile(Fragment): 0:1(10): error: GLSL 1.50 is not
supported. Supported versions are: 1.10, 1.20, 1.30, 1.00 ES, 3.00 ES, 3.10 ES,
and 3.20 ES

Somehow it looks like kwin is not using a core GL context with sufficiently
high version? My graphics setup is

Vendor: Intel Open Source Technology Center (0x8086)
Device: Mesa DRI Intel(R) HD Graphics 530 (Skylake GT2)  (0x191b)
Version: 17.2.0
Accelerated: yes
Video memory: 3072MB
Unified memory: yes
Preferred profile: core (0x1)
Max core profile version: 4.5
Max compat profile version: 3.0
Max GLES1 profile version: 1.1
Max GLES[23] profile version: 3.2


[1] https://smani.fedorapeople.org/taskswitcher.jpg 


Using kwin-5.10.5-1.fc28.x86_64 on Fedora Rawhide

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kcoreaddons] [Bug 384037] DesktopFileParser::convert: incorrect service file name?

2017-08-26 Thread Sandro Mani
https://bugs.kde.org/show_bug.cgi?id=384037

--- Comment #1 from Sandro Mani  ---
Hmm actually it does not fix the broken rendering of the taskswitcher (I didn't
notice that compositing got disabled while debugging), but I think the rest of
the issue is still a valid case.

-- 
You are receiving this mail because:
You are watching all bug changes.

[frameworks-kcoreaddons] [Bug 384037] New: DesktopFileParser::convert: incorrect service file name?

2017-08-26 Thread Sandro Mani
https://bugs.kde.org/show_bug.cgi?id=384037

Bug ID: 384037
   Summary: DesktopFileParser::convert: incorrect service file
name?
   Product: frameworks-kcoreaddons
   Version: 5.37.0
  Platform: Other
OS: Linux
Status: UNCONFIRMED
  Severity: normal
  Priority: NOR
 Component: general
  Assignee: mp...@kde.org
  Reporter: manisan...@gmail.com
CC: kdelibs-b...@kde.org
  Target Milestone: ---

Note: I'm not sure whether this is a bug in kf5-kcoreaddons or kwin

DesktopFileParser::convert has the following code:

// some .desktop files still use the legacy ServiceTypes= key
QString fileName = service.toLower().replace(QLatin1Char('/'),
QLatin1Char('-'))+QStringLiteral(".desktop");

However, kwin (and others, judging from ls /usr/share/kservicetypes5/), don't
have a dash in their service desktop filenames, i.e.:

kwineffect.desktop
kwinscript.desktop
kwinwindowswitcher.desktop
etc

Currently, DesktopFileParser::convert makes kwin look for i.e.
kwin-windowswitche.desktop and so on, which results in errors like

kf5.kcoreaddons.desktopparser: Could not locate service type file
kservicetypes5/kwin-effect.desktop, tried ("/home/sandro/.local/share",
"/usr/share/kde-settings/kde-profile/default/share", "/usr/local/share",
"/usr/share")

and some functionality not working (in this case, the task switcher).

I suppose either DesktopFileParser::convert generates an incorrect filename, or
kwin names its service files incorrectly (see
https://smani.fedorapeople.org/taskswitcher.jpg).

Using
- kf5-kcoreaddons-5.37.0-1.fc28.x86_64
- kwin-5.10.5-1.fc28.x86_64
on Fedora rawhide.

-- 
You are receiving this mail because:
You are watching all bug changes.