Quotes around UUID/LABEL not correctly handled

[Giovanni Bortolozzo]

Binary package hint: upstart

I submit this bug to upstart as stated in the manpage of mountall.

All the local filesystem but the / are not mounted on a fresh boot.

This prevent dbus (and then hal/network-manager/gdm services) to start.

This is the output of 'mountall -v' (I kill the process after a while since it seems to be stuck). In attachment my fstab.

root@nemo:~# mountall -v
mount / [3106] exited normally
mount /proc [3107] exited normally
mount /proc/sys/fs/binfmt_misc [3108] exited normally
mount /sys [3109] exited normally
mount /sys/fs/fuse/connections [3110] exited normally
mount /sys/kernel/debug [3111] exited normally
mount /sys/kernel/security [3112] exited normally
mount /dev [3113] exited normally
mount /dev/pts [3114] exited normally
mount /dev/shm [3115] exited normally
mount /var/run [3116] exited normally
mount /var/lock [3117] exited normally
mount /lib/init/rw [3118] exited normally
virtual filesystems finished
remote finished
^C
root@nemo:~#

ProblemType: Bug
Architecture: i386
Date: Wed Sep 16 23:32:34 2009
DistroRelease: Ubuntu 9.10
NonfreeKernelModules: nvidia
Package: mountall 0.1.5
ProcEnviron:
 PATH=(custom, user)
 LANG=it_IT.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-10.34-generic
SourcePackage: mountall
Uname: Linux 2.6.31-10-generic i686

[Scott James Remnant (Canonical) (canonical-scott)]

I think that you have the NTFS issue which should be fixed with 0.1.6

[Scott James Remnant (Canonical) (canonical-scott)]

Please test 0.1.6

[Giovanni Bortolozzo (giovanni-bortolozzo)]

Yestarday evening a post a followup to my bug report, but seems that it never arrived (maybe is still in queue in my local smtp server)

Anyway. I examinated my fstab and I see that for the local filesystem that are not mounted (all but the / directory) the UUID was between "" (these was line that I manually added to the fstab). I modified the file removing these "" and after reboot all the local filesystem are mounted.

So seems that there is a problem on parsing the fstab file.

Anyway this evening I'll try the new version both and without the "" in the fstab.

Just another question, not related to this bug report. Can this problem that I have (bug #404976) be related to mountall?

Regads
Giovanni

[Scott James Remnant (Canonical) (canonical-scott)]

The quotes sounds entirely plausible to be the issue

[Giovanni Bortolozzo (giovanni-bortolozzo)]

I test the new 0.16 and it works and dbus (and then hal/network-manager/gdm) starts.

I try also to add again the quotes around UUID and there is still the problem.

I see that also you correct the manpage.

So let me know if you need that I test something else (if you have time can you quickly read my bug report for the bug #404976)

Thanks
Giovanni

[Scott James Remnant (Canonical) (canonical-scott)]

On Thu, 2009-09-17 at 18:16 +0000, Giovanni Bortolozzo wrote:

> I test the new 0.16 and it works and dbus (and then hal/network-
> manager/gdm) starts.
>
> I try also to add again the quotes around UUID and there is still the
> problem.
>
Yeah, they're not technically supported but looks like mount itself does
so mountall should too. I'll be fixing that.

> So let me know if you need that I test something else (if you have time
> can you quickly read my bug report for the bug #404976)
>
I've quickly read it, and it sounds like exactly the kind of bug that
mountall was written to solve.

Scott
--
Scott James Remnant
<email address hidden>

[Scott James Remnant (Canonical) (canonical-scott)]

The ubuntu-boot PPA has a new version of mountall which I hope will fix this problem, please install that and let me know how it works out.

[Giovanni Bortolozzo (giovanni-bortolozzo)]

I test it, and the problem with quotes is still present.

borto:~> apt-cache policy mountall
mountall:
  Installato: 0.1.7
  Candidato: 0.1.7
  Tabella versione:
 *** 0.1.7 0
        500 http://it.archive.ubuntu.com karmic/main Packages
        100 /var/lib/dpkg/status

[Scott James Remnant (Canonical) (canonical-scott)]

On Tue, 2009-09-22 at 21:06 +0000, Giovanni Bortolozzo wrote:

> I test it, and the problem with quotes is still present.
>
> borto:~> apt-cache policy mountall
> mountall:
> Installato: 0.1.7
> Candidato: 0.1.7
> Tabella versione:
> *** 0.1.7 0
> 500 http://it.archive.ubuntu.com karmic/main Packages
> 100 /var/lib/dpkg/status
>
This is not the version from the PPA.

You'll have to download it and force install it, otherwise APT will use
the version from the archive.

Scott
--
Scott James Remnant
<email address hidden>

[Giovanni Bortolozzo (giovanni-bortolozzo)]

Ops... since now I never know what a PPA is.

This evening a I will test that version

[Giovanni Bortolozzo (giovanni-bortolozzo)]

It works. AFAIK, now the quotes in the fstab are handle correctly.

Do you have any suggestion, to my other problem (#404976)?

Thanks
Giovanni

[Scott James Remnant (Canonical) (canonical-scott)]

This bug was fixed in the package mountall - 0.1.8

---------------
mountall (0.1.8) karmic; urgency=low

  [ Scott James Remnant ]
  * Further work on the fix from the previous version where the root
    filesystem would always be considered "local", retain that from the
    POV of the {virtual,local,remote}-filesystems events, but do mount
    the root straight away when it's virtual since there's no device to
    wait until it's ready. LP: #431204.
  * If a remote filesystem is already mounted and doesn't need a remount,
    don't wait for a network device to come up. LP: #430348.

  * Ignore single and double quotes in fstab device specifications, since
    mount -a used to. LP: #431064.
  * Never write mtab when mounting a mount with showthroughs (ie. /var)
    and instead update mtab once we've moved it into place
    later. LP: #434172.

  [ Kees Cook ]
  * src/mountall.c: rework nftw hooks to use a global for argument passing
    instead of using nested functions and the resulting trampolines that
    cause an executable stack. LP: #434813.
  * debian/rules: revert powerpc exception, since the cause is fixed by
    removing the nested functions.

 -- Scott James Remnant <email address hidden> Wed, 23 Sep 2009 14:19:01 -0700