Opened 6 years ago

Last modified 6 years ago

#8099 new defect

ext3 formatted usb mounted but not visible in journal

Reported by: ghunt Owned by: tomeu
Priority: normal Milestone: 9.1.0-cancelled
Component: sugar-datastore Version: not specified
Keywords: relnote Cc: jg, tomeu, Rmyers, martin.langhoff, paulwaite, marco
Blocked By: Blocking:
Deployments affected: Action Needed: code
Verified: no

Description

build 711. I have 4 KINGSTON usb 1 gig sticks. All ext3's mount, but do not become visible in journal. Those which mount as vfat do become visible.

Attachments (1)

ext3_non_mount_bug_logs.tgz (28.2 KB) - added by paulwaite 6 years ago.
Logs showing non-mount of ext3 usb stick

Download all attachments as: .zip

Change History (15)

comment:1 Changed 6 years ago by mstone

  • Action Needed changed from never set to reproduce
  • Cc jg tomeu added
  • Component changed from not assigned to distro
  • Owner set to mstone

comment:2 Changed 6 years ago by Rmyers

  • Cc Rmyers added

I noticed similar behavior when adding an ext2 formatted sd card a while back. With the sd I didn't consider it a bug, as an ext formatted sd could be considered an extension of the file system, rather than a pass around stick with journal entries on it.

Just noting that this might be considered proper behavior for a sd card. If it is decided that _any_ usb should show in the journal -- and I think that this can be argued as being sensible, thought should be given as to whether a non-fat sd should show as journal or as file system.

comment:3 Changed 6 years ago by mstone

Jim, Tomeu, Rmyers: any thoughts on who should own this?

comment:4 Changed 6 years ago by tomeu

  • Owner changed from mstone to tomeu

Most probably an issue in the Journal or the DataStore.

Can we get logs as per http://wiki.laptop.org/go/Attaching_Sugar_Logs_to_Tickets ?

Thanks!

comment:5 Changed 6 years ago by mstone

  • Keywords relnote polish:8.2.0 added
  • Milestone changed from Not Triaged to 9.1.0

Changed 6 years ago by paulwaite

Logs showing non-mount of ext3 usb stick

comment:6 Changed 6 years ago by paulwaite

  • Cc martin.langhoff added

Found that an ext3-formatted USB stick mounts fine on the machine onto /media, however in the Journal log there are permission errors, because the newly mounted filesystem is owned by root. Logs attached.

A vfat-formatted USB stick mounts fine and is read by the Journal.

comment:7 Changed 6 years ago by martin.langhoff

  • Cc paulwaite added

comment:8 Changed 6 years ago by martin.langhoff

Note that this would be trivial to fix just by setting the mount options to "uid=olpc".

comment:9 Changed 6 years ago by marco

  • Component changed from distro to sugar
  • Keywords blocks:8.2.0 added; polish:8.2.0 removed
  • Milestone changed from 9.1.0 to 8.2.0 (was Update.2)

comment:10 Changed 6 years ago by marco

  • Component changed from sugar to distro

The problem is that hal doesn't allow us to pass an uid:

volume.mount.valid_options = {'ro', 'sync', 'dirsync', 'noatime', 'nodiratime', 'noexec',
'quiet', 'remount', 'exec', 'acl', 'user_xattr', 'data='} (string list)

This looks like a Fedora 9 bug, I get exactly the same permission issue on my stock GNOME/Fedora 9. So assigning it back to distro.

comment:11 Changed 6 years ago by martin.langhoff

  • Cc marco added

From the distro POV, what it's doing is correct.

Sugar wants something special however - that the disk gets mounted in a way that allows us to write to the root. Perhaps that's not a reasonable thing to do...?

From that POV, perhaps Sugar should see if it can write to the root of the disk, and if it cannot, do not write the hidden index file there (not write it at all, or place it /tmp )

comment:12 Changed 6 years ago by marco

  • Keywords blocks:8.2.0 removed

comment:13 Changed 6 years ago by marco

  • Component changed from distro to sugar-datastore
  • Milestone changed from 8.2.0 (was Update.2) to 9.1.0

Oh, you are right. Ok, I think this will have to be fixed by the datastore rework we are planning for 9.1 then. The current implementation depends strongly on the existence of the index afaik (but Tomeu please chime in if I'm wrong).

comment:14 Changed 6 years ago by marco

  • Action Needed changed from reproduce to code
Note: See TracTickets for help on using tickets.