Rename Controller interface to Adapter

Add support for Port objects

This commit only includes support for ATA ports. Support for SAS and
FibreChannel will follow.

Also generate D-Bus docs for the Controller interface

Fix type in udev rules for pci-id

Initial stab at exporting storage controllers via D-Bus

Guesstimate the buffer size to use for benchmarking

This gives more precise results and puts an upper bound on time taken
to benchmark (minutes).

Abort benchmark on IO errors

This happens a lot with DVDs because of "encryption".

Add DriveWwn, DriveRotationalRate and DriveWriteCache properties

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add a method to benchmark drives

Signed-off-by: David Zeuthen <davidz@redhat.com>

Bug 24885 — LVM LVs now have NOPOLICY=0

Commit a95d3 caused the NOPOLICY rules get cut short for LVs. Move the NOPOLICY
check close to the top, since its (1) very cheap, and (2) very generic, i. e.
rules further down might want to tweak it for special cases.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Update udev rules to cope better with device-mapper

See https://bugzilla.redhat.com/show_bug.cgi?id=528909#c24 for details.

Post-release version bump to 010

Update NEWS for release

Don't use hyphens in param names

It may upset some tools reading the introspection XML.

Bug 24718 — Proper handling of missing mkfs.*/fsck.*

Introduce two new errors:
  org.freedesktop.DeviceKit.Disks.Error.FilesystemDriverMissing
  org.freedesktop.DeviceKit.Disks.Error.FilesystemToolsMissing

which are thrown instead of a general Error.Failed when file system tools (such
as mkfs.<type> or xfs_admin) are not installed, or the kernel does not support
the fs type for FilesystemMount().

Note that this does not yet cover a missing cryptsetup.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Bug 24679 — Support creating minix file systems

Signed-off-by: David Zeuthen <davidz@redhat.com>

Bug 24757 – Simplify helper-change-luks-password

Use "set -e" to do the error handling which was previously done manually.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Bug 24757 — Bashism in luks helper breaks password changing

Fix bashism in devkit-disks-helper-change-luks-password which broke
LuksChangePassphrase() due to POSIX shell not knowing about "-e".

Signed-off-by: David Zeuthen <davidz@redhat.com>

Bug 24778 — throw_error() segfaults for daemon-internally called methods

Some functions like devkit_disks_device_drive_ata_smart_refresh_data() are used
internally by the daemon, in which case the DBusGMethodInvocation context is
NULL. If these methods throw an error (observed on "daemon is inhibited"), this
NULL context is propagated all the way down to throw_error, which previously
tried to call dbus_g_method_return_error() with a NULL context; there is no
D-Bus method call to terminate, so this caused a segfault.

Change throw_error() to just use g_warning() in this case, so that the error
appears on the daemon's stderr. This makes it generally safe to internally call
D-Bus exported methods.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Use BLKPG_ADD_PARTITION when adding a partition

This is similar to the previous commit. We don't want to rely on
libparted because that won't work if another partition on the main
device is busy.

Use BLKPG_DEL_PARTITION when deleting partitions instead of libparted

Sadly, libparted gets this wrong since it does the equivalent of
'blockdev --rereadpt' which fails if one or more partitions are busy.

Also disallow deleting an extended partition if logical partitions
exist. We can undo this if we add some code to also do DEL_PARTITION
for the logical partitions (including scrubbing signatures).

Also ignore a device if DM_UDEV_DISABLE_OTHER_RULES_FLAG is set

https://bugzilla.redhat.com/show_bug.cgi?id=528909#c10

Add new LinuxMdComponentPosition property

Use unregister facility in dbus-glib 0.82 and misc life-cycle fixes

Pass -F to mkfs.ext[234] to allow creating a filesystem on the whole disk

Allow creating a partition table with same scheme as existing one

Use 'udevadm settle' instead of 15-second timeout

Suggested by Milan Broz <mbroz@redhat.com> in

https://bugzilla.redhat.com/show_bug.cgi?id=530721#c1

Work around blkid incorrectly detecting FAT on extended partitions

This is really a blkid bug but work around it for now by clearing Id*
properties for extended partitions.

When updating holders/slaves, defer the updates to an idle handler

This is needed because sometimes updates may depend on the actual
device just being added.

Specifically, this patch fixes a problem with manually unlocking LUKS
devices, e.g.

 # echo xyz123 | cryptsetup -T 1 luksOpen /dev/sdb1 foobar

Here we get /dev/dm-0 at some point and then we want to update
/dev/sdb1. But the update of /dev/sdb1 depends on the existence of
/dev/dm-0 - specifically update_info_luks() won't set :DeviceIsLuks
and :Luks before /dev/dm-0 really exists.

Bug 24673 — Support creating swap fs with labels

mkswapfs supports labels just fine, add support for it to FilesystemCreate().

Signed-off-by: David Zeuthen <davidz@redhat.com>

Pass -T to cryptsetup to handle incorrect passphrases

Also don't sleep on the error path. Also filed a bug for the
root-problem and included the bug reference in the tirade of why we
need to sleep 15 seconds etc.

https://bugzilla.redhat.com/show_bug.cgi?id=530721

Various device-mapper and cryptsetup fixes

Post-release version bump to 009

Update NEWS for release

Bug 24499 – media players have USB drive icon

Set "media-player" icon for devices which have ID_MEDIA_PLAYER (as set by
media-player-info).

Signed-off-by: David Zeuthen <davidz@redhat.com>

Bug 24468 – part-id crashes if drive device node does not exist

This fixes a corner case crash if a partition device node exists (like
/dev/sda5), but the corresponding drive device node (like /dev/sda) does not.
This should not happen in a running system, but was reported to occur during a
distribution upgrade.

Just handle the situation gracefully with a proper error message instead of
crashing.

http://bugs.freedesktop.org/show_bug.cgi?id=24468

Signed-off-by: David Zeuthen <davidz@redhat.com>

Bug 24264 – Crash on removing NULL value from hash in device_remove()

Properly handle the situtation where we're processing a 'change' event
but the 'remove' event has happened but is still queued up in our own
process. Also handle device file renames and 'move' uevents.

https://bugs.freedesktop.org/show_bug.cgi?id=24264

Bug 24351 – Firewire hard drive is considered system-internal

Firewire subsystem is called "ieee1394" in current Linux. Now check for both
"ieee1394" and "firewire". This fixes firewire drives to not be considered
system internal any more.

https://bugs.freedesktop.org/show_bug.cgi?id=24351

Signed-off-by: David Zeuthen <davidz@redhat.com>

Bug 24343 – Unintended side-effect when calling DriveDetach()

Some laptops like the Dell Mini 10 have builtin SD card readers which
are wired through an internal USB bus. The current behaviour is that
if you click the eject button in nautilus, the card reader device gets
ejected and disconnected, and cannot be reactivated ever, except by a
reboot.

The cause of this is that DK-d's update_info_drive() sets "can_detach"
for all USB devices currently, which causes
devkit-disks-helper-drive-detach to be called on them.

https://bugs.freedesktop.org/show_bug.cgi?id=24343

Bug 24129 – Please mount vfat disks with shortname=mixed

The previous default, shortname=lower, breaks all-uppercase file names ("touch
FOO" creates "foo"), thus breaks rsync, and Windows compatibility.

The default was changed in the Linux kernel as well, see
http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=955234755ce4a2c33cfc558912aa8f2148cc1fc6

https://bugs.freedesktop.org/show_bug.cgi?id=24129

Signed-off-by: David Zeuthen <davidz@redhat.com>

Bug 24054 – Devickit-disks doesn't support changing ntfs label

Bug 24053 – MMC cards aren't automounted.

Show MMC/MemStick cards by default. The previous whitelist in the
udev rules ignored them.

https://bugs.freedesktop.org/show_bug.cgi?id=24053

Signed-off-by: David Zeuthen <davidz@redhat.com>
Signed-off-by: Martin Pitt <martin.pitt@ubuntu.com>

Bug 24052 – CDROM eject button is locked while CDROM is mounted

Unlike in the hal world, we do not have a daemon polling CD drives for eject
button presses. In order to make hardware tray eject buttons work, unlock the
tray after mounting a CD.

This is pretty much equivalent to yanking out USB sticks, which we already
handle reasonably (detecting disappeared device, force-unmounting).

https://bugs.freedesktop.org/show_bug.cgi?id=24052

Signed-off-by: David Zeuthen <davidz@redhat.com>

Actually inhibit the daemon when Inhibit() is called

Guess this was forgotten with the polkit-1 porting effort.

Post-release version bump to 008

Update NEWS for release

Bug 23926 – Error creating partition

When creating a partition we may already have received the uevent for
the new partition. Check for this before waiting for it.

Bug 23541 – PartitionTableCreate method times out

... when 'none' is specified as a parameter.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Bug 23579 – Must allow DBus Properties interface for g-d-u to work

Bug 23761 – Some leaks

g_hash_table_get_values returns a newly-allocated GList. The callers
of devkit_disks_daemon_local_get_all_devices need to free it.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Bug 23762 – Fix typo

Bug 23764 – Small leak in update_drive_properties_from_sysfs

Bug 23823 – NULL dereference

Bug 23824 – Unused variable

Bug 23825 – Leak in update_info_mount_state

Bug 23826 – NULL dereference

Bug 23827 – Undefined value

Bug 23828 – Undefined value in umount-devkit.c

Bug 23829 – Undefined value in partutil.c

Use WindowsStyleCaps for property names

Technically any UTF-8 string can be used for a property name. However,
some tools are incapable of handling property names with dashes ('-')
in them. So change to using the recommending naming. See

http://lists.freedesktop.org/archives/devkit-devel/2009-September/000430.html

for discussion.

This is based on a patch from Davide Bettio <davide.bettio@kdemail.net>.

Also be paranoid about the UUID containing '/' characters

This can't really happen (for known filesystems) so this "fix" is
really only for people playing along at home. See

 http://bugs.freedesktop.org/show_bug.cgi?id=23235

for more details.

Rework ATA SMART handling

Basically make everything a lot simpler. Also don't store blobs in a
database. For now at least - we might want to add that feature back at
some point.

Bug 22914 – Require password to eject a CD or DVD mounted by the same user

Apparently this got messed up in the port to polkit1.

Post-release version bump to 007

Update NEWS for release

Bug 23235 – Cannot mount disc with / (slash) in label

Make FilesystemMount() accept an 'auth_no_user_interaction' option

This allows automounters to request that no user interaction should ever
take place. See

 https://bugs.freedesktop.org/show_bug.cgi?id=22652

for discussion.

Ignore loop and ram devices for now

Add a new :device-presentation-nopolicy property

Desktops should read this property and avoid automounting etc. if it
is set to 1.

Bug 23309 – Udev rules probe floppy drives when they shouldn't

Fix up object path mapping

In particular this failed with AoE devices which has basenames of the
form "etherd!e0.1". Should work for everything now.

Hide recovery, boot, and hidden partitions from desktops

Set DKD_PRESENTATION_HIDE for partitions which desktops should hide, such as
EFI, Apple bootstrap, or recovery partitions.

For recovery partitions is a direct port of the earlier hal fdis:
http://cgit.freedesktop.org/hal/tree/fdi/policy/10osvendor/20-storage-methods.fdi#n80

Apple boostrap, EFI, and hidden partitions now don't rely on labels any more,
but directly check partitioning schema (mbr, GUID) and types.

libgdu evaluates this and exports it as device-presentation-hide
property, which gvfs respects.

https://bugs.freedesktop.org/show_bug.cgi?id=22707

Signed-off-by: David Zeuthen <davidz@redhat.com>

Require latest libatasmart

This version of libatasmart has a new ABI/API with new features. We
won't initially take advantage of these new features so just do a
simple port for now.

Rely on the disk to spin itself down

In other words, instead of playing games with watching
/sys/block/sdX/stat files and sending a STANDBY IMMEDIATE command when
these haven't changed for a while, just use the STANDBY command itself
with the timeout value.

This avoids having to wake up the daemon every five seconds. In
addition, this change ensures that device specific minimums are
respected.

We didn't do this in the original commit because while testing the
feature, some of disks, notably ones from WD, appeared to not respect
the STANDBY command. However, when moved to another enclosure (the
disks was initially in a SATA PM enclosure, it got moved to a SAS
enclosure) things started working again. Also turns out the WD disk in
question has a device specific timeout of 10 minutes.

Add LinuxMdCreate method

Also fix a bug in PartitionTableCreate so we don't return until the
partition table has actually been created.

Add Device:drive-is-rotational property

This new property can be used to make a distinction between hard disks
and SSDs, see

http://people.freedesktop.org/~david/palimpsest-new-treeview.png

for an example.

Fix up ATA SMART detection

Recent udev started using ata_id instead of scsi_id.

Also migrate from SYSFS to ATTR/ATTRS.

Misc fixes for optical disc detection

Move data files into a separate directory

Move implementation of DrivePollMedia into a separate process

Much much better to do this from a helper process since some
devices/drivers may hang the process in state D (uninterruptible IO)
until the device was removed. This was the last place where we opened
the device from the daemon.

Work around media detection problems

Seems that the kernel won't revalidate media unless O_NONBLOCK is
passed. This contradicts how things are supposed to work (O_NONBLOCK
is a hint to avoid testing things like media state), see
http://bugzilla.kernel.org/show_bug.cgi?id=13029 for more information.

So to be on the safe side, open the device both with and without
O_NONBLOCK.

Add --poll-for-media option to devkit-disks(1)

Pass size of buffer to avoid Z_BUF_ERROR when compressing ATA SMART blob

Fix polling process title and turn off noisy debugging spew

Fix segfault when starting a Linux MD array

Add a way to spin down drives

Post-release version bump to 006

Update NEWS for release

Also includes news about the 004 release since that slipped in the
previous release...

Use AM_SILENT_RULES if available

Clean up build a bit

Remove unused SMART test files

Sort out dbus-glib registration mess

We sometimes segfaulted when stopping a RAID device. The problem here
was that we kept the exported D-Bus object around until it was
finalized (by using weak refs). This was problematic because several
change events were treated as add events but the state of the RAID
device was such that we didn't want to add the object anyway. So we
ended up unreffing the object. Which means that the job completion
func accessed a unreferenced object.

The solution here is to remove the object from our hashes as soon as
we want to remove it.

Since we don't have dbus_g_connection_unregister_g_object() just yet
(available in dbus-glib >= 0.81) we just keep the object on the
bus. If we later wants to add a new object but the old object is still
around we use an undocumented hack to unregister the object.

You will also need eggdbus >= 0.5, otherwise introspection won't work.

Add API to check/repair Linux MD RAID arrays

Use ID_CDROM_MEDIA to detect optical drives

This is available since udev 142 that we already depend on. This
change makes my USB stick from Sandisk "work" as "expected":

http://people.freedesktop.org/~david/thanks-sandisk.png

Update udev rules

 - depend on udev >= 142 since that will pull in /sbin/blkid
 - avoid probing dm or md if it has been done already
 - don't create symlinks for dm

GIO modules, for the polkit action lookup module, need to be prefixed with lib

Better handling of Floppy and Zip drives

Welcome to the 1980s

Bug 22149 – Do not poll floppy drives

Avoid polling PC floppy drives since that makes a lot of
noise. DrivePollMedia() still works, e.g.

 dbus-send --system --print-reply --dest=org.freedesktop.DeviceKit.Disks  \
           /org/freedesktop/DeviceKit/Disks/devices/fd0                   \
           org.freedesktop.DeviceKit.Disks.Device.DrivePollMedia

Note, though, that udev will not run /sbin/blkid on the device. So
desktops wanting to support this needs to ignore the fstype for floppy
drives and just attempt to mount it anyway.

Fix bug when retrieving historical ATA SMART data

Use gudev-1.0 instead of devkit-gobject

Can't believe how easy this port was...