Firewire hard drive is considered system-internal

Bug #442604 reported by ttoine
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
obsolete
Fix Released
Medium
devicekit-disks (Ubuntu)
Fix Released
Medium
Martin Pitt

Bug Description

Binary package hint: devicekit-disks

Simply connect a firewire hard drive, and the system will not mount it. There is an error message for auto mount, and if I go to shortcuts menu to open the firewire hard drive, I am prompted for my password (like sudo)

It is on Karmic only, I didn't had this problem before. I tried to add my username to the user group "disk" but it doesn't change anything.

ProblemType: Bug
Architecture: i386
Date: Sun Oct 4 23:32:13 2009
DistroRelease: Ubuntu 9.10
HotplugNewDevices: /dev/sdd /dev/sdd3 /dev/sdd2 /dev/sdd1
HotplugNewMounts: /dev/sdd2 /media/8da758ee-4416-4dc3-b542-4eae6cc61f58 ext3 rw,nosuid,nodev,relatime,errors=continue,data=writeback 0 0
Package: devicekit-disks 007-1
ProcEnviron:
 LANG=fr_FR.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-6.7-rt
SourcePackage: devicekit-disks
Symptom: storage
Uname: Linux 2.6.31-6-rt i686

Revision history for this message
ttoine (ttoine) wrote :
Revision history for this message
Martin Pitt (pitti) wrote :

Thanks for your report. I have an attempted fix for this. Can you please add my PPA

  https://launchpad.net/~pitti/+archive/ppa/

(System -> Admin -> Software Sources -> Other software -> Add)

and upgrade your system? This will pull in version 007-1karmic1test1 of devicekit-disks. Please disconnect and reconnect the drive after the upgrade and check if it gets automounted without password then.

If it does not work, please do this in a terminal:

  ls -lR /sys|gzip -9 > /tmp/sys.txt.gz

and attach /tmp/sys.txt.gz here.

Thanks!

Changed in devicekit-disks (Ubuntu):
assignee: nobody → Martin Pitt (pitti)
status: New → Incomplete
importance: Undecided → Medium
tags: added: regression-potential
Revision history for this message
In , Martin Pitt (pitti) wrote :

Firewire hard drives are detected as system internal:

added: /org/freedesktop/DeviceKit/Disks/devices/sdd
Showing information for /org/freedesktop/DeviceKit/Disks/devices/sdd
  native-path: /sys/devices/pci0000:00/0000:00:06.0/0000:01:06.0/fw-host0/0050770e00000e0d/0050770e00000e0d-0/host9/target9:0:0/9:0:0:0/block/sdd
  device: 8:48
  device-file: /dev/sdd
    by-id: /dev/disk/by-id/ieee1394-0050770e00000e0d:0:0
    by-path: /dev/disk/by-path/pci-0000:01:06.0-ieee1394-0x0050770e00000e0d:0:0
  detected at: dim. 04 oct. 2009 23:33:05 CEST
  system internal: 1
  removable: 0

This is because dk-disks only checks for subsystem=="firewire". However, current kernel calls that "ieee1394".

Revision history for this message
In , Martin Pitt (pitti) wrote :

Created an attachment (id=30116)
Patch

I had the reporter test a package with this patch, and he confirmed that automounting now works fine.

In case you are interested, http://launchpadlibrarian.net/33131716/sys.txt.gz is a sysfs dump. The linked Ubuntu bug also has udev and dk-disks monitor log, etc.

Revision history for this message
In , Martin Pitt (pitti) wrote :

Oh, I kept "firewire" because hal does the same, it seems safe to do so, and older/future kernels might have actually called it "firewire".

Revision history for this message
ttoine (ttoine) wrote : Re: [Bug 442604] Re: Auto mounting firewire hard drive needs sudo password

Martin,

Thanks for your fast answer. I just add you PPA to Synaptic and upgraded
my system, it is ok, the different kind of partitions (fat32, ntfs,
ext3 ans HFS+) of my firewire disks are now auto mounted with your patch.

I still have problem with firewire: Ffado driver for firewire sound card
can't find my Presonus Firebox, wich work well with Ubuntu Hardy and the
freebob driver.

Do I have to report another bug against Ffado, or is it the same kind of
problem ? In order to help you, you will find enclosed the report of the
command line you suggest in case of problem.

Toine

>

Revision history for this message
Martin Pitt (pitti) wrote : Re: Auto mounting firewire hard drive needs sudo password

Thanks for testing. Great, then I'll supply the patch upstream and upload it soon.

> firewire sound card

That's an entirely different problem, I'm afraid. Can you please report it against libffado?

Changed in devicekit-disks (Ubuntu):
status: Incomplete → In Progress
summary: - Auto mounting firewire hard drive needs sudo password
+ Firewire hard drive is considered system-internal
Changed in devicekit:
status: Unknown → Confirmed
Revision history for this message
Martin Pitt (pitti) wrote :

Patch sent upstream for review.

Changed in devicekit-disks (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
ttoine (ttoine) wrote : Re: [Bug 442604] Re: Auto mounting firewire hard drive needs sudo password

That's an entirely different problem, I'm afraid. Can you please report
it against libffado?

I will do it, thanks for solving the problem.

Toine

Revision history for this message
In , Zeuthen (zeuthen) wrote :

(In reply to comment #2)
> Oh, I kept "firewire" because hal does the same, it seems safe to do so, and
> older/future kernels might have actually called it "firewire".

Oh, you guys are still shipping the old firewire stack? ;-)

Anyway, committed, thanks.

Revision history for this message
In , Martin Pitt (pitti) wrote :

> Oh, you guys are still shipping the old firewire stack? ;-)

We don't, but who knows when they decide to rename it back to "firewire" (which is admittedly much more readable than ieee1943, erm, 1349, erm, 1394, erm "the old iPod connector" :)

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package devicekit-disks - 007-1ubuntu1

---------------
devicekit-disks (007-1ubuntu1) karmic; urgency=low

  * Upload current Debian git head to pick up recent bug fixes.
  * debian/rules: Enable quilt patch system. Add quilt build dependency.
  * Add 01-mkfs-tempdir.patch: Daemon does not create /var/run/DeviceKit-disks/,
    so mkfs jobs fail. Just create the directory in /tmp, this is what /tmp is
    for, after all. (See https://bugs.freedesktop.org/show_bug.cgi?id=24265)
  * Add 00git-fix-inhibit.patch: Actually make Inhibit() work again. Taken
    from upstream git head. (LP: #428133)
  * Add 02-unlock-CD-trays-after-mounting.patch: 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,
    LP: #397734)
  * Add 03-fix-subsystem-check-for-firewire.patch: 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,
    LP: #442604)
  * Add 04-mount-vfat-with-shortname-mixed-by-default.patch: 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 for 2.6.32 as well.
    (https://bugs.freedesktop.org/show_bug.cgi?id=24129, LP: #428174)
  * Add 05-dont-remove-NULL-values-from-hash-tables.patch: In device_remove(),
    device_file, object_path, or dev are sometimes NULL. Do not attempt to
    remove NULL from the hash tables, since this crashes. This is mainly a
    robustification patch, it does not really fix the underlying
    state bookkeeping. (http://bugs.freedesktop.org/show_bug.cgi?id=24264,
    LP: #414407)

 -- Martin Pitt <email address hidden> Fri, 09 Oct 2009 09:55:41 +0200

Changed in devicekit-disks (Ubuntu):
status: Fix Committed → Fix Released
Changed in devicekit:
status: Confirmed → Fix Released
Revision history for this message
ttoine (ttoine) wrote : Re: [Bug 442604] Re: Firewire hard drive is considered system-internal

@Lionel,
I have the hardware to test it. Let me know when the fix is released in
a repository (Medibuntu, PPA, etc...), so I can test it on Karmic Beta
and report if it works or not.

@Tobby,
Thanks a lot for your answer. Don't matter about your time. I hope
everything go well for you.

Toine

Changed in devicekit:
importance: Unknown → Medium
Changed in devicekit:
importance: Medium → Unknown
Changed in devicekit:
importance: Unknown → Medium
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.