[Kde-hardware-devel] Review Request 109418: Workaround races between Solid::Backends::UDisks2::DeviceManager and DeviceBackends

[Alexander Mezin]

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

Review request for Solid and Lukáš Tinkl.


Description
-------

DeviceManager and DeviceBackends are subscribed to the same signals. DeviceManager's slots can be executed earlier than DeviceBackend's, so when deviceAdded signals are emitted by DeviceManager, DeviceBackend could contain non-updated data.
Usually the problem appears with optical disks: device notifier shows inserted disks as generic storage volumes or shows wrong label.

Possible workaround is forcing refresh of DeviceBackend's property cache before emitting deviceAdded signal. For optical disks drive's properties should also be updated, because OpticalDisc reads drive's properties


This addresses bug 314778.
    http://bugs.kde.org/show_bug.cgi?id=314778


Diffs
-----

  solid/solid/backends/udisks2/udisksdevicebackend.h 829fa41e8700e68abbb27c4a326f35eaffc39898 
  solid/solid/backends/udisks2/udisksdevicebackend.cpp 2a01b86690077df3738f740c50b29c46b40d4287 
  solid/solid/backends/udisks2/udisksmanager.h fb929ce13d69aea462cc6b96149f284b991fe39c 
  solid/solid/backends/udisks2/udisksmanager.cpp 71b9e3edebcf818f32567fb15bc391dccfbf7981 

Diff: http://git.reviewboard.kde.org/r/109418/diff/


Testing
-------

Tested on KDE 4.10.1


Thanks,

Alexander Mezin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-hardware-devel/attachments/20130311/17352e49/attachment.html>