Debian Bug report logs - #360165
initscripts: Don't mount /proc/bus/usb if it's not necessary

Display info messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Sam Morris <sam@robots.org.uk>

To: Debian Bug Tracking System <submit@bugs.debian.org>

Subject: initscripts: Don't mount /proc/bus/usb if it's not necessary

Date: Fri, 31 Mar 2006 01:27:56 +0100

Package: initscripts
Version: 2.86.ds1-13
Severity: wishlist

Since kernel 2.6.14 (I think), /proc/bus/usb has not been necessary
since udev will create device nodes for usb devices in /dev/bus/usb.
Since libusb 2:0.1.10a-22, libusb try and look for devices in /dev/bus/usb
before searching /proc/bus/usb.

I have commented out the code that mounts the usbfs in mountkernfs.sh
with no ill effect. It would be nice if the script would only mount the
usbfs if /dev/bus/usb is not available to manage the device nodes for
USB devices.

-- System Information:
Debian Release: 3.1
  APT prefers testing
  APT policy: (530, 'testing'), (520, 'unstable'), (510, 'experimental')
Architecture: i386 (i686)
Kernel: Linux 2.6.16-1-k7
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)

Versions of packages initscripts depends on:
ii  debianutils                2.15.3        Miscellaneous utilities specific t
ii  e2fsprogs                  1.37-2sarge1  ext2 file system utilities and lib
ii  libc6                      2.3.6-3       GNU C Library: Shared libraries an
ii  lsb-base                   3.0-16        Linux Standard Base 3.0 init scrip
ii  mount                      2.12p-4sarge1 Tools for mounting and manipulatin

-- no debconf information

Message #10 received at 360165@bugs.debian.org (full text, mbox, reply):

From: Petter Reinholdtsen <pere@hungry.com>

To: 360165@bugs.debian.org, 360165-submitter@bugs.debian.org

Subject: Re: initscripts: Don't mount /proc/bus/usb if it's not necessary

Date: Fri, 08 Sep 2006 13:43:33 +0200

[Sam Morris 2006-03-31]
> I have commented out the code that mounts the usbfs in
> mountkernfs.sh with no ill effect. It would be nice if the script
> would only mount the usbfs if /dev/bus/usb is not available to
> manage the device nodes for USB devices.

Any idea how to detect when /dev/bus/usb mounting isn't required?
Patches are most welcome.  I'm not yet ready to comment out the code
yet, but would welcome some iftest to disable it when it isn't needed.

Friendly,
-- 
Petter Reinholdtsen

Message #18 received at 360165@bugs.debian.org (full text, mbox, reply):

From: Henrique de Moraes Holschuh <hmh@debian.org>

To: Petter Reinholdtsen <pere@hungry.com>, 360165@bugs.debian.org

Cc: 360165-submitter@bugs.debian.org

Subject: Re: [Pkg-sysvinit-devel] Bug#360165: initscripts: Don't mount /proc/bus/usb if it's not necessary

Date: Fri, 8 Sep 2006 10:01:01 -0300

On Fri, 08 Sep 2006, Petter Reinholdtsen wrote:
> Any idea how to detect when /dev/bus/usb mounting isn't required?

It is required until all users of /proc/bus/usb/devices and other users of
/proc/bus/usb/* migrate to the new way of doing things.  May I remind you
that udev is not yet mandatory in Debian, so we will also have to check if
udev is active on top of checking /proc/filesystems for usbfs...

This is the sort of thing we should announce the deprecation, and schedule
for etch+1.   By that time, we will only have to support kernels that have
both usbfs AND udev-class /dev/bus/usb support, and maybe Debian will even
have to decide to go with mandatory udev (and udev will be trustable and
stable enough to actually be mandatory, anyway ;-) ).

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh

Message #23 received at 360165@bugs.debian.org (full text, mbox, reply):

From: Petter Reinholdtsen <pere@hungry.com>

To: 360165@bugs.debian.org, 360165-submitter@bugs.debian.org

Subject: Re: [Pkg-sysvinit-devel] Bug#360165: initscripts: Don't mount /proc/bus/usb if it's not necessary

Date: Fri, 8 Sep 2006 15:08:04 +0200

[Henrique de Moraes Holschuh]
> It is required until all users of /proc/bus/usb/devices and other users of
> /proc/bus/usb/* migrate to the new way of doing things.  May I remind you
> that udev is not yet mandatory in Debian, so we will also have to check if
> udev is active on top of checking /proc/filesystems for usbfs...

Right.  Checking if udev is active is trivial by looking for
/dev/.udev.  Anything else required before disabling the
/proc/bus/usb/ mounting?

> This is the sort of thing we should announce the deprecation, and
> schedule for etch+1.

Actually, I believe it is enough to add logic in the code to only
disable it when it make sense, and then leave that code in the scripts
while we wait for it to become obsolete.

> By that time, we will only have to support kernels that have both
> usbfs AND udev-class /dev/bus/usb support, and maybe Debian will
> even have to decide to go with mandatory udev (and udev will be
> trustable and stable enough to actually be mandatory, anyway ;-) ).

I guess this is the time such code become obsolete. :)

Friendly,
-- 
Petter Reinholdtsen

Message #34 received at 360165@bugs.debian.org (full text, mbox, reply):

From: Henrique de Moraes Holschuh <hmh@debian.org>

To: Petter Reinholdtsen <pere@hungry.com>, 360165@bugs.debian.org

Cc: 360165-submitter@bugs.debian.org

Subject: Re: Bug#360165: [Pkg-sysvinit-devel] Bug#360165: initscripts: Don't mount /proc/bus/usb if it's not necessary

Date: Fri, 8 Sep 2006 12:01:44 -0300

On Fri, 08 Sep 2006, Petter Reinholdtsen wrote:
> /dev/.udev.  Anything else required before disabling the
> /proc/bus/usb/ mounting?

Sure.  Nothing in userspace using it. Which is the point I was trying to
make, but apparently failed to do.

libusb is not the only user of usbfs.  If the kernel supports it, we *have*
to mount it until we have given all userspace enough time to migrate away
from it.

Unless you are 100% sure libusb is the sole user of usbfs, of course. If
that's true, then we can write the kernel version + udev version + udev
active check required right now.

> Actually, I believe it is enough to add logic in the code to only
> disable it when it make sense, and then leave that code in the scripts

Right now, AFAIK, you can disable it ONLY if the kernel doesn't support it
in the first place.

-- 
  "One disk to rule them all, One disk to find them. One disk to bring
  them all and in the darkness grind them. In the Land of Redmond
  where the shadows lie." -- The Silicon Valley Tarot
  Henrique Holschuh

Message #42 received at 360165-close@bugs.debian.org (full text, mbox, reply):

From: Petter Reinholdtsen <pere@debian.org>

To: 360165-close@bugs.debian.org

Subject: Bug#360165: fixed in sysvinit 2.86.ds1-62

Date: Tue, 23 Jun 2009 17:02:06 +0000

Source: sysvinit
Source-Version: 2.86.ds1-62

We believe that the bug you reported is fixed in the latest version of
sysvinit, which is due to be installed in the Debian FTP archive:

initscripts_2.86.ds1-62_i386.deb
  to pool/main/s/sysvinit/initscripts_2.86.ds1-62_i386.deb
sysv-rc_2.86.ds1-62_all.deb
  to pool/main/s/sysvinit/sysv-rc_2.86.ds1-62_all.deb
sysvinit-utils_2.86.ds1-62_i386.deb
  to pool/main/s/sysvinit/sysvinit-utils_2.86.ds1-62_i386.deb
sysvinit_2.86.ds1-62.diff.gz
  to pool/main/s/sysvinit/sysvinit_2.86.ds1-62.diff.gz
sysvinit_2.86.ds1-62.dsc
  to pool/main/s/sysvinit/sysvinit_2.86.ds1-62.dsc
sysvinit_2.86.ds1-62_i386.deb
  to pool/main/s/sysvinit/sysvinit_2.86.ds1-62_i386.deb



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to 360165@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Petter Reinholdtsen <pere@debian.org> (supplier of updated sysvinit package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing ftpmaster@debian.org)


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Tue, 23 Jun 2009 18:18:53 +0200
Source: sysvinit
Binary: sysvinit sysvinit-utils sysv-rc initscripts
Architecture: source i386 all
Version: 2.86.ds1-62
Distribution: unstable
Urgency: low
Maintainer: Debian sysvinit maintainers <pkg-sysvinit-devel@lists.alioth.debian.org>
Changed-By: Petter Reinholdtsen <pere@debian.org>
Description: 
 initscripts - Scripts for initializing and shutting down the system
 sysv-rc    - System-V-like runlevel change mechanism
 sysvinit   - System-V-like init utilities
 sysvinit-utils - System-V-like utilities
Closes: 67095 316468 339955 360165 422940 457896 461103 471101 483392 484883 501724 504877 510455 510912 512003 519520 526398 529805 534300
Changes: 
 sysvinit (2.86.ds1-62) unstable; urgency=low
 .
   [ Kel Modderman ]
   * Add missing semi-colon in LSB header of init.d/urandom (Closes:
     #501724).
   * Force C locale for df command used in init.d/mountoverflowtmp.
     (Closes: #510912)
   * Update startpar source to upstream 0.52 (from 0.50) and apply debian
     compatibility patch (sent upstream). Also workaround /dev/pts not
     being mounted early in runlevel S by having startpar use an own
     version of getpt(2) system call. (Closes: #457896, #484883)
   * Enable CONCURRENCY=makefile mode in debian/sysv-rc/etc/init.d/rc.
     This mode uses startpar and the /etc/init.d/.depend.* makefile-like
     information created by insserv to calculate boot script execution
     order. The splash API is not supported in this mode of boot. This
     mode of boot is considered very experimental at this time.
   * Allow startpar using concurrency modes to operate in runlevel S, now
     that startpar can manage to work properly in absence of /dev/pts.
   * Use uscan dversionmangle option to remove .ds\d+ string from modifed
     orig.tar.gz. This fixes the debian-watch-file-should-mangle-version
     lintian warning. Patch taken from diff on LP: #312764.
   * Add comment to debian/sysv-rc/etc/init.d/rc about debug=echo, it
     should never be left uncommented for a real boot process and should
     only be used for interactive debugging. (Closes: #510455)
   * Do not mount usbfs any more, it was deprecated long ago.
     Applications should all use /dev/bus/usb, which can be managed more
     flexibly (assign permissions with udev, and the like). Thanks Martin
     Pitt. (Closes: #483392, #422940, #360165, #471101)
   * Add Homepage field to debian/control.
   * Add interface for disabling system init script start links with
     update-rc.d, and enabling them again. (Closes: #67095)
   * When update-rc.d in "start|stop"-mode is invoked without the final
     dot, script warnings are displayed just before the "usage" message
     is shown. Check that last argument to start|stop command is a period
     earlier. Thanks to Patrick <pet@painfullscratch.nl>.  (Closes: #512003)
   * Use -delete option of find(1) instead of piping through xargs in
     debian/initscripts/lib/init/bootclean.sh. (Closes: #316468, #461103)
   * Make init.d/rc simpler by dropping support for sourcing .sh files
     after the policy finally changed in 3.8.1.  Update to Standards
     version 3.8.1. (Closes: #339955, #519520)
   * Allow initscripts postinst maintainer script to not fail when
     attempting to mkdir /dev/pts and /dev/shm when running in a
     fakechroot environment. This is done by checking for FAKECHROOT env
     variable before bailing out. Patch thanks to Daniel Kahn Gillmor.
     (Closes: #504877)
   * Update debian/NOTES to reflect current affairs.
 .
   [ Petter Reinholdtsen ]
   * Move the startpar binary to a more proper location, from sysvinit
     to sysvinit-utils.
   * Comment out the code disabling fsck when running on battery.  It
     need changes in fsck to work properly (Closes: #526398).  This
     Reopens #326647.
   * Add a /usr/sbin/service command (Closes: #534300).  Modified the
     manual page based on ideas from the sysvconfig package.  Thanks to
     Steve Langasek and Ubuntu.  Add conflict with sysvconfig and
     chkconfig providing the same program.
   * Improve init.d/umountfs umount ordering code to avoid being
     confused while still solving #391673 (Closes: #529805).  Patch
     from Tim Phipps.
   * Make sysv-rc depend on a version of sysvinit-utils with a properly
     working startpar, in case concurrent booting is enabled.
   * Drop shell style concurrent booting, and make CONCURRENCY=shell
     behave like CONCURRENTY=startpar, as startpar concurrency now work
     properly.
Checksums-Sha1: 
 77ff3ecfa1130e320dacfaec70b044d381c85fca 1483 sysvinit_2.86.ds1-62.dsc
 0fe9f119888b22ab789e2b9babeabcb95efb0b5d 155616 sysvinit_2.86.ds1-62.diff.gz
 81cb9e47352efc1de8e72c31d3b4f7c6c4c6d69f 103082 sysvinit_2.86.ds1-62_i386.deb
 34da4a531585d44301ece3088c5981d687a002c5 86618 sysvinit-utils_2.86.ds1-62_i386.deb
 f2f446c6e5c9fdbb1c93822fa33e209967d09545 69010 initscripts_2.86.ds1-62_i386.deb
 a2ebfd00011db4f3d0609634b2409acba7e14826 65638 sysv-rc_2.86.ds1-62_all.deb
Checksums-Sha256: 
 89db98da56f8c2417aee098f44cc301c2821409bbe1682515a86fa311363c843 1483 sysvinit_2.86.ds1-62.dsc
 f655ed6fdfd2591dadc399aaae9fe6dc1782d165feb908ff240b7ad2d1f0e4f2 155616 sysvinit_2.86.ds1-62.diff.gz
 718dfa0d930d38130358cef22d5b86ae2e078eb7ea9615b6a75a3ed8307128f8 103082 sysvinit_2.86.ds1-62_i386.deb
 ab26b862341d93190797c4b79bcd4acf25185eccb12f6127b98954cbf667ba60 86618 sysvinit-utils_2.86.ds1-62_i386.deb
 bf8f5330b35ba5ea5a42fbd43d8d43dcf893534983114a83d5daff7db2d6031f 69010 initscripts_2.86.ds1-62_i386.deb
 bf3e86f69effe488a950c8c04955505e6dedb73d577b6c038edab4c4376092d3 65638 sysv-rc_2.86.ds1-62_all.deb
Files: 
 8aff0cbff66ff250babf4be897c88dda 1483 admin required sysvinit_2.86.ds1-62.dsc
 6dd9d90daa7384f24198e2a20534d9cd 155616 admin required sysvinit_2.86.ds1-62.diff.gz
 88235b6992eccdbc9689efd79c021149 103082 admin required sysvinit_2.86.ds1-62_i386.deb
 40006b5c47ecd8e7af6f2fb0356eecbe 86618 admin required sysvinit-utils_2.86.ds1-62_i386.deb
 a8cd6f0d2982e6cfb7489d56a449f079 69010 admin required initscripts_2.86.ds1-62_i386.deb
 2b8ebaa8b9a72a66abf0e7c5ffb90fd9 65638 admin required sysv-rc_2.86.ds1-62_all.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)

iD8DBQFKQQiC20zMSyow1ykRAjJ7AKDts2KBER9fjpOcCIrHnwyy9aR/xgCgy0r5
ZBq+xDveg/S8a8ikptgAiC0=
=O+4B
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.