[PATCH] [IPMI] remove unused target and action in Makefile

2008-02-26 Thread Denis Cheng
Kbuild system handle this automatically. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/char/ipmi/Makefile |4 1 files changed, 0 insertions(+), 4 deletions(-) diff --git a/drivers/char/ipmi/Makefile b/drivers/char/ipmi/Makefile index 553f0a4..eb8a1a8 100644 --- a/d

[PATCH 2/2] [GFS2] re-support special inode

2008-02-25 Thread Denis Cheng
, block device file, fifo pipe, and socket file, lose many important features as a common file system. this one line patch re add special inode support. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/gfs2/inode.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/f

[PATCH 1/2] [GFS2] remove gfs2_dev_iops

2008-02-25 Thread Denis Cheng
struct inode_operations gfs2_dev_iops is always the same as gfs2_file_iops, since Jan 2006, when GFS2 merged into mainstream kernel. So one of them could be removed. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/gfs2/inode.c |2 +- fs/gfs2/ops_inode.c | 10 -- f

[PATCH] drivers/block/: add __devinitdata to all pci_device_id table of block drivers

2008-02-02 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/block/DAC960.c |2 +- drivers/block/cciss.c|2 +- drivers/block/cpqarray.c |2 +- drivers/block/sx8.c |2 +- drivers/block/umem.c |2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff -

[PATCH] xfs: add __init/__exit mark to specific init/cleanup functions

2008-02-02 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/xfs/linux-2.6/xfs_super.c |2 +- fs/xfs/linux-2.6/xfs_vnode.c |2 +- fs/xfs/support/ktrace.c |4 ++-- fs/xfs/support/uuid.c|2 +- fs/xfs/xfs_vfsops.c |4 ++-- 5 files changed, 7 insertions

[PATCH 1/3] uio: Kconfig improvements

2008-02-02 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/uio/Kconfig | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/uio/Kconfig b/drivers/uio/Kconfig index b778ed7..d8ab7e6 100644 --- a/drivers/uio/Kconfig +++ b/drivers/uio/Kconfig @@ -1,

[PATCH 2/3] uio: mark pci_device_id hilscher_pci_ids[] __devinitdata

2008-02-02 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/uio/uio_cif.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/uio/uio_cif.c b/drivers/uio/uio_cif.c index 838bae4..4a5a97e 100644 --- a/drivers/uio/uio_cif.c +++ b/drivers/uio/uio_cif.c @@ -116,7

[PATCH 3/3] uio: vm_operations_struct ->nopage to ->fault method conversion

2008-02-02 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/uio/uio.c | 19 --- 1 files changed, 8 insertions(+), 11 deletions(-) diff --git a/drivers/uio/uio.c b/drivers/uio/uio.c index cc246fa..47e0c32 100644 --- a/drivers/uio/uio.c +++ b/drivers/uio/uio.c @@ -417,30 +

[PATCH] ide-pci-generic: kill the unused ifdef/endif/MODULE code

2008-02-02 Thread Denis Cheng
de-pci-generic.all-generic-ide parameter also documented in Documentation/kernel-parameters.txt Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- Documentation/kernel-parameters.txt |3 +++ drivers/ide/pci/Makefile|3 ++- drivers/ide/pci/generic.c |

[PATCH] ide-core: remove conditional compiling with MODULE in ide-core.c

2008-02-01 Thread Denis Cheng
cs, module parameters also can be input on the kernel command line, with this style: ide-core.options="ide=nodma hdd=cdrom idebus=..." so Documentation/kernel-parameters.txt also updated. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- Documentation/kernel-parameters.txt |

[PATCH] wireless/ath5k: renamed to ath5k_pci_driver to fix Section mismatch warnings

2008-02-01 Thread Denis Cheng
the struct pci_driver refered ath5k_pci_id_table which in __devinit section, the sparse tool suggest this renamed to "*driver", kills mismatch warnings. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/net/wireless/ath5k/base.c |6 +++--- 1 files changed, 3

[PATCH] fs/dlm/: add __init and __exit marks to init and exit functions

2008-02-01 Thread Denis Cheng
it moves 365 bytes from .text to .init.text, and 30 bytes from .text to .exit.text, saves memory. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/dlm/config.c|2 +- fs/dlm/debug_fs.c |2 +- fs/dlm/lockspace.c |2 +- fs/dlm/memory.c|2 +- fs/dlm/netlink.c

[PATCH] fs/char_dev.c: chrdev_open marked static and removed from fs.h

2008-01-31 Thread Denis Cheng
there is an outdated comment in serial_core.c also fixed. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/serial/serial_core.c |4 ++-- fs/char_dev.c|2 +- include/linux/fs.h |1 - 3 files changed, 3 insertions(+), 4 deletions(-) diff -

[PATCH] dlm/user.c: static initialization improvements

2008-01-28 Thread Denis Cheng
also change name_prefix from char pointer to char array. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/dlm/user.c | 13 +++-- 1 files changed, 7 insertions(+), 6 deletions(-) diff --git a/fs/dlm/user.c b/fs/dlm/user.c index 4f74154..2cc5415 100644 --- a/fs/dlm/user.c ++

[PATCH] USB: Use menuconfig objects

2008-01-24 Thread Denis Cheng
separately. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/usb/atm/Kconfig|9 - drivers/usb/gadget/Kconfig |9 + drivers/usb/serial/Kconfig | 11 --- 3 files changed, 13 insertions(+), 16 deletions(-) diff --git a/drivers/usb/atm/Kconfig b/drivers/u

[PATCH] kernel/params.c: fix the module name length in param_sysfs_builtin

2008-01-21 Thread Denis Cheng
ed in linux/module.h, I think this is enough for module names: #define MODULE_NAME_LEN (64 - sizeof(unsigned long)) Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- kernel/params.c |8 +++- 1 files changed, 3 insertions(+), 5 deletions(-) diff --git a/kernel/params.c b/kernel/par

[PATCH] kernel/params.c: fix the module name length in param_sysfs_builtin

2008-01-21 Thread Denis Cheng
think this is enough for module names: #define MODULE_NAME_LEN (64 - sizeof(unsigned long)) Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- kernel/params.c |8 +++- 1 files changed, 3 insertions(+), 5 deletions(-) diff --git a/kernel/params.c b/kernel/params.c index 7686417..a08

[PATCH] PROC_FS: get and set the smp affinity of tasks by read-write /proc//smp_affinity

2008-01-03 Thread Denis Cheng
<[EMAIL PROTECTED]> Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/proc/base.c | 57 1 files changed, 57 insertions(+), 0 deletions(-) diff --git a/fs/proc/base.c b/fs/proc/base.c index 7411bfb..ca0cbc2 100644 --- a/fs

[PATCH] 3W RAID drivers: memset not needed in probe

2007-12-08 Thread Denis Cheng
the memory return from scsi_host_alloc is alloced by kzalloc, which is already zero initilized, so memset not needed. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/scsi/3w-9xxx.c |2 -- drivers/scsi/3w-.c |2 -- 2 files changed, 0 insertions(+), 4 deletions(-)

[PATCH 20/20] net/iucv/iucv.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-06 Thread Denis Cheng
these three list_head are all local variables, but can also use LIST_HEAD. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/iucv/iucv.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/net/iucv/iucv.c b/net/iucv/iucv.c index 7698f6c..f13fe88 100644 --- a/ne

[PATCH 18/20] drivers/char/ipmi/ipmi_msghandler.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-06 Thread Denis Cheng
these three list_head are all local variables, but can also use LIST_HEAD. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/char/ipmi/ipmi_msghandler.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/char/ipmi/ipmi_msghandler.c b/drivers/cha

[PATCH 19/20] drivers/dma/iop-adma.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-06 Thread Denis Cheng
these three list_head are all local variables, but can also use LIST_HEAD. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/dma/iop-adma.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/dma/iop-adma.c b/drivers/dma/iop-adma.c index e5c62b7..b

[PATCH 17/20] net/xfrm/xfrm_state.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-06 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/xfrm/xfrm_state.c |2 +- 1 files changed, 1 insertions(+), 1 del

[PATCH 16/20] net/x25/: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-06 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/x25/x25_forward.c |2 +- net/x25/x25_link.c|2 +- n

[PATCH 15/20] net/lapb/lapb_iface.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-06 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/lapb/lapb_iface.c |2 +- 1 files changed, 1 insertions(+), 1 del

[PATCH 14/20] net/ipv4/cipso_ipv4.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-06 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/ipv4/cipso_ipv4.c |2 +- 1 files changed, 1 insertions(+), 1 del

[PATCH 13/20] net/core/dev.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-06 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/core/dev.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-)

[PATCH 12/20] fs/reiserfs/xattr.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-06 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/reiserfs/xattr.c |2 +- 1 files changed, 1 insertions(+), 1 del

[PATCH 11/20] fs/afs/cell.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-06 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/afs/cell.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-)

[PATCH 10/20] drivers/scsi/ipr.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-06 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/scsi/ipr.c |2 +- 1 files changed, 1 insertions(+), 1 del

[PATCH 09/20] drivers/s390/: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-06 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/s390/block/dcssblk.c |2 +- drivers/s390/char/raw3270.c

[PATCH 08/20] drivers/net/wan/lapbether.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-05 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/net/wan/lapbether.c |2 +- 1 files changed, 1 insertions

[PATCH 07/20] drivers/edac/: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-05 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/edac/edac_device.c |2 +- drivers/edac/edac_mc.c |2 +- d

[PATCH 07/20] drivers/edac/: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-05 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/edac/edac_device.c |2 +- drivers/edac/edac_mc.c |2 +- d

[PATCH 06/20] drivers/char: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-05 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/char/hvc_console.c |2 +- drivers/char/

[PATCH 05/20] crypto/async_tx/async_tx.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-05 Thread Denis Cheng
single list_head variable initialized with LIST_HEAD_INIT could almost always can be replaced with LIST_HEAD declaration, this shrinks the code and looks better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- crypto/async_tx/async_tx.c |3 +-- 1 files changed, 1 insertions

[PATCH 04/20] arch/x86/mm/pageattr_32.c: use LIST_HEAD instead of LIST_HEAD_INIT

2007-12-05 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- arch/x86/mm/pageattr_32.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/x86/mm/pageattr_32.c b/arch/x86/mm/pageattr_32.c index 260073c..f223cc0 100644 --- a/arch/x86/mm/pageattr_32.c +++ b/arch/x86/mm/pagea

[PATCH] drivers/base: use LIST_HEAD instead of call to INIT_LIST_HEAD in __init

2007-12-05 Thread Denis Cheng
LIST_HEAD has been widely used, so switch to this simpler method. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/base/attribute_container.c |9 + drivers/base/base.h|1 - drivers/base/init.c|1 - 3 files changed, 1 insertions(

[PATCH] mm/backing-dev.c: fix percpu_counter_destroy call bug in bdi_init

2007-12-02 Thread Denis Cheng
e int i is enough, int j is not needed. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- mm/backing-dev.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/mm/backing-dev.c b/mm/backing-dev.c index b0ceb29..e8644b1 100644 --- a/mm/backing-dev.c +++ b/mm/backing-dev.c

[PATCH] [RESEND] crypto test: use print_hex_dump from kernel.h instead

2007-11-26 Thread Denis Cheng
Cc: Randy Dunlap <[EMAIL PROTECTED]> Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- this is against the lastest cryptodev tree. crypto/tcrypt.c |9 - 1 files changed, 4 insertions(+), 5 deletions(-) diff --git a/crypto/tcrypt.c b/crypto/tcrypt.c index 1e12b86..ae

[PATCH] [RESEND] crypto test: use print_hex_dump from instead

2007-11-26 Thread Denis Cheng
ap <[EMAIL PROTECTED]> Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- crypto/tcrypt.c | 30 -- 1 files changed, 16 insertions(+), 14 deletions(-) diff --git a/crypto/tcrypt.c b/crypto/tcrypt.c index 24141fb..665aa87 100644 --- a/crypto/tcrypt.c +++ b/cry

[PATCH 2/2] ide-scsi: use print_hex_dump from

2007-11-25 Thread Denis Cheng
these utilities implemented in lib/hexdump.c are more handy, please use this. Cc: Randy Dunlap <[EMAIL PROTECTED]> Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- there are still much other private hexdump implementations in the source, which reinvent the wheel, we can find

[PATCH 1/2] crypto test: use print_hex_dump from

2007-11-25 Thread Denis Cheng
these utilities implemented in lib/hexdump.c are more handy, please use this. Cc: Randy Dunlap <[EMAIL PROTECTED]> Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- crypto/tcrypt.c | 21 +++-- 1 files changed, 7 insertions(+), 14 deletions(-) diff --git a/crypt

[PATCH] SLUB: killed the unused "end" variable

2007-11-12 Thread Denis Cheng
Since the macro "for_each_object" introduced, the "end" variable becomes unused anymore. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- mm/slub.c |2 -- 1 files changed, 0 insertions(+), 2 deletions(-) diff --git a/mm/slub.c b/mm/slub.c index 84f59fd..9acb

[PATCH] [sysfs]: make readlink result shorter when the symlink and its target shared some base sysfs subdirectory

2007-10-31 Thread Denis Cheng
000448 Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/sysfs/symlink.c | 38 +++--- 1 files changed, 27 insertions(+), 11 deletions(-) diff --git a/fs/sysfs/symlink.c b/fs/sysfs/symlink.c index 3eac20c..230a925 100644 --- a/fs/sysfs/symlink.c +++ b/fs/sys

[PATCH] [net/ipv4]: fib_seq_show function adjustment to get a more sensable output of /proc/net/route

2007-10-22 Thread Denis Cheng
0 0 00FF0 0 0 eth00101A8C000030 0 0 0 0 0 Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/ipv4/fib_hash.c | 15 +++ 1 files changed, 7 insertions(+),

[PATCH] fs/ocfs2/: removed unneeded initial value and function's return value

2007-09-26 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/ocfs2/super.c | 17 - 1 files changed, 8 insertions(+), 9 deletions(-) diff --git a/fs/ocfs2/super.c b/fs/ocfs2/super.c index c034b51..b98ec12 100644 --- a/fs/ocfs2/super.c +++ b/fs/ocfs2/super.c @@ -105,7 +105,7 @@

[PATCH] fs/dcache.c: revamped to use a more straightforward and efficient way to operate

2007-09-25 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/dcache.c | 39 ++- 1 files changed, 18 insertions(+), 21 deletions(-) diff --git a/fs/dcache.c b/fs/dcache.c index 678d39d..05ed44a 100644 --- a/fs/dcache.c +++ b/fs/dcache.c @@ -1138,29 +1

[PATCH] vfs: use the predefined d_unhashed inline function instead

2007-09-23 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/dcache.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/fs/dcache.c b/fs/dcache.c index 678d39d..62e1800 100644 --- a/fs/dcache.c +++ b/fs/dcache.c @@ -1546,7 +1546,7 @@ static void d_move_locked(struct

[PATCH 2/2] drivers/net/: all drivers/net/ cleanup with ARRAY_SIZE

2007-09-02 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/net/apne.c |2 +- drivers/net/arm/am79c961a.c|2 +- drivers/net/atarilance.c |2 +- drivers/net/atl1/atl1_hw.c |2 +- drivers/net/

[PATCH 1/2] net/: all net/ cleanup with ARRAY_SIZE

2007-09-02 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/atm/proc.c |2 +- net/decnet/dn_dev.c |2 +- net/ipv4/af_inet.c |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/net/atm/proc.c b/net/atm/proc.c index 99fc1fe..a3e52ff 100644 --- a/net/atm/proc.c

[PATCH] net/ipv4/af_inet.c: use ARRAY_SIZE macro from kernel.h instead

2007-09-01 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/ipv4/af_inet.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/net/ipv4/af_inet.c b/net/ipv4/af_inet.c index e681034..d5e8b67 100644 --- a/net/ipv4/af_inet.c +++ b/net/ipv4/af_inet.c @@ -939,7 +939,7 @@ static

[PATCH 3/3] netlink: use a statically allocated nl_table instead

2007-09-01 Thread Denis Cheng
if the table is always fixed size with MAX_LINKS entries, why not use a statically allocated table straightforwardly? Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/netlink/af_netlink.c | 11 ++- 1 files changed, 2 insertions(+), 9 deletions(-) diff --git a/net/n

[PATCH 2/3] netlink: the temp variable name max is ambiguous

2007-09-01 Thread Denis Cheng
with the macro max provided by , so changed its name to a more proper one: limit Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/netlink/af_netlink.c | 12 ++-- 1 files changed, 6 insertions(+), 6 deletions(-) diff --git a/net/netlink/af_netlink.c b/net/netlink/af_net

[PATCH 1/3] netlink: use the macro min(x,y) provided by instead

2007-09-01 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/netlink/af_netlink.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/net/netlink/af_netlink.c b/net/netlink/af_netlink.c index 5681ce3..8bb14e3 100644 --- a/net/netlink/af_netlink.c +++ b/net/netlink/af_net

[PATCH] netlink: use container_of instead

2007-08-28 Thread Denis Cheng
this could make future redesign of struct netlink_sock easier. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/netlink/af_netlink.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/net/netlink/af_netlink.c b/net/netlink/af_netlink.c index 5681ce3..a78d962

[PATCH 1/2] Documentation: fix type error

2007-08-27 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- .../filesystems/ramfs-rootfs-initramfs.txt |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/Documentation/filesystems/ramfs-rootfs-initramfs.txt b/Documentation/filesystems/ramfs-rootfs-initramfs.txt

[PATCH 2/2] Documentation: add entries to filesystems/00-INDEX for several untracked files

2007-08-27 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- Documentation/filesystems/00-INDEX |8 1 files changed, 8 insertions(+), 0 deletions(-) diff --git a/Documentation/filesystems/00-INDEX b/Documentation/filesystems/00-INDEX index 5717858..9e2341e 100644 --- a/Documen

[PATCH] [arch/i386/boot]: for better readability in clearing BSS

2007-08-25 Thread Denis Cheng
/compressed/head.S arch/i386/kernel/head.S but their label names in .bss section are different, when reading these files, they are apt to confuse, this patch kept them in the same style of clearing. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- arch/i386/boot/compressed/head.S

[PATCH] vfs: use list_for_each_entry instead

2007-08-15 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/super.c |7 ++- 1 files changed, 2 insertions(+), 5 deletions(-) diff --git a/fs/super.c b/fs/super.c index fc8ebed..f287c15 100644 --- a/fs/super.c +++ b/fs/super.c @@ -334,15 +334,12 @@ struct super_block *sget(

[PATCH] gfs2: better code for translating characters

2007-08-12 Thread Denis Cheng
the original code could work, but I think this code could work better. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/gfs2/ops_fstype.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/fs/gfs2/ops_fstype.c b/fs/gfs2/ops_fstype.c index cf5aa50..b9a7759

[PATCH] net/sched: mark "NET_CLS_RSVP6 depends on IPV6"

2007-07-31 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/sched/Kconfig |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/net/sched/Kconfig b/net/sched/Kconfig index 8a74cac..5d3749c 100644 --- a/net/sched/Kconfig +++ b/net/sched/Kconfig @@ -300,6 +300,7 @@ config NET_CL

[PATCH 2/3] fs/gfs2: use the declaration of gfs2_dops in the header file instead

2007-07-31 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/gfs2/ops_fstype.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/fs/gfs2/ops_fstype.c b/fs/gfs2/ops_fstype.c index cf5aa50..9a5e840 100644 --- a/fs/gfs2/ops_fstype.c +++ b/fs/gfs2/ops_fstype.c @@ -28,6

[PATCH 3/3] fs/gfs2: also mark struct {dentry,vm}_operations const

2007-07-31 Thread Denis Cheng
these struct *_operations are all method tables, thus should be const. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/gfs2/ops_dentry.c |3 +-- fs/gfs2/ops_dentry.h |2 +- fs/gfs2/ops_vm.c |4 ++-- fs/gfs2/ops_vm.h |4 ++-- include/linux/dcache.h

[PATCH 1/3] fs/gfs2: mark struct *_operations const

2007-07-31 Thread Denis Cheng
these struct *_operations are all method tables, thus should be const. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/gfs2/eaops.c |8 fs/gfs2/eaops.h |4 ++-- fs/gfs2/glock.c |2 +- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/fs/gfs2/eaops.

[PATCH] fs/gfs2: mark struct *_operations const

2007-07-30 Thread Denis Cheng
these struct *_operations are all method tables, thus should be const. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/gfs2/eaops.c|8 fs/gfs2/eaops.h|4 ++-- fs/gfs2/glock.c|2 +- fs/gfs2/ops_dentry.c |3 +-- fs/gfs2/ops_dentry.h

[PATCH] gfs2/locking/dlm: use an temp variable to reduce a spin_unlock

2007-07-25 Thread Denis Cheng
this is more clear. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/gfs2/locking/dlm/plock.c | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/fs/gfs2/locking/dlm/plock.c b/fs/gfs2/locking/dlm/plock.c index fba1f1d..1f7b038 100644 --- a/fs/gfs2/locki

[PATCH 1/1] fs: remove the unused mempages parameter

2007-07-23 Thread Denis Cheng
Since the mempages parameter is actually not used, they should be removed. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- If applied, now there is only files_init use the mempages parameter, files_init(mempages); but i don't think the adaptation to mempages in files_ini

[PATCH] fs/fuse/dev.c: use zero_user_page instead

2007-07-20 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- I'm not very sure zero_user_page is correctly used here, so please feel free to give comments. and why here it uses KM_USER1 not KM_USER0, What are the differences? fs/fuse/dev.c |8 +++- 1 files changed, 3 insertions(+),

[PATCH] fs: use kmem_cache_zalloc instead

2007-07-19 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- fs/file_table.c |3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/fs/file_table.c b/fs/file_table.c index d17fd69..37453ff 100644 --- a/fs/file_table.c +++ b/fs/file_table.c @@ -102,12 +102,11 @@ struc

[PATCH 2/2] nbd: change a parameter's type to remove a memcpy call

2007-07-19 Thread Denis Cheng
ransformed accordingly. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/block/nbd.c |7 ++- 1 files changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c index 86639c0..a4d8508 100644 --- a/drivers/block/nbd.c +++ b/drivers/block/

[PATCH 1/2] nbd: use list_for_each_entry_safe to make it more consolidated and readable

2007-07-19 Thread Denis Cheng
Thus the traverse of the loop may delete nodes, use the safe version. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- drivers/block/nbd.c |6 ++ 1 files changed, 2 insertions(+), 4 deletions(-) diff --git a/drivers/block/nbd.c b/drivers/block/nbd.c index c129510..86639c0 100644

[PATCH 2/2] net/core: some functions' definition order adjustment for readability

2007-07-18 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/core/dev.c | 407 1 files changed, 201 insertions(+), 206 deletions(-) diff --git a/net/core/dev.c b/net/core/dev.c index 16842af..ee567dd 100644 --- a/net/core/dev.c +++ b/ne

[PATCH 1/2] net/core: merge the content of dev_mcast.c into dev.c

2007-07-18 Thread Denis Cheng
- removed three function declarations from header file to mark them static, - reduced one file Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- this one is just merging by concatenating, and I'll try to adjust some function definitions' order to make it more readable.

[PATCH 3/3] [net/core] move __dev_addr_discard adjacent to dev_addr_discard for readability

2007-07-17 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/core/dev.c | 28 ++-- 1 files changed, 14 insertions(+), 14 deletions(-) diff --git a/net/core/dev.c b/net/core/dev.c index 17c9cbd..6357f54 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -2715,20 +

[PATCH 2/3] [net/core] merge dev_unicast_discard and dev_mc_discard into one

2007-07-17 Thread Denis Cheng
this two functions could share the dev->_xmit_lock acquired context. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/core/dev.c | 16 1 files changed, 4 insertions(+), 12 deletions(-) diff --git a/net/core/dev.c b/net/core/dev.c index 3ba63aa..17c9cbd 100644

[PATCH 1/3] [net/core] move dev_mc_discard from dev_mcast.c to dev.c

2007-07-17 Thread Denis Cheng
), they should merged into one to eliminate duplicates in acquiring and releasing the dev->_xmit_lock, this would be done in my following patch. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- include/linux/netdevice.h |2 -- net/core/dev.c| 14 +-

[PATCH 3/4] [NET] merge dev_unicast_discard and dev_mc_discard into one

2007-07-17 Thread Denis Cheng
this two functions can share the dev->_xmit_lock acquired context. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/core/dev.c | 16 1 files changed, 4 insertions(+), 12 deletions(-) diff --git a/net/core/dev.c b/net/core/dev.c index 3ba63aa..17c9cbd 100644

[PATCH 4/4] [NET] move __dev_addr_discard adjacent to dev_addr_discard for readability

2007-07-17 Thread Denis Cheng
Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/core/dev.c | 28 ++-- 1 files changed, 14 insertions(+), 14 deletions(-) diff --git a/net/core/dev.c b/net/core/dev.c index 17c9cbd..6357f54 100644 --- a/net/core/dev.c +++ b/net/core/dev.c @@ -2715,20 +

[PATCH 1/4] [NET] move dev_mc_discard from dev_mcast.c to dev.c

2007-07-17 Thread Denis Cheng
mechanism that: (netif_tx_lock_bh / __dev_addr_discard / netif_tx_unlock_bh), they should merged into one to eliminate duplicates in acquiring and releasing the dev->_xmit_lock, this would be done in my following patch. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- include/linux/netdevice

[PATCH 2/4] [NET] make non-global function __dev_addr_discard static

2007-07-17 Thread Denis Cheng
and remove its declaration in header file include/linux/netdevice.h Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- include/linux/netdevice.h |1 - net/core/dev.c|2 +- 2 files changed, 1 insertions(+), 2 deletions(-) diff --git a/include/linux/netdevice.h b/include

[PATCH 2/2] add static delaration and init_module fixes

2007-07-15 Thread Denis Cheng
As stated by kernel janitors TODO list: Remove unneeded historic code / New API conversions. - get rid of init_module / cleanup_module and with the __init / __exit code section declarations, the #ifdef MODULE #endif is not needed any more. Signed-off-by: Denis Cheng <[EM

[PATCH] replace kmem_cache_alloc with kmem_cache_zalloc to remove some following zero initializations.

2007-07-12 Thread Denis Cheng
>From 4d87e14b67890f06885a76b5792ca034de2e9d06 Mon Sep 17 00:00:00 2001 From: Denis Cheng <[EMAIL PROTECTED]> Date: Thu, 12 Jul 2007 11:53:58 +0800 Subject: [PATCH] replace kmem_cache_alloc with kmem_cache_zalloc to remove some following zero initializations. Signed-off-by: Denis Chen

[PATCH] trivial: the memset operation on a automatic array variable should be optimized out by data initialization

2007-06-22 Thread Denis Cheng
From: Denis Cheng <[EMAIL PROTECTED]> the explicit memset call could be optimized out by data initialization, thus all the fill working can be done by the compiler implicitly. and C standard guaranteed all the unspecified data field initialized to zero. Signed-off-by: Denis Cheng &

[PATCH] trivial: the memset operation on a automatic array variable should be optimized out by data initialization

2007-06-18 Thread Denis Cheng
From: Denis Cheng <[EMAIL PROTECTED]> the explicit memset call could be optimized out by data initialization, thus all the fill working can be done by the compiler implicitly. Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- Is there some comments on this? --- arch/x86_64/mm

[PATCH] the memset operation on a automatic array variable should be optimized out by data initialization

2007-06-18 Thread Denis Cheng
Is there some comments on this? Signed-Off-By: Denis Cheng <[EMAIL PROTECTED]> -- --- arch/x86_64/mm/init.c.orig 2007-06-07 10:08:04.0 +0800 +++ arch/x86_64/mm/init.c 2007-06-18 14:43:15.0 +0800 @@ -406,8 +406,7 @@ void __cpuinit zap_low_mappings(int cpu) #

the memset operation on a automatic array variable can be removed by data Initialization

2007-06-18 Thread Denis Cheng
Signed-Off-By: Denis Cheng <[EMAIL PROTECTED]> --- arch/x86_64/mm/init.c.orig 2007-06-07 10:08:04.0 +0800 +++ arch/x86_64/mm/init.c 2007-06-18 14:43:15.0 +0800 @@ -406,8 +406,7 @@ void __cpuinit zap_low_mappings(int cpu) #ifndef CONFIG_NUMA void __init paging_ini

the memset operation on a automatic array variable can be removed by data Initialization

2007-06-18 Thread Denis Cheng
Signed-Off-By: Denis Cheng <[EMAIL PROTECTED]> --- arch/x86_64/mm/init.c 2007-06-07 10:08:04.0 +0800 +++ /tmp/init.c 2007-06-18 14:43:15.0 +0800 @@ -406,8 +406,7 @@ void __cpuinit zap_low_mappings(int cpu) #ifndef CONFIG_NUMA void __init paging_ini

[PATCH] since the definition of dst_discard_in and dst_discard_out are the same,

2007-06-01 Thread Denis Cheng
they should merged into one Signed-off-by: Denis Cheng <[EMAIL PROTECTED]> --- net/core/dst.c | 17 - 1 files changed, 4 insertions(+), 13 deletions(-) diff --git a/net/core/dst.c b/net/core/dst.c index 764bccb..c6a0587 100644 --- a/net/core/dst.c +++ b/net/core