Review Request: UDisks mount vfat filesystems with flush option.

[Konstantinos Smanis]

On Tue, Jun 12, 2012 at 11:33 AM, Maarten De Meyer
<de.meyer.maarten at gmail.com> wrote:
>
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/105224/
>
> Review request for kdelibs, Solid and Lukáš Tinkl.
> By Maarten De Meyer.
>
> Description
>
> This patch mounts vfat filesystems (mostly usb devices) with the flush
> option when using the UDisks backend. The hal backend already did this.
> The flush option makes sure changes are written to the device immediately.
>
> Please let me know it this should be implemented in an other (optional)
> way or if this should be in the UDev rules directly.
>
> ps. I do not have commit rights.
>
> Testing
>
> Compiled without error, mounted multiple devices without problems.
>
> Bugs: 273792
>
> Diffs
>
> solid/solid/backends/udisks/udisksstorageaccess.cpp (7d72a88)
>
> View Diff

I tested the patch against 4.8.3, works as expected.

However, I too think that this option should be implemented somewhere
else in the stack (udisks/udev: whatever is most fit). Although a sane
default, you should be able to configure it more easily.

--
Konstantinos Smanis