Message ID | 1254938690.4051.24.camel@lenovo |
---|---|
State | Accepted |
Headers | show |
Colin Ian King wrote: > For: Hardy Lum, netbook-lpia branch > > LP#445673 > > https://bugs.launchpad.net/ubuntu/+bug/445673 > > unionfs does not have the delete=all mount option compiled > in. Enable this by adding -DUNIONFS_DELETE_ALL to CFLAGS. > > Without the delete=all option configured one cannot mount > a unionfs without having to fallback on using whiteouts which > can consume a considerable amount of inodes to mark deleted > files. This can cause problems on unionfs systems with limited > inode reservations where storage is very limited. > > Attached: the patch > > ACK. Do you think this might also apply to Jaunty? Or did we use aufs ? They all run together.
On Wed, Oct 07, 2009 at 07:04:50PM +0100, Colin Ian King wrote: > For: Hardy Lum, netbook-lpia branch > > LP#445673 > > https://bugs.launchpad.net/ubuntu/+bug/445673 > > unionfs does not have the delete=all mount option compiled > in. Enable this by adding -DUNIONFS_DELETE_ALL to CFLAGS. > > Without the delete=all option configured one cannot mount > a unionfs without having to fallback on using whiteouts which > can consume a considerable amount of inodes to mark deleted > files. This can cause problems on unionfs systems with limited > inode reservations where storage is very limited. > Sounds reasonable and the patch looks pretty safe. Acked-by: Andy Whitcroft <apw@canonical.com> -apw
Colin Ian King wrote: > For: Hardy Lum, netbook-lpia branch > > LP#445673 > > https://bugs.launchpad.net/ubuntu/+bug/445673 > > unionfs does not have the delete=all mount option compiled > in. Enable this by adding -DUNIONFS_DELETE_ALL to CFLAGS. > > Without the delete=all option configured one cannot mount > a unionfs without having to fallback on using whiteouts which > can consume a considerable amount of inodes to mark deleted > files. This can cause problems on unionfs systems with limited > inode reservations where storage is very limited. > > Attached: the patch > > Applied to hardy-lum
From 07932dd65714d0fa292f59468eddfc01b659afec Mon Sep 17 00:00:00 2001 From: Colin Ian King <colin.king@canonical.com> Date: Wed, 7 Oct 2009 17:50:50 +0000 Subject: [PATCH] UBUNTU: unionfs: enable delete=all mount option BugLink: https://bugs.launchpad.net/ubuntu/+bug/445673 unionfs does not have the delete=all mount option compiled in. Enable this by adding -DUNIONFS_DELETE_ALL to CFLAGS. Without the delete=all option configured one cannot mount a unionfs without having to fallback on using whiteouts which can consume a considerable amount of inodes to mark deleted files. This can cause problems on unionfs systems with limited inode reservations where storage is limited. Signed-off-by: Colin Ian King <colin.king@canonical.com> --- ubuntu/fs/unionfs/Makefile | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/ubuntu/fs/unionfs/Makefile b/ubuntu/fs/unionfs/Makefile index 5d553c0..195fe1d 100644 --- a/ubuntu/fs/unionfs/Makefile +++ b/ubuntu/fs/unionfs/Makefile @@ -3,7 +3,7 @@ SUP_MAJOR= 2 SUP_MINOR= 6 SUP_PATCH= 18 -EXTRA_CFLAGS+=-DUNIONFS_VERSION=\"${UNIONFS_VERSION}\" -DSUP_MAJOR=${SUP_MAJOR} -DSUP_MINOR=${SUP_MINOR} -DSUP_PATCH=${SUP_PATCH} -DUNIONFS_UNSUPPORTED +EXTRA_CFLAGS+=-DUNIONFS_VERSION=\"${UNIONFS_VERSION}\" -DSUP_MAJOR=${SUP_MAJOR} -DSUP_MINOR=${SUP_MINOR} -DSUP_PATCH=${SUP_PATCH} -DUNIONFS_UNSUPPORTED -DUNIONFS_DELETE_ALL # This will enable full debugging support # EXTRA_CFLAGS+=-DUNIONFS_DEBUG -- 1.5.4.3