Re: [Bug 317161] [NEW] udev init script should use partition size from /etc/default/tmpfs

2009-01-19 Thread Scott James Remnant
Also, does adding the option in a remount actually have any effect?

Scott
-- 
Scott James Remnant
sc...@canonical.com

-- 
udev init script should use partition size from /etc/default/tmpfs 
https://bugs.launchpad.net/bugs/317161
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


Re: [Bug 317161] [NEW] udev init script should use partition size from /etc/default/tmpfs

2009-01-15 Thread Scott James Remnant
On Wed, 2009-01-14 at 16:51 +, Iuri Diniz wrote:

 I'm tunning ubuntu to a low memory device without swap (eee pc 701), I'm
 currently managing the max size of tmps partitions by setting size on
 /etc/default/tmpfs, but udev doens't use that settings, so I've made a
 patch to correct it.
 
 the default behavior to udev init script (kernel behavior) is to set
 tmpfs partition to half size of system ram that I think is too much, 8
 MB would be sufficient
 
Actually, I believe you're mistaking tmpfs with ramfs here.  Space is
not reserved for a tmpfs, it simply sets the maximum size to which the
tmpfs may grow.

If a tmpfs is only using 5MB, then it will only consume 5MB of memory;
it's backed by the page cache, so there is little overhead for such
things as filesystem (in fact, there isn't one!)


/dev is not writable by ordinary users, there is no danger of users
filling this filesystem with other data.

However there it is preferred that this filesystem not become full, if
the udev daemon is unable to make device nodes, or maintain its
database, then system features such as hotplug fail to work and newly
connected devices are not available.

It is for this reason that we choose not to limit the size of the
filesystem.

 status wontfix

Scott
-- 
Scott James Remnant
sc...@canonical.com


** Changed in: udev (Ubuntu)
   Status: New = Won't Fix

-- 
udev init script should use partition size from /etc/default/tmpfs 
https://bugs.launchpad.net/bugs/317161
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs


Re: [Bug 317161] [NEW] udev init script should use partition size from /etc/default/tmpfs

2009-01-15 Thread Iuri Diniz
Hello Scott,

On Thu, Jan 15, 2009 at 2:03 PM, Scott James Remnant
sc...@canonical.comwrote:

 Actually, I believe you're mistaking tmpfs with ramfs here.  Space is
 not reserved for a tmpfs, it simply sets the maximum size to which the
 tmpfs may grow.

 If a tmpfs is only using 5MB, then it will only consume 5MB of memory;
 it's backed by the page cache, so there is little overhead for such
 things as filesystem (in fact, there isn't one!)


I know that




 /dev is not writable by ordinary users, there is no danger of users
 filling this filesystem with other data.

 However there it is preferred that this filesystem not become full, if
 the udev daemon is unable to make device nodes, or maintain its
 database, then system features such as hotplug fail to work and newly
 connected devices are not available.


I tottaly agree with you



 It is for this reason that we choose not to limit the size of the
 filesystem.

I think that you misunderstood me, I'm not suggesting to limit the size, I'm
suggesting to allow people if they want, they can limit the size. Currently
on ubuntu, I can do this on /dev/shm, /var/lock, /var/run, /lib/init/rw [1],
but I can't on /dev

The default behavior of my patch is to do nothing because the default
contents of /etc/default/tmpfs on my ubuntu is to not limit size. So the
patch only add support to allow people to choose the size, the patch is not
imposing the size.

IMHO I think that allowing people to choose the max size a good thing, but,
anyway, the final decision is yours, and I will respect it.

Thank you for your feedback

[1] see the contents of /etc/init.d/mountkernfs.sh,
/etc/init.d/mountdevsubfs.sh, /etc/init.d/mtab.sh
-- 
Iuri Diniz
http://iuridiniz.com [Sou um agitador, não um advogado]
http://blog.igdium.com [Linux on Limbo]

-- 
udev init script should use partition size from /etc/default/tmpfs 
https://bugs.launchpad.net/bugs/317161
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

[Bug 317161] [NEW] udev init script should use partition size from /etc/default/tmpfs

2009-01-14 Thread Iuri Diniz
Public bug reported:

I'm tunning ubuntu to a low memory device without swap (eee pc 701), I'm
currently managing the max size of tmps partitions by setting size on
/etc/default/tmpfs, but udev doens't use that settings, so I've made a
patch to correct it.

the default behavior to udev init script (kernel behavior) is to set
tmpfs partition to half size of system ram that I think is too much, 8
MB would be sufficient

using this patch, udev size will use UDEV_SIZE option if setted on
/etc/default/tmpfs, if not setted it will use TMPFS_SIZE option if
setted, and if not setted too, the default behavior is assumed

this is my first patch, I don' t if I made it correctly following the
guidelines

** Affects: udev (Ubuntu)
 Importance: Undecided
 Status: New

-- 
udev init script should use partition size from /etc/default/tmpfs 
https://bugs.launchpad.net/bugs/317161
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs