linux.git/fs/udf/ialloc.c, branch v2.6.34

dquot: cleanup dquot initialize routine

2010-03-04T23:20:30+00:00

Get rid of the initialize dquot operation - it is now always called from
the filesystem and if a filesystem really needs it's own (which none
currently does) it can just call into it's own routine directly.

Rename the now static low-level dquot_initialize helper to __dquot_initialize
and vfs_dq_init to dquot_initialize to have a consistent namespace.

Signed-off-by: Christoph Hellwig 
Signed-off-by: Jan Kara

dquot: cleanup dquot drop routine

2010-03-04T23:20:30+00:00

Get rid of the drop dquot operation - it is now always called from
the filesystem and if a filesystem really needs it's own (which none
currently does) it can just call into it's own routine directly.

Rename the now static low-level dquot_drop helper to __dquot_drop
and vfs_dq_drop to dquot_drop to have a consistent namespace.

Signed-off-by: Christoph Hellwig 
Signed-off-by: Jan Kara

dquot: cleanup inode allocation / freeing routines

2010-03-04T23:20:28+00:00

Get rid of the alloc_inode and free_inode dquot operations - they are
always called from the filesystem and if a filesystem really needs
their own (which none currently does) it can just call into it's
own routine directly.

Also get rid of the vfs_dq_alloc/vfs_dq_free wrappers and always
call the lowlevel dquot_alloc_inode / dqout_free_inode routines
directly, which now lose the number argument which is always 1.

Signed-off-by: Christoph Hellwig 
Signed-off-by: Jan Kara

udf: Don't write integrity descriptor too often

2009-04-02T11:36:28+00:00

We update information in logical volume integrity descriptor after each
allocation (as LVID contains free space, number of directories and files on
disk etc.). If the filesystem is on some phase change media, this leads to its
quick degradation as such media is able to handle only 10000 overwrites or so.
We solve the problem by writing new information into LVID only on umount,
remount-ro and sync. This solves the problem at the price of longer media
inconsistency (previously media became consistent after pdflush flushed dirty
LVID buffer) but that should be acceptable.

Report by and patch written in cooperation with
Rich Coe .

Signed-off-by: Jan Kara

Fix the udf code not to pass structs on stack where possible.

2009-04-02T10:29:47+00:00

Signed-off-by: Pekka Enberg 
Signed-off-by: Jan Kara

udf: Use lowercase names of quota functions

2009-03-26T01:18:36+00:00

Use lowercase names of quota functions instead of old uppercase ones.

Signed-off-by: Jan Kara

CRED: Wrap task credential accesses in the UDF filesystem

2008-11-13T23:39:03+00:00

Wrap access to task credentials so that they can be separated more easily from
the task_struct during the introduction of COW creds.

Change most current->(|e|s|fs)[ug]id to current_(|e|s|fs)[ug]id().

Change some task->e?[ug]id to task_e?[ug]id().  In some places it makes more
sense to use RCU directly rather than a convenient wrapper; these will be
addressed by later patches.

Signed-off-by: David Howells 
Reviewed-by: James Morris 
Acked-by: Serge Hallyn 
Acked-by: Jan Kara 
Signed-off-by: James Morris

udf: Fix error paths in udf_new_inode()

2008-08-19T09:05:05+00:00

I case we failed to allocate memory for inode when creating it, we did not
properly free block already allocated for this inode. Move memory allocation
before the block allocation which fixes this issue (thanks for the idea go to
Ingo Oeser ). Also remove a few superfluous
initializations already done in udf_alloc_inode().

Reviewed-by: Ingo Oeser 
Signed-off-by: Jan Kara

udf: Fix lock inversion between iprune_mutex and alloc_mutex (v2)

2008-08-19T09:04:36+00:00

A memory allocation inside alloc_mutex must not recurse back into the
filesystem itself because that leads to lock inversion between iprune_mutex and
alloc_mutex (and thus to deadlocks - see traces below). alloc_mutex is actually
needed only to update allocation statistics in the superblock so we can drop it
before we start allocating memory for the inode.

tar           D ffff81015b9c8c90     0  6614   6612
 ffff8100d5a21a20 0000000000000086 0000000000000000 00000000ffff0000
 ffff81015b9c8c90 ffff81015b8f0cd0 ffff81015b9c8ee0 0000000000000000
 0000000000000003 0000000000000000 0000000000000000 0000000000000000
Call Trace:
 [] __mutex_lock_slowpath+0x64/0x9b
 [] mutex_lock+0xa/0xb
 [] shrink_icache_memory+0x38/0x200
 [] shrink_slab+0xe3/0x15b
 [] try_to_free_pages+0x221/0x30d
 [] isolate_pages_global+0x0/0x31
 [] __alloc_pages_internal+0x252/0x3ab
 [] cache_alloc_refill+0x22e/0x47b
 [] kmem_cache_alloc+0x3b/0x61
 [] cache_alloc_refill+0x2fe/0x47b
 [] __kmalloc+0x76/0x9c
 [] :udf:udf_new_inode+0x202/0x2e2
 [] :udf:udf_create+0x2f/0x16d
 [] :udf:udf_lookup+0xa6/0xad
...
kswapd0       D ffff81015b9d9270     0   125      2
 ffff81015b903c28 0000000000000046 ffffffff8028cbb0 00000000fffffffb
 ffff81015b9d9270 ffff81015b8f0cd0 ffff81015b9d94c0 000000000271b490
 ffffe2000271b458 ffffe2000271b420 ffffe20002728dc8 ffffe20002728d90
Call Trace:
 [] __set_page_dirty+0xeb/0xf5
 [] get_dirty_limits+0x1d/0x22f
 [] __mutex_lock_slowpath+0x64/0x9b
 [] mutex_lock+0xa/0xb
 [] :udf:udf_bitmap_free_blocks+0x47/0x1eb
 [] :udf:udf_discard_prealloc+0xc6/0x172
 [] :udf:udf_clear_inode+0x1e/0x48
 [] clear_inode+0x6d/0xc4
 [] dispose_list+0x56/0xee
 [] shrink_icache_memory+0x1d0/0x200
 [] shrink_slab+0xe3/0x15b
 [] kswapd+0x346/0x447
...

Reported-by: Tibor Tajti 
Reviewed-by: Ingo Oeser 
Signed-off-by: Jan Kara

udf: replace all adds to little endians variables with le*_add_cpu

2008-04-17T12:22:28+00:00

replace all:
	little_endian_variable = cpu_to_leX(leX_to_cpu(little_endian_variable) +
	                                    expression_in_cpu_byteorder);
with:
	leX_add_cpu(&little_endian_variable, expression_in_cpu_byteorder);
sparse didn't generate any new warning with this patch

Signed-off-by: Marcin Slusarz