[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 Maik Qualmann changed: What|Removed |Added Version Fixed In||7.2.0 Resolution|--- |FIXED Status|REPORTED|RESOLVED -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #53 from Maik Qualmann --- Git commit 46d4abd0cfcbdbdbab1f1189186e1737caec2839 by Maik Qualmann. Committed on 14/01/2021 at 06:49. Pushed by mqualmann into branch 'master'. call wait() after stop() before call start() again Otherwise a running task has not yet ended and and is deleted when the QThreadPool is started again. Related: bug 429307, bug 427380, bug 421043 M +1-0core/utilities/facemanagement/threads/facepreviewloader.cpp https://invent.kde.org/graphics/digikam/commit/46d4abd0cfcbdbdbab1f1189186e1737caec2839 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #52 from Maik Qualmann --- Git commit d62937a9e0c3a496f8ccbdbe72835d08899d6718 by Maik Qualmann. Committed on 14/01/2021 at 06:33. Pushed by mqualmann into branch 'master'. the DynamicThread should not be a QRunnable Otherwise it is a bit strange that a QRunnable started a QRunnable, only the QThreadPool should do this. Related: bug 429307, bug 427380, bug 421043 M +2-3core/libs/threads/dynamicthread.cpp M +7-3core/libs/threads/dynamicthread.h https://invent.kde.org/graphics/digikam/commit/d62937a9e0c3a496f8ccbdbe72835d08899d6718 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #51 from Maik Qualmann --- Git commit df6e4fcf0157246f6d1d646f6d768d2b27e48f89 by Maik Qualmann. Committed on 05/01/2021 at 11:42. Pushed by mqualmann into branch 'master'. this could be a cause of the crash in the preview loader Related: bug 429307, bug 427380, bug 421043 M +4-0core/libs/threadimageio/fileio/loadsavetask.cpp M +5-0core/libs/threadimageio/preview/previewtask.cpp M +2-0core/libs/threadimageio/thumb/thumbnailtask.cpp https://invent.kde.org/graphics/digikam/commit/df6e4fcf0157246f6d1d646f6d768d2b27e48f89 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #50 from Maik Qualmann --- Git commit 1fcedb1fa11d6f48c1265dc309ea56138a5c45bc by Maik Qualmann. Committed on 22/11/2020 at 21:19. Pushed by mqualmann into branch 'master'. try to fix the face detection stopping M +4-2core/utilities/maintenance/facesdetector.cpp https://invent.kde.org/graphics/digikam/commit/1fcedb1fa11d6f48c1265dc309ea56138a5c45bc -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #49 from Maik Qualmann --- Please try it again. Very rarely have I had it before that the face detection "stops" - it should be another problem. Maik -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #48 from Simon Westersund --- Created attachment 133371 --> https://bugs.kde.org/attachment.cgi?id=133371=edit Debug logs from stuck face detection in 8f1b5e900f I retested with the 8f1b5e900f commit, and now it seems like the face detection gets stuck and doesn't progress. My GDB console with debug prints enabled printed nothing for several minutes, which is definitely not normal. Digikam did not crash, it just could not progress somehow... I was able to cancel the detection by pressing the button next to the progress indicator in the bottom right corner and then exit Digikam. I have attached the last bit of debug logs, including when I interrupted the face detection after it had stalled. -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #47 from Maik Qualmann --- Git commit 006263c2beefdd97a793af8d38a38dba068566ac by Maik Qualmann. Committed on 15/11/2020 at 14:52. Pushed by mqualmann into branch 'master'. revert old change to the DynamicThread M +2-2core/libs/threads/dynamicthread.cpp https://invent.kde.org/graphics/digikam/commit/006263c2beefdd97a793af8d38a38dba068566ac -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #46 from Simon Westersund --- Hello Maik, I reproduced the problem and ran into the ASSERT breakpoint. Is there anything special you would want me to provide from gdb, or are backtraces and debug logs still enough? The short backtrace currently looks like this: > (gdb) bt > #0 0x73cbe615 in raise () at /usr/lib/libc.so.6 > #1 0x73ca7862 in abort () at /usr/lib/libc.so.6 > #2 0x7427c9ac in () at /usr/lib/libQt5Core.so.5 > #3 0x7427bd59 in qt_assert_x(char const*, char const*, char const*, > int) () at /usr/lib/libQt5Core.so.5 > #4 0x7645cbf4 in Digikam::PreviewLoadingTask::~PreviewLoadingTask() > (this=0x60f13860, __in_chrg=) at > /home/simon/Development/kde/digikam/core/libs/threadimageio/preview/previewtask.cpp:58 > #5 0x7645cc62 in Digikam::PreviewLoadingTask::~PreviewLoadingTask() > (this=0x60f13860, __in_chrg=) at > /home/simon/Development/kde/digikam/core/libs/threadimageio/preview/previewtask.cpp:59 > #6 0x76470750 in Digikam::LoadSaveThread::run() > (this=0x60e1a980) at > /home/simon/Development/kde/digikam/core/libs/threadimageio/fileio/loadsavethread.cpp:130 > #7 0x7649fb1e in Digikam::DynamicThread::Private::run() > (this=0x60df6020) at > /home/simon/Development/kde/digikam/core/libs/threads/dynamicthread.cpp:191 > #8 0x742bddc2 in () at /usr/lib/libQt5Core.so.5 > #9 0x742b9e8f in () at /usr/lib/libQt5Core.so.5 > #10 0x73e553e9 in start_thread () at /usr/lib/libpthread.so.0 > #11 0x73d81293 in clone () at /usr/lib/libc.so.6 Best regards, Simon -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #45 from Maik Qualmann --- Git commit d3cdb8c2f92612c8660ba46984da7c36f5691ba6 by Maik Qualmann. Committed on 15/11/2020 at 11:58. Pushed by mqualmann into branch 'master'. fix crash in the DatabaseWriter M +1-1core/utilities/facemanagement/workers/databasewriter.cpp https://invent.kde.org/graphics/digikam/commit/d3cdb8c2f92612c8660ba46984da7c36f5691ba6 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #44 from Maik Qualmann --- Hi Gilles, digiKam now crashes when I start it. A fresh build was created. (gdb) bt #0 0x77c98937 in Digikam::FacePipeline::Private::createThumbnailLoadThread() () at /usr/lib64/libdigikamgui.so.7.2.0 #1 0x77ca984e in Digikam::DatabaseWriter::DatabaseWriter(Digikam::FacePipeline::WriteMode, Digikam::FacePipeline::Private*) () at /usr/lib64/libdigikamgui.so.7.2.0 #2 0x77c95bf2 in Digikam::FacePipeline::plugDatabaseEditor() () at /usr/lib64/libdigikamgui.so.7.2.0 #3 0x77928599 in Digikam::DigikamItemView::DigikamItemView(QWidget*) () at /usr/lib64/libdigikamgui.so.7.2.0 #4 0x779922c8 in Digikam::StackedView::StackedView(QWidget*) () at /usr/lib64/libdigikamgui.so.7.2.0 #5 0x77982009 in Digikam::ItemIconView::ItemIconView(QWidget*, Digikam::DModelFactory*) () at /usr/lib64/libdigikamgui.so.7.2.0 #6 0x778e8c05 in Digikam::DigikamApp::setupView() () at /usr/lib64/libdigikamgui.so.7.2.0 #7 0x778db212 in Digikam::DigikamApp::DigikamApp() () at /usr/lib64/libdigikamgui.so.7.2.0 #8 0x00405de3 in main () Maik -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #43 from caulier.gil...@gmail.com --- Maik, Please, double check well my important last commit with data providers classes which are polymorphic and provide a copy constructor/operator. My fix must be safe (at least after plenty of tests, all is fine here). Gilles -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #42 from caulier.gil...@gmail.com --- Git commit 06304b820fcee383b5a747af806c82cc44655b3b by Gilles Caulier. Committed on 14/11/2020 at 22:43. Pushed by cgilles into branch 'master'. Fix Clazy report about copy constructor and copy operator used with polymorphic classes from FaceEngine. Store data provider classes instance while training faces stage as map of pointers. data are allocated with new operator. M +12 -8core/libs/facesengine/recognition/dataproviders.cpp M +11 -1core/libs/facesengine/recognition/dataproviders.h M +2-1 core/libs/facesengine/recognition/facialrecognition_wrapper_recognize.cpp M +2-2 core/libs/facesengine/recognition/recognitiontrainingprovider.cpp M +5-5core/utilities/facemanagement/workers/databasewriter.cpp M +2-2core/utilities/facemanagement/workers/databasewriter.h M +2-2core/utilities/facemanagement/workers/detectionworker.cpp M +2-2core/utilities/facemanagement/workers/detectionworker.h M +4-4core/utilities/facemanagement/workers/recognitionworker.cpp M +2-2core/utilities/facemanagement/workers/recognitionworker.h M +11 -10 core/utilities/facemanagement/workers/trainerworker.cpp M +1-1core/utilities/facemanagement/workers/trainerworker.h https://invent.kde.org/graphics/digikam/commit/06304b820fcee383b5a747af806c82cc44655b3b -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #41 from Maik Qualmann --- Git commit 1bc64bfd86efcf228075f3d1b0979716bde4bdb8 by Maik Qualmann. Committed on 13/11/2020 at 05:05. Pushed by mqualmann into branch 'master'. i think we should use dynamic_cast here M +30 -16 core/libs/threadimageio/engine/managedloadsavethread.cpp https://invent.kde.org/graphics/digikam/commit/1bc64bfd86efcf228075f3d1b0979716bde4bdb8 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #40 from Maik Qualmann --- Git commit 5db0ea0bacb8bb9f06636aac2f57418f782b0055 by Maik Qualmann. Committed on 12/11/2020 at 11:39. Pushed by mqualmann into branch 'master'. fix breakpoint patch M +1-0core/libs/threadimageio/preview/previewtask.cpp https://invent.kde.org/graphics/digikam/commit/5db0ea0bacb8bb9f06636aac2f57418f782b0055 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #39 from Maik Qualmann --- Git commit f14ffc9e1a3f636fbb8a0ea798e18570843d5e48 by Maik Qualmann. Committed on 12/11/2020 at 11:36. Pushed by mqualmann into branch 'master'. fix breakpoint patch when stopping loading M +5-4core/libs/threadimageio/preview/previewtask.cpp M +1-1core/libs/threadimageio/preview/previewtask.h https://invent.kde.org/graphics/digikam/commit/f14ffc9e1a3f636fbb8a0ea798e18570843d5e48 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #38 from Maik Qualmann --- Git commit cb4d73e665223ca3aad005e566a732e891151fbc by Maik Qualmann. Committed on 12/11/2020 at 06:48. Pushed by mqualmann into branch 'master'. add breakpoint to the destructor from preview task M +9-3core/libs/threadimageio/preview/previewtask.cpp M +1-0core/libs/threadimageio/preview/previewtask.h https://invent.kde.org/graphics/digikam/commit/cb4d73e665223ca3aad005e566a732e891151fbc -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #37 from Maik Qualmann --- Ok, very interesting backtrace. A destructor from the preview task (0x55fd48e0) was called although it is not finished yet. That confirms my theory, but at the moment I have no idea. In principle it is not possible in the program sequence. So we need a backtrace that may contain this. i will add a breakpoint. Maik -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #36 from Simon Westersund --- Created attachment 133247 --> https://bugs.kde.org/attachment.cgi?id=133247=edit Debug logs + GDB backtrace from 3d387983fe Here is a bit of the debug log and the backtraces again. I took the liberty of redacting the file paths partially :) -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #35 from Simon Westersund --- (In reply to Maik Qualmann from comment #34) > Did you activate the debug output with: > > export QT_LOGGING_RULES="digikam.*=true" > > I expect debug output like this: > > digikam.general: LoadingTask status is set to stopping now... 0x47aefb0 > digikam.general: PreviewLoadingTask is at the end 0x47aefb0 > digikam.general: PreviewLoadingTask destructor was called 0x47aefb0 > > Maik Thanks for pointing this out, I didn't do that! -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #34 from Maik Qualmann --- Did you activate the debug output with: export QT_LOGGING_RULES="digikam.*=true" I expect debug output like this: digikam.general: LoadingTask status is set to stopping now... 0x47aefb0 digikam.general: PreviewLoadingTask is at the end 0x47aefb0 digikam.general: PreviewLoadingTask destructor was called 0x47aefb0 Maik -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #33 from Simon Westersund --- Created attachment 133246 --> https://bugs.kde.org/attachment.cgi?id=133246=edit GDB backtrace with latest master 3d387983fe Hello Gilles and Maik, I rebuilt the master branch from today and reran the same old face detection test. The crash reproduced around 5% into the process. The backtrace is attached. Maik, you mentioned that we should expect some added debug messages right before the crash. However, I didn't see anything other than the "New Thread" and "Thread exited" messages from GDB right before the crash. Some hundred lines before the crash, there were a couple of errors about unknown image types, due to *.xcf files in my library. For example: > digikam.metaengine: Cannot load metadata from file /path/to/image.xcf (Error > # 11 : /path/to/image.xcf: The file contains data of an unknown image type > digikam.metaengine: Error: source file is not HEIF image. -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #32 from Maik Qualmann --- I will deal with this approach. Despite all that, I would like to have a new backtrace. Because additional debug messages are included. So I would have a little more of the terminal output before the crash. Maik -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #31 from caulier.gil...@gmail.com --- Maik, I think i discovered the origin of the wrong virtual calls which crash here. Clazy static analyzer report plenty of warnings relevant of this check: https://github.com/KDE/clazy/blob/master/docs/checks/README-copyable-polymorphic.md Solution: disabled copy constructor in relevant classes if possible (i currently do it) or implemented clone() operator. Some pointers : https://stackoverflow.com/questions/4122789/copy-object-keep-polymorphism https://www.fluentcpp.com/2017/09/08/make-polymorphic-copy-modern-cpp/ Did you already take a look of this approach ? Gilles -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #30 from caulier.gil...@gmail.com --- Maik, I just enable this Clazy check for you : https://github.com/KDE/clazy/blob/master/docs/checks/README-virtual-call-ctor.md Report will be updated in one hour Gilles -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #29 from Simon Westersund --- Maik and Gilles, I'm using Manjaro's default kernels. They have their own Kernel settings through which users can install some kernels, and that is what I have used. I can try to look for a way to install a generic kernel and test that. Would you prefer a 5.8.x kernel, or the latest 5.4 LTS? -- Simon -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #28 from Maik Qualmann --- Well, I know it's not an "RT". But now I rule out that the Pronlem is within our reach. OpenSUSE Tumbleweed doesn't use a pre-emptive kernel. I think if it was really safe they would. Another thing is this message on memcpy: __memmove_avx_unaligned_erms () at /usr/lib/libc.so.6 This indicates a bug in libc6 if it is compiled with X32 support. There are corresponding entries at Red Hat, they assert that they are not affected because they are not using X32 support. We have vtable problems, the loading notification and the jumg into the virtual function postProcess(). In the meantime I can rule out that we leave "dead" loading tasks in the cache. Maik -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #27 from caulier.gil...@gmail.com --- Maik, I use pre-emptive Linux kernel (recompiled by hand) in my office to reduce time-latency between process. It's not a real time kernel, but it reduce time glitch in kernel scheduler loop. Remember that in my office we implement real time data acquisition and measurement systems. I'm not sure if this kind of Kernel patches can have a side effect in user space. Gilles -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #26 from Maik Qualmann --- You have installed a preemptive kernel. Does your distribution also have a "normal" kernel? Can you install this to test? Maik -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #25 from Simon Westersund --- Created attachment 132258 --> https://bugs.kde.org/attachment.cgi?id=132258=edit GDB backtrace with latest master 925e074a71 Here is the latest backtrace. We are now back to the original pure virtual function call problem. -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #24 from Simon Westersund --- (In reply to Maik Qualmann from comment #23) > There is this message in the last backtrace and it surprises me after > research: > > __memmove_avx_unaligned_erms () at /usr/lib/libc.so.6 > > Please post the output of "uname -a" > > Maik Here is the "uname -a" output. I redacted the hostname. Linux $(hostname) 5.8.11-1-MANJARO #1 SMP PREEMPT Wed Sep 23 14:35:40 UTC 2020 x86_64 GNU/Linux -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #23 from Maik Qualmann --- There is this message in the last backtrace and it surprises me after research: __memmove_avx_unaligned_erms () at /usr/lib/libc.so.6 Please post the output of "uname -a" Maik -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #22 from Simon Westersund --- Created attachment 132247 --> https://bugs.kde.org/attachment.cgi?id=132247=edit GDB backtrace with latest master 66f96e26b5 With the latest master, there is a segmentation fault crash, instead of pure virtual fuction call. I attached a backtrace. -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #21 from Maik Qualmann --- Git commit d02237e021ae5dfe104052cb855154e05949a959 by Maik Qualmann. Committed on 09/10/2020 at 17:51. Pushed by mqualmann into branch 'master'. try with url encoded file path as cache key M +38 -27 core/libs/threadimageio/fileio/loadingdescription.cpp https://invent.kde.org/graphics/digikam/commit/d02237e021ae5dfe104052cb855154e05949a959 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #20 from Simon Westersund --- Created attachment 132231 --> https://bugs.kde.org/attachment.cgi?id=132231=edit GDB backtrace with latest master 182bef3be8 Here is a backtrace from the latest testing. The face detection crashed after around 8% completion. -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #19 from Maik Qualmann --- Git commit 43171bcf551344f450cb815bb4b3de309e78f747 by Maik Qualmann. Committed on 08/10/2020 at 18:58. Pushed by mqualmann into branch 'master'. new variant to send notifications from loader task M +2-6core/libs/threadimageio/fileio/loadingcache.cpp M +2-2core/libs/threadimageio/fileio/loadingcache.h M +15 -5core/libs/threadimageio/fileio/loadsavetask.cpp M +4-0core/libs/threadimageio/fileio/loadsavetask.h M +5-5core/libs/threadimageio/preview/previewtask.cpp M +5-5core/libs/threadimageio/thumb/thumbnailtask.cpp https://invent.kde.org/graphics/digikam/commit/43171bcf551344f450cb815bb4b3de309e78f747 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #18 from Maik Qualmann --- Git commit 59ee1d16a091cf5d85150ccaaf5601292f8e1928 by Maik Qualmann. Committed on 08/10/2020 at 18:22. Pushed by mqualmann into branch 'master'. do not notify the current loading task M +5-5core/libs/threadimageio/fileio/loadsavetask.cpp M +5-5core/libs/threadimageio/preview/previewtask.cpp M +5-5core/libs/threadimageio/thumb/thumbnailtask.cpp https://invent.kde.org/graphics/digikam/commit/59ee1d16a091cf5d85150ccaaf5601292f8e1928 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #17 from Simon Westersund --- Created attachment 132218 --> https://bugs.kde.org/attachment.cgi?id=132218=edit GDB backtrace with latest master 793a18d716 after deleting DImg_ImageMagick_Plugin.so Then I tested deleting DImg_ImageMagick_Plugin.so from here: rm /home/simon/local/opt/digikam/plugins/digikam/dimg/DImg_ImageMagick_Plugin.so Then the face recognition ran for a few minutes before crashing. -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #16 from Simon Westersund --- Created attachment 132217 --> https://bugs.kde.org/attachment.cgi?id=132217=edit GDB backtrace with latest master 793a18d716 I simply built the latest master, without removing DImg_ImageMagick_Plugin.so first. That caused in a very quick crash (within seconds from starting the face recognition). -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #15 from Maik Qualmann --- Git commit 793a18d716ee5bbe8835733118cccd25cbeea04d by Maik Qualmann. Committed on 08/10/2020 at 06:09. Pushed by mqualmann into branch 'master'. read Magick formats only at plugin start M +47 -47 core/dplugins/dimg/imagemagick/dimgimagemagickplugin.cpp M +7-0core/dplugins/dimg/imagemagick/dimgimagemagickplugin.h https://invent.kde.org/graphics/digikam/commit/793a18d716ee5bbe8835733118cccd25cbeea04d -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #14 from Maik Qualmann --- Git commit 18b0993b36d01e436e637dae9bad22d65a8092be by Maik Qualmann. Committed on 08/10/2020 at 05:49. Pushed by mqualmann into branch 'master'. disable MagickInfo for a test and use small static list M +10 -2core/dplugins/dimg/imagemagick/dimgimagemagickplugin.cpp https://invent.kde.org/graphics/digikam/commit/18b0993b36d01e436e637dae9bad22d65a8092be -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #13 from Maik Qualmann --- Hmm, strange that GetMagickInfo is always called in a different thread in all backtraces. Can you delete the "DImg_ImageMagick_Plugin.so" plugin for testing? Maik -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #12 from Simon Westersund --- Created attachment 132199 --> https://bugs.kde.org/attachment.cgi?id=132199=edit GDB backtrace with latest master b12004d560 Tested again and this time digikam crashed again, unfortunately. The backtrace remains the same as earlier. -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #11 from Simon Westersund --- (In reply to Maik Qualmann from comment #8) > Git commit 6478053c9490a73bb8b8c499750b2961627e18d2 by Maik Qualmann. > Committed on 07/10/2020 at 16:27. > Pushed by mqualmann into branch 'master'. > > further changes to the preview loader > > M +33 -40 core/libs/threadimageio/fileio/loadsavetask.cpp > M +38 -45 core/libs/threadimageio/preview/previewtask.cpp > M +27 -34 core/libs/threadimageio/thumb/thumbnailtask.cpp > > https://invent.kde.org/graphics/digikam/commit/ > 6478053c9490a73bb8b8c499750b2961627e18d2 Hello Maik, I tested with this commit now and the whole collection was scanned through without crashing :) I will still test with your latest QHash patch. Hopefully my previous test run was not just a coincidence! -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 caulier.gil...@gmail.com changed: What|Removed |Added CC||caulier.gil...@gmail.com --- Comment #10 from caulier.gil...@gmail.com --- Maik, Since few days, I managed to parse whole digiKam code with Clazy static analyzer, more centered on Qt used. And some results are interesting. I recommend to take a look : https://www.digikam.org/reports/clazy/master/ If you want to make a Clazy try, look here : https://github.com/KDE/clazy It's just another Clang plugin in fact. Just install clang-devel package, configure, compile, and install Clazy code. After that, just use my script here : https://invent.kde.org/graphics/digikam/-/blob/master/project/reports/clazy.sh The compilation traces with the Clazy warnings are exported to a log file and later exported to an HTML page generated with a Python 3 script. There are few dependencies but nothing exotic. At end results are exported to the Web site. Perhaps you will found something about Preview Thread... Gilles Gilles -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #9 from Maik Qualmann --- Git commit b12004d56014da117887938741897b171e9f9167 by Maik Qualmann. Committed on 07/10/2020 at 19:00. Pushed by mqualmann into branch 'master'. use a QHash with pointer as key M +11 -11 core/libs/threadimageio/fileio/loadingcache.cpp https://invent.kde.org/graphics/digikam/commit/b12004d56014da117887938741897b171e9f9167 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #8 from Maik Qualmann --- Git commit 6478053c9490a73bb8b8c499750b2961627e18d2 by Maik Qualmann. Committed on 07/10/2020 at 16:27. Pushed by mqualmann into branch 'master'. further changes to the preview loader M +33 -40 core/libs/threadimageio/fileio/loadsavetask.cpp M +38 -45 core/libs/threadimageio/preview/previewtask.cpp M +27 -34 core/libs/threadimageio/thumb/thumbnailtask.cpp https://invent.kde.org/graphics/digikam/commit/6478053c9490a73bb8b8c499750b2961627e18d2 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 Maik Qualmann changed: What|Removed |Added Severity|normal |crash CC||metzping...@gmail.com -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #7 from Maik Qualmann --- Git commit 9519ba801a52577565ac53143ca01a0320e687af by Maik Qualmann. Committed on 06/10/2020 at 20:39. Pushed by mqualmann into branch 'master'. use a loading flag M +5-17 core/libs/threadimageio/fileio/loadsavetask.cpp M +0-1core/libs/threadimageio/fileio/loadsavetask.h M +5-9core/libs/threadimageio/preview/previewtask.cpp M +5-9core/libs/threadimageio/thumb/thumbnailtask.cpp https://invent.kde.org/graphics/digikam/commit/9519ba801a52577565ac53143ca01a0320e687af -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #6 from Simon Westersund --- Created attachment 132178 --> https://bugs.kde.org/attachment.cgi?id=132178=edit GDB backtrace with latest master b7d3d74918 Hi Maik, I tested again with the latest master, but the same function still somehow does a pure virtual method call: Digikam::LoadingCache::notifyNewLoadingProcess(Digikam::LoadingProcess*, Digikam::LoadingDescription const&) To me, this backtrace looks exactly like the original backtrace. -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #5 from Maik Qualmann --- Git commit 80be654bda3352554eb8f5e8905734f30c7d7a29 by Maik Qualmann. Committed on 05/10/2020 at 18:17. Pushed by mqualmann into branch 'master'. i think this new loader code fix all race conditions M +1-15 core/libs/threadimageio/fileio/loadingcache.cpp M +0-2core/libs/threadimageio/fileio/loadingcache.h M +49 -51 core/libs/threadimageio/fileio/loadsavetask.cpp M +49 -51 core/libs/threadimageio/preview/previewtask.cpp M +42 -39 core/libs/threadimageio/thumb/thumbnailtask.cpp https://invent.kde.org/graphics/digikam/commit/80be654bda3352554eb8f5e8905734f30c7d7a29 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #4 from Maik Qualmann --- Git commit 6a3e2da4943e992bd81ab8820095f1df5587badd by Maik Qualmann. Committed on 04/10/2020 at 20:43. Pushed by mqualmann into branch 'master'. revert dynamic cast and lock cache again M +1-11 core/libs/threadimageio/fileio/loadingcache.cpp M +37 -31 core/libs/threadimageio/fileio/loadsavetask.cpp M +4-0core/libs/threadimageio/preview/previewtask.cpp M +30 -24 core/libs/threadimageio/thumb/thumbnailtask.cpp https://invent.kde.org/graphics/digikam/commit/6a3e2da4943e992bd81ab8820095f1df5587badd -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #3 from Simon Westersund --- I tested again with GDB enabled and now the face scan crashed with a segmentation fault, after it printed the warning that was added. The backtrace was as follows: [New Thread 0x7fff390e8640 (LWP 92762)] digikam.general: SharedLoadingTask not found!!! --Type for more, q to quit, c to continue without paging-- Thread 70 "Thread (pooled)" received signal SIGSEGV, Segmentation fault. [Switching to Thread 0x7fffd2dee640 (LWP 92023)] 0x742e2493 in ?? () from /usr/lib/libQt5Core.so.5 (gdb) bt #0 0x742e2493 in () at /usr/lib/libQt5Core.so.5 #1 0x76224c3a in QMutexLocker::QMutexLocker(QBasicMutex*) (this=0x7fffd2ded3e0, m=0x6109c020) at /usr/include/qt/QtCore/qmutex.h:233 #2 0x763b0df9 in Digikam::DImg::setAttribute(QString const&, QVariant const&) (this=0x61860510, key=..., value=...) at /home/simon/Development/kde/digikam/core/libs/dimg/dimg_props.cpp:233 #3 0x763a9984 in Digikam::DImg::load(QString const&, int, Digikam::DImgLoaderObserver*, Digikam::DRawDecoding const&) (this=0x61860510, filePath=..., loadFlagsInt=63, observer=0x61860340, rawDecodingSettings=...) at /home/simon/Development/kde/digikam/core/libs/dimg/dimg_fileio.cpp:113 #4 0x763a96b1 in Digikam::DImg::load(QString const&, Digikam::DImgLoaderObserver*, Digikam::DRawDecoding const&) (this=0x61860510, filePath=..., observer=0x61860340, rawDecodingSettings=...) at /home/simon/Development/kde/digikam/core/libs/dimg/dimg_fileio.cpp:62 #5 0x76479082 in Digikam::PreviewLoadingTask::execute() (this=0x61860330) at /home/simon/Development/kde/digikam/core/libs/threadimageio/preview/previewtask.cpp:306 #6 0x7648beac in Digikam::LoadSaveThread::run() (this=0x612fb160) at /home/simon/Development/kde/digikam/core/libs/threadimageio/fileio/loadsavethread.cpp:154 #7 0x764bb2ce in Digikam::DynamicThread::Private::run() (this=0x612fb1f0) at /home/simon/Development/kde/digikam/core/libs/threads/dynamicthread.cpp:191 #8 0x742e5dc2 in () at /usr/lib/libQt5Core.so.5 #9 0x742e1e8f in () at /usr/lib/libQt5Core.so.5 #10 0x73e7d3e9 in start_thread () at /usr/lib/libpthread.so.0 #11 0x73da9293 in clone () at /usr/lib/libc.so.6 -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #2 from Maik Qualmann --- Git commit 6812249e06eb14efd0ad9fd24604f4c59c3e211f by Maik Qualmann. Committed on 04/10/2020 at 19:09. Pushed by mqualmann into branch 'master'. add dynamic cast to SharedLoadingTask M +11 -1core/libs/threadimageio/fileio/loadingcache.cpp https://invent.kde.org/graphics/digikam/commit/6812249e06eb14efd0ad9fd24604f4c59c3e211f -- You are receiving this mail because: You are watching all bug changes.
[digikam] [Bug 427333] Face detection crashes due to pure virtual method call in LoadingCache
https://bugs.kde.org/show_bug.cgi?id=427333 --- Comment #1 from Simon Westersund --- Created attachment 132117 --> https://bugs.kde.org/attachment.cgi?id=132117=edit bootstrap.local output of CMake configuration Attached also my build configuration, if it might help differentiate from the other related bugs, which were resolved. -- You are receiving this mail because: You are watching all bug changes.
