Add udisks_ata_identify_get_word() utility function

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add logging statements so it's possible to pin-point probing time

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

UDisksLinuxDevice: Don't include the parent_instance member in the docs

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Forgot to add files

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Introduce UDisksLinuxDevice as a replacement for GUdevDevice

This is essentially a GUdevDevice but with extra information such as
the probed ATA IDENTIFY data and, in the future, anything else we
might need.

The big idea here is that we want to depend _less_ on udev for
peripheral things that doesn't make sense to store in the udev
database, such as the properties in [1]. The win here is

 a. We can simplify ata_id somewhat - it still needs to extract serial
    numbers, WWNs etc. (because those are needed for symlinks in the
    /dev/disk/by-* hierarchy) but it does not need to extract and
    print what is in e.g. [1].

 b. If we need information in udisks from e.g. the ATA IDENTIFY data
    we don't need to add it to ata_id (or scsi_id or whatever), then
    wait for a systemd release and then instruct distributors to
    properly handle the dependency - instead, we can just access the
    IDENTIFY data ourselves. Much simpler and less error prone.

Since probing is synchronous, we create a thread for this and whenever
there's an uevent we push a new request onto a work queue. The probing
thread will handle incoming requests sequentially (in the future we
could have a pool of worker threads and run things in parallel) and
pop the results back to the main thread, also sequentially.

The only probing we have right now is for IDENTIFY DEVICE and IDENTIFY
PACKET DEVICE for ATA disks. We can add more in the future.

Note that this patch doesn't actually achieve b. yet - upcoming
patches will move from reading udev properties (such as looking at the
value of ID_ATA_FEATURE_SET_PM) to extracting this from the IDENTIFY
data itself.

[1] :
 # udevadm info -q all -n /dev/sda|grep ID_ATA_
 E: ID_ATA_DOWNLOAD_MICROCODE=1
 E: ID_ATA_FEATURE_SET_HPA=1
 E: ID_ATA_FEATURE_SET_HPA_ENABLED=1
 E: ID_ATA_FEATURE_SET_PM=1
 E: ID_ATA_FEATURE_SET_PM_ENABLED=1
 E: ID_ATA_FEATURE_SET_SECURITY=1
 E: ID_ATA_FEATURE_SET_SECURITY_ENABLED=0
 E: ID_ATA_FEATURE_SET_SECURITY_ENHANCED_ERASE_UNIT_MIN=2
 E: ID_ATA_FEATURE_SET_SECURITY_ERASE_UNIT_MIN=2
 E: ID_ATA_FEATURE_SET_SECURITY_FROZEN=1
 E: ID_ATA_FEATURE_SET_SMART=1
 E: ID_ATA_FEATURE_SET_SMART_ENABLED=1
 E: ID_ATA_ROTATION_RATE_RPM=0
 E: ID_ATA_SATA=1
 E: ID_ATA_SATA_SIGNAL_RATE_GEN1=1
 E: ID_ATA_SATA_SIGNAL_RATE_GEN2=1
 E: ID_ATA_WRITE_CACHE=1
 E: ID_ATA_WRITE_CACHE_ENABLED=1

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Factor out ATA routines

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Use the right device file UDisksObjectInfo.one_liner for a partition of a drive

E.g. use /dev/sdb1, not /dev/sdb

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Don't count /dev/md* devices with bogus UUIDs as MD-RAID arrays

For example, if you use the FAULTY personality there is no UUID and we
shouldn't present the device as a RAID array as it really isn't one.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Nuke debug spew

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

UDisksObjectInfo: add one_liner field

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Only use #pragma GCC diagnostic on GCC >= 4.6

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

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add support for option 'no-block' in the Block.Format() method

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Revert "Serialize authorization checks"

Turns out this is not really a good idea - for example, consider two
users on different seats - if one is slow to authenticate, the other
one will suffer.

This reverts commit cc373db17ac448a6c088f6497a90fe288e560c21.

Serialize authorization checks

This is useful for e.g. applications that offer UI on multiple disks
at the same time. Without the serialization up to N authentication
dialogs for the same action can be shown.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Always pass MD devices of the form /dev/mdNNN when assembling RAID arrays

This is to be consistent with array creation.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add Manager.MDRaidCreate() method for RAID Array creation

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

UDisksClient: Add reference to Job:Operation to get_job_description() docs

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add missing known jobs for MD-RAID to Job:Description docs

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add missing known jobs for ATA Secure erase to Job:Description docs

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add translations for eu, fa, hr and sq from Transifix

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Update POTFILES.in with new source files

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Remove fa_IR translation

This language code is invalid. Also removed from the udisks transifex instance.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add publish-docs-master target

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Update gtk-doc types file

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Include docs for MD-RAID implementation details

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Update gtk-doc sections

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add indexes of deprecated and new-in-2.1 symbols

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Re-include udisks_client_get_drive_info() in API docs

Otherwise we can't tell people that it's deprecated.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Use Deprecated and Since annotations to mark deprecated and new API for 2.1

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Use @since annotations to mark new D-Bus API available 2.1

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Merge branch 'master' into wip/mdraid

Start writing NEWS for 2.0.90

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Bump version to 2.0.90

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Explain post-release actions in HACKING

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Document the release process

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Update my email address

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Post-release version bump to 2.0.1

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Update NEWS for release

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Update translations from Transifex

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add docs for udisks_linux_drive_ata_secure_erase_sync()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add docs for udisks_linux_drive_ata_apply_configuration()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Remove udisks_daemon_util_on_other_seat() from sections.txt

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Fix docs for udisks_daemon_util_file_set_contents()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Fix docs for udisks_daemon_launch_threaded_job()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Fix docs for udisks_daemon_launch_spawned_job_sync()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Fix docs for udisks_daemon_launch_spawned_job()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Fix docs for udisks_daemon_launch_simple_job()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add docs for udisks_base_job_set_auto_estimate()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add docs for udisks_base_job_get_auto_estimate()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Fix up docs for udisks_daemon_util_file_set_contents()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Fix docs for udisks_daemon_util_on_same_seat()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

docs: use CDATA for email addresses to avoid them being mangled

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

docs: add some more meta information to the man pages (version, source, manual)

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

docs: Document udisksctl(1)'s status command

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

docs: Document udisksctl(1)'s info command

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

docs: update man page dates to "October 2012"

In the future we shall update it only in minor releases .0,
e.g. 2.0.0, 2.2.0, 2.4.0 and so on.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

docs: Link to freedesktop.org/wiki/Software/dbus in udisks(8)

... so it's 100% clear what "system message bus" means.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

docs: Remove "(available online here)" comment in udisks(8) man page

We don't do that for other chapters.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

docs: only reference hdparm(8) at the end of the paragraph

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Bump version number to 2.0.0

This is not the release (at least NEWS needs to be updated) but pretty
close. Basically only need to e.g. check that nothing breaks by going
from 1.99.0 to 2.0.0

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Don't require that users define UDISKS_API_IS_SUBJECT_TO_CHANGE

We're very close to the 2.0 release (two-point-oh!) so nuke this.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Move bash completion script into /usr/share/bash-completion/completions

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Fix glaringly wrong documentation for Filesystem.Mount()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Mention the right file when complaing about /proc/swaps

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Make it work without requiring the kernel to be CONFIG_SWAP=y

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

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Don't complain about missing /etc/fstab file

It's not inconceivable that this file is missing so don't loudly
complain about it.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Don't complain about missing /etc/crypttab file

Some systems don't have it at all - notably F18 doesn't have if unless
you use encrypted root. Also make the file monitor pick up when the
file is deleted. This was reported in bug 53147:

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

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Only do the isohybrid hack for the first partition

Otherwise we end up showing two partitions and that is unwanted. See

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

for details.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Fix same seat detection if built with systemd, but not booted with it

If we haven't booted with systemd, then we can only assume it's always
the same seat.

Based on patch by Takashi Iwai <tiwai@suse.com>.

https://bugzilla.novell.com/show_bug.cgi?id=769570
https://bugs.freedesktop.org/show_bug.cgi?id=55309

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Consistently refer to MD-RAID as "Linux RAID" in user-visible strings

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Set MDRaid:ChunkSize for raid0 but not for raid1

It so happens that redundancy != striped - for example raid0 has the
former, but not the latter and raid1 is the other way around. And
raid4, raid5, raid6 and raid10 has both properties.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Nuke debug spew

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Handle nested arrays

We actually need mdadm(8) changes for this to work properly otherwise
the MD_* properties end up overwriting each other.

We also have to be careful since a single GUdevDevice instance can be
in two distinct UDisksLinuxMDRaidObject.

http://people.freedesktop.org/~david/gnome-disks-mdraid-20120920-1.png

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Mention that MDRaid:SyncAction is the empty string for arrays w/o redundancy

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

UDisksClient: include RAID level in description of RAID Array

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Slightly rework UDisksObjectInfo docs

Also include the UDisksObject instance in the struct.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Skip homehost when calcuating @name member in UDisksObjectInfo for MD-RAID

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Initialize reference count to 1

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Deprecate udisks_client_get_drive_info() with udisks_client_get_object_info()

The new function is a bit more abstract, works on more types and
returns a UDisksObjectInfo object. This allows us to add information
in the future.

Also introduce symbolic icons as requested in bug

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

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Revert "Simplify codepath when removing a RAID disk"

Doesn't seem to reliably work - probably an mdadm(8) bug. Revert for
now, make a note to investigate later.

This reverts commit fb3df672113a380f2d7555bf5dcac0814b4a0c80.

Export MDRaid:ChunkSize property

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add MDRaid:BitmapLocation property

This is useful for the GUI to convey to the user when a bitmap is in
use.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Simplify codepath when removing a RAID disk

No need for two mdadm(8) invocations, can use --set-faulty and
--remove in the same invocation.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Express RAID member state as an array of strings, instead of just a string

The sysfs file /sys/block/md0/md/dev-sda/state is a comma-separated
list of states, not just one state.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

configure.ac: raise gudev dependency

udisks is using g_udev_device_get_usec_since_initialized(), which
was added in gudev 165.

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

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add @start-degraded option to MDRaid.Start()

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add MDRaid.RemoveDevice() method

This is used in the "-" button in Disks, see

http://people.freedesktop.org/~david/gnome-disks-mdraid-20120914-2.png

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Tweak MDRaid:ActiveDevices some more

We need to iterate over md/dev-*, not just md/rd%d, otherwise we miss
spares and failed devices. With this change, Disks can show the user
what really is going on

http://people.freedesktop.org/~david/gnome-disks-mdraid-20120914-1.png

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Slightly change how the MDRaid:ActiveDevice property works

It's easier to use from applications if it's an array with a fixed
size.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Make Block:PreferredDevice work for partitions of RAID arrays

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

UDisksClient: ignore partitions when finding block device for RAID Array

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add MDRaid:ActiveDevices property

This property contains a list of the active devices in the array. It
can be used to implement a dialog like the following:

 http://people.freedesktop.org/~david/palimpsest-md-raid-components-on-mp.png

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Make udisks_daemon_util_resolve_link() work when @name contains slashes

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Skip partitions of MD-RAID devices

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Add MDRaid:SyncCompleted property

We update this by polling every second (on the second) for a subset of
states that are not 'idle'.

http://people.freedesktop.org/~david/gnome-disks-mdraid-20120913-2.png

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Fix corner-cases for detecting RAID arrays

If the array device was detected before any of the components we would
fail to detect the array. Fix that.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Strip trailing white-space from md/sync_action sysfs attr

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Avoid leaking memory when parsing sysfs attributes with integers

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Watch for changes on the md/sync_action and md/degraded sysfs files

The md(4) driver does not use the usual uevent 'change' mechanism for
notification - instead it excepts user-space to select(2)-ish on a fd
for the sysfs attribute. Annoying. See

 http://www.kernel.org/doc/Documentation/md.txt

for more details.

Signed-off-by: David Zeuthen <zeuthen@gmail.com>

Nuke debug spew

Signed-off-by: David Zeuthen <zeuthen@gmail.com>