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
,
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
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
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 -
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
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,
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
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 +
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 |
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 |
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
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
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 -
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
++
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
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
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
<[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
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(-)
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
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
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
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
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
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
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
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(-)
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
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(-)
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
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
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
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
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
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/
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
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
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(
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
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
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
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
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
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
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
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(+),
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 @@
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
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
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/
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
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
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
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
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
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
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
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
/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
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(
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
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
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
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
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.
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
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
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
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(+),
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
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/
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
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
- 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.
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 +
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
),
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 +-
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
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 +
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
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
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
>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
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 &
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
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)
#
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
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
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
89 matches
Mail list logo