<?xml version="1.0" encoding="UTF-8" standalone="yes" ?>
<!DOCTYPE bugzilla SYSTEM "https://bugs.kde.org/page.cgi?id=bugzilla.dtd">

<bugzilla version="5.0.6"
          urlbase="https://bugs.kde.org/"
          
          maintainer="sysadmin@kde.org"
>

    <bug>
          <bug_id>293926</bug_id>
          
          <creation_ts>2012-02-12 18:46:17 +0000</creation_ts>
          <short_desc>Polling of solid and mtab for dynamic collections horribly broken</short_desc>
          <delta_ts>2012-09-28 07:53:11 +0000</delta_ts>
          <reporter_accessible>1</reporter_accessible>
          <cclist_accessible>1</cclist_accessible>
          <classification_id>2</classification_id>
          <classification>Applications</classification>
          <product>amarok</product>
          <component>Collections/Local</component>
          <version>2.5.0</version>
          <rep_platform>openSUSE</rep_platform>
          <op_sys>Linux</op_sys>
          <bug_status>RESOLVED</bug_status>
          <resolution>FIXED</resolution>
          
          
          <bug_file_loc></bug_file_loc>
          <status_whiteboard></status_whiteboard>
          <keywords></keywords>
          <priority>NOR</priority>
          <bug_severity>normal</bug_severity>
          <target_milestone>2.6</target_milestone>
          
          
          <everconfirmed>0</everconfirmed>
          <reporter name="Stefan Brüns">stefan.bruens</reporter>
          <assigned_to name="Amarok Bugs">amarok-bugs-null</assigned_to>
          <cc>bart.cerneels</cc>
    
    <cc>eljefedelito</cc>
    
    <cc>florian</cc>
    
    <cc>ian.monroe</cc>
    
    <cc>nicholas</cc>
    
    <cc>pndiku</cc>
    
    <cc>ralf-engels</cc>
    
    <cc>rudd-o</cc>
          
          <cf_commitlink>http://commits.kde.org/amarok/92fe9ac171b9920ba65adf244c7dbe23d694d209</cf_commitlink>
          <cf_versionfixedin>2.7</cf_versionfixedin>
          <cf_sentryurl></cf_sentryurl>
          <votes>0</votes>

      

      

      

          <comment_sort_order>oldest_to_newest</comment_sort_order>  
          <long_desc isprivate="0" >
    <commentid>1226119</commentid>
    <comment_count>0</comment_count>
    <who name="Stefan Brüns">stefan.bruens</who>
    <bug_when>2012-02-12 18:46:17 +0000</bug_when>
    <thetext>+++ This bug was initially created as a clone of Bug #249760 +++
Currently, amarok polls for changes every second. This polling involves:
1. Reading and parsing mtab (via KMountpoint)
2. Reading and parsing fstab (via KMountpoint)
3. Querying solid for a list of all StorageAccess devices.

Especially the last point is problematic, as this involves querying UDisks, creating a huge number of DBus roundtrips.

A fix for solid s mtab monitoring is in reviewboard. After this is applied, the changes in MediaCache should be reverted.</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1226818</commentid>
    <comment_count>1</comment_count>
    <who name="Myriam Schweingruber">myriam</who>
    <bug_when>2012-02-14 11:34:30 +0000</bug_when>
    <thetext>Could you please provide a link to that fix in reviewboard?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1226835</commentid>
    <comment_count>2</comment_count>
    <who name="Peter C. Ndikuwera">pndiku</who>
    <bug_when>2012-02-14 12:04:04 +0000</bug_when>
    <thetext>I see these 3:

https://git.reviewboard.kde.org/r/103956/
https://git.reviewboard.kde.org/r/103954/
https://git.reviewboard.kde.org/r/103957/

Could they be the ones?</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1227069</commentid>
    <comment_count>3</comment_count>
    <who name="Stefan Brüns">stefan.bruens</who>
    <bug_when>2012-02-15 00:30:54 +0000</bug_when>
    <thetext>https://git.reviewboard.kde.org/r/103954/ is the relevant request</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1231272</commentid>
    <comment_count>4</comment_count>
    <who name="Mario Bensi">mbensi</who>
    <bug_when>2012-02-28 15:24:50 +0000</bug_when>
    <thetext>Git commit b16cfec44104ded4a8959af4e790fe2f7d6d2745 by Mario Bensi.
Committed on 28/02/2012 at 16:22.
Pushed by bensi into branch &apos;KDE/4.8&apos;.

Fix for mtab monitoring on Linux

Add a QSocketNotifier pointing to the mtab fd.

REVIEW: 103954
Related: bug 293914

M  +15   -1    solid/solid/backends/fstab/fstabwatcher.cpp
M  +4    -0    solid/solid/backends/fstab/fstabwatcher.h

http://commits.kde.org/kdelibs/b16cfec44104ded4a8959af4e790fe2f7d6d2745</thetext>
  </long_desc><long_desc isprivate="0" >
    <commentid>1301017</commentid>
    <comment_count>5</comment_count>
    <who name="Stefan Brüns">stefan.bruens</who>
    <bug_when>2012-09-27 10:23:12 +0000</bug_when>
    <thetext>Fixed in 2.7

http://commits.kde.org/amarok/92fe9ac171b9920ba65adf244c7dbe23d694d209</thetext>
  </long_desc>
      
      

    </bug>

</bugzilla>