[PATCH 3/9] net: fix trivial typos

2015-10-18 Thread Geliang Tang
s/regsiter/register/ Signed-off-by: Geliang Tang --- drivers/net/ethernet/amd/amd8111e.h | 2 +- drivers/net/ethernet/atheros/atl1c/atl1c_hw.c | 2 +- drivers/net/ethernet/intel/igb/e1000_phy.c| 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net

[PATCH 9/9] usb: gadget: fix a trivial typo

2015-10-18 Thread Geliang Tang
s/regsiter/register/ Signed-off-by: Geliang Tang --- drivers/usb/gadget/udc/pch_udc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/usb/gadget/udc/pch_udc.c b/drivers/usb/gadget/udc/pch_udc.c index 3181fc9..7a04157 100644 --- a/drivers/usb/gadget/udc/pch_udc.c

[PATCH 6/9] clk: keystone: fix a trivial typo

2015-10-18 Thread Geliang Tang
s/regsiter/register/ Signed-off-by: Geliang Tang --- drivers/clk/keystone/pll.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/clk/keystone/pll.c b/drivers/clk/keystone/pll.c index 3f553d0..a26ba21 100644 --- a/drivers/clk/keystone/pll.c +++ b/drivers/clk/keystone

[PATCH 5/9] staging: wilc1000: fix trivial typos

2015-10-18 Thread Geliang Tang
s/regsiter/register/ Signed-off-by: Geliang Tang --- drivers/staging/wilc1000/linux_wlan.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wilc1000/linux_wlan.c b/drivers/staging/wilc1000/linux_wlan.c index ccab89f..6dc5faf 100644 --- a/drivers/staging

[PATCH 2/9] scsi: fix trivial typos

2015-10-18 Thread Geliang Tang
s/regsiter/register/ Signed-off-by: Geliang Tang --- drivers/scsi/isci/registers.h | 4 ++-- drivers/scsi/mpt2sas/mpt2sas_base.h | 2 +- drivers/scsi/mpt3sas/mpt3sas_base.h | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/scsi/isci/registers.h b/drivers/scsi

[PATCH 4/9] dma: fix a trivial typo

2015-10-18 Thread Geliang Tang
s/regsiter/register/ Signed-off-by: Geliang Tang --- drivers/dma/ste_dma40.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c index 750d1b3..dd3e7ba 100644 --- a/drivers/dma/ste_dma40.c +++ b/drivers/dma/ste_dma40.c @@ -2907,7

[PATCH 1/9] drm: fix trivial typos

2015-10-18 Thread Geliang Tang
s/regsiter/register/ Signed-off-by: Geliang Tang --- drivers/gpu/drm/amd/include/atombios.h | 2 +- drivers/gpu/drm/radeon/cayman_blit_shaders.c| 2 +- drivers/gpu/drm/radeon/evergreen_blit_shaders.c | 2 +- drivers/gpu/drm/radeon/r600_blit_shaders.c | 2 +- 4 files changed, 4

[PATCH 1/3] staging: comedi: make ni_tio_has_gate2_registers return boolean

2015-10-18 Thread Geliang Tang
This patch makes ni_tio_has_gate2_registers return boolean, since this function only uses either one or zero as its return value. Signed-off-by: Geliang Tang --- drivers/staging/comedi/drivers/ni_tio.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging

[PATCH 3/3] staging: xgifb: make XGIfb_has_VB return boolean

2015-10-18 Thread Geliang Tang
This patch makes XGIfb_has_VB return boolean, since this function only uses either one or zero as its return value. Signed-off-by: Geliang Tang --- drivers/staging/xgifb/XGI_main_26.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/staging/xgifb/XGI_main_26.c

[PATCH 2/3] staging: lustre: make sptlrpc_flavor_has_bulk return boolean

2015-10-18 Thread Geliang Tang
This patch makes sptlrpc_flavor_has_bulk return boolean, since this function only uses either one or zero as its return value. Signed-off-by: Geliang Tang --- drivers/staging/lustre/lustre/include/lustre_sec.h | 2 +- drivers/staging/lustre/lustre/ptlrpc/sec.c | 6 +++--- 2 files

[PATCH v2 2/2] staging: IB/ipath: remove sched.h header

2015-10-18 Thread Geliang Tang
sched.h header in ipath_*.c is now unnecessary, since I have added sched.h in ipath_kernel.h. So remove it. Signed-off-by: Geliang Tang --- Changes in v2: - split it into two patches. --- drivers/staging/rdma/ipath/ipath_driver.c | 1 - drivers/staging/rdma/ipath/ipath_intr.c| 1

[PATCH v2 1/2] staging: IB/ipath: use TASK_COMM_LEN in ipath_portdata

2015-10-18 Thread Geliang Tang
Use comm[TASK_COMM_LEN] instead of comm[16] in ipath_kernel.h. And add sched.h header in it. Signed-off-by: Geliang Tang --- Changes in v2: - split it into two patches. --- drivers/staging/rdma/ipath/ipath_kernel.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers

[PATCH v2 0/3] pstore: add pstore unregister

2015-10-18 Thread Geliang Tang
On Fri, Oct 16, 2015 at 08:50:42AM -0700, Kees Cook wrote: > On Fri, Oct 16, 2015 at 8:25 AM, Geliang Tang wrote: > > +static struct dentry *dir; > > Since this is no longer a local, it should probably be renamed > something more descriptive. > I renamed it pstore_ftrace_d

[PATCH v2 1/3] pstore: add vmalloc error check

2015-10-18 Thread Geliang Tang
When vmalloc is failed, let write_pmsg return -ENOMEM. Signed-off-by: Geliang Tang Acked-by: Kees Cook --- Changes in v2: - update commit log. --- fs/pstore/pmsg.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/fs/pstore/pmsg.c b/fs/pstore/pmsg.c index feb5dd2..5a2f05a 100644 --- a/fs

[PATCH v2 3/3] pstore: add pstore unregister

2015-10-18 Thread Geliang Tang
ystem. Signed-off-by: Geliang Tang --- Changes in v2: - Add pstore filesystem unregister. - update commit log. --- fs/pstore/Kconfig | 2 +- fs/pstore/Makefile | 6 +++--- fs/pstore/ftrace.c | 23 ++- fs/pstore/inode.c | 7 +++ fs/pstore/internal.h

[PATCH v2 2/3] pstore: add a helper function pstore_register_kmsg

2015-10-18 Thread Geliang Tang
Add a new wraper function pstore_register_kmsg to keep the consistency with other similar pstore_register_* functions. Signed-off-by: Geliang Tang --- Changes in v2: - update commit log. --- fs/pstore/platform.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/fs

[PATCH 1/3] pstore: add vmalloc error check

2015-10-16 Thread Geliang Tang
If vmalloc is failed, return -ENOMEM. Signed-off-by: Geliang Tang Acked-by: Kees Cook --- Send it again. --- fs/pstore/pmsg.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/fs/pstore/pmsg.c b/fs/pstore/pmsg.c index feb5dd2..5a2f05a 100644 --- a/fs/pstore/pmsg.c +++ b/fs/pstore/pmsg.c

[PATCH 2/3] pstore: add a helper function pstore_register_kmsg

2015-10-16 Thread Geliang Tang
Add a new wraper function pstore_register_kmsg to keep the consistency with the other similar pstore_register_* functions. Signed-off-by: Geliang Tang --- Send it again. --- fs/pstore/platform.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/fs/pstore/platform.c b/fs

[PATCH 3/3] pstore: add 'rmmod ramoops' support

2015-10-16 Thread Geliang Tang
y and free it. 5) Set console flags, make sure it can be registered again after be unregistered. Signed-off-by: Geliang Tang --- fs/pstore/ftrace.c | 15 ++- fs/pstore/internal.h | 4 fs/pstore/platform.c | 40 fs/pstore/p

[PATCH] pstore: add a helper function pstore_register_kmsg

2015-10-13 Thread Geliang Tang
Add a new wraper function pstore_register_kmsg to keep the consistency with the other similar pstore_register_* functions. Signed-off-by: Geliang Tang --- fs/pstore/platform.c | 7 ++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/fs/pstore/platform.c b/fs/pstore/platform.c

[PATCH] pstore: add vmalloc error check

2015-10-13 Thread Geliang Tang
If vmalloc is failed, return -ENOMEM. Signed-off-by: Geliang Tang --- fs/pstore/pmsg.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/fs/pstore/pmsg.c b/fs/pstore/pmsg.c index feb5dd2..5a2f05a 100644 --- a/fs/pstore/pmsg.c +++ b/fs/pstore/pmsg.c @@ -37,6 +37,8 @@ static ssize_t

Re: [PATCH] pcnet32: fix a logic error with pci_set_dma_mask

2015-10-13 Thread Geliang Tang
On Mon, Oct 12, 2015 at 05:38:14AM -0700, David Miller wrote: > From: Geliang Tang > Date: Fri, 9 Oct 2015 03:45:39 -0700 > > > pcnet32 can't work on my machine recently. It says "architecture > > does not support 32bit PCI busmaster DMA". There is a log

[PATCH v2] mISDN: use kstrdup() in dsp_pipeline_build

2015-10-12 Thread Geliang Tang
Use kstrdup instead of strlen-kmalloc-strcpy. Remove unneeded NULL test, it will be tested inside kstrdup. Remove 0 length string test, it has been tested in the caller of dsp_pipeline_build. Signed-off-by: Geliang Tang --- Changes in v2: - Remove unneeded NULL test. --- drivers/isdn/mISDN

[PATCH v3] misc: mic: replace kfree with put_device

2015-10-11 Thread Geliang Tang
Handle a failed device_register(), replace kfree() with put_device(), which will call cosm/mbus/scif_release_dev(). Signed-off-by: Geliang Tang --- Changes in v3: - Add the same change to cosm_bus.c Changes in v2: - Just refresh this patch. --- drivers/misc/mic/bus/cosm_bus.c | 2

[PATCH] misc: mic: use kstrdup() in cosm_sysfs

2015-10-10 Thread Geliang Tang
Use kstrdup instead of kmalloc and strncpy. Signed-off-by: Geliang Tang --- drivers/misc/mic/cosm/cosm_sysfs.c | 23 --- 1 file changed, 4 insertions(+), 19 deletions(-) diff --git a/drivers/misc/mic/cosm/cosm_sysfs.c b/drivers/misc/mic/cosm/cosm_sysfs.c index 29d6863

[PATCH] mISDN: use kstrdup() in dsp_pipeline_build

2015-10-10 Thread Geliang Tang
Use kstrdup instead of strlen-kmalloc-strcpy. Signed-off-by: Geliang Tang --- drivers/isdn/mISDN/dsp_pipeline.c | 9 ++--- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/drivers/isdn/mISDN/dsp_pipeline.c b/drivers/isdn/mISDN/dsp_pipeline.c index 8b1a66c..c60722d 100644 --- a

[PATCH] pcnet32: fix a logic error with pci_set_dma_mask

2015-10-09 Thread Geliang Tang
pcnet32 can't work on my machine recently. It says "architecture does not support 32bit PCI busmaster DMA". There is a logic error in it: pci_set_dma_mask() return 0 means return successfully. Signed-off-by: Geliang Tang --- drivers/net/ethernet/amd/pcnet32.c | 2 +- 1 file chang

[PATCH] IB/ipath: use TASK_COMM_LEN in ipath_portdata

2015-10-08 Thread Geliang Tang
Use comm[TASK_COMM_LEN] instead of comm[16]. Add linux/sched.h header in ipath_kernel.h, and remove linux/sched.h header from ipath_*.c which have included ipath_kernel.h. Signed-off-by: Geliang Tang --- drivers/staging/rdma/ipath/ipath_driver.c | 1 - drivers/staging/rdma/ipath

[PATCH] IB/hfi1: use TASK_COMM_LEN in hfi1_ctxtdata

2015-10-08 Thread Geliang Tang
Use comm[TASK_COMM_LEN] instead of comm[16]. Signed-off-by: Geliang Tang --- drivers/staging/rdma/hfi1/hfi.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rdma/hfi1/hfi.h b/drivers/staging/rdma/hfi1/hfi.h index 8ca171b..a35213e 100644 --- a/drivers/staging

Re: [lkp] [dm ioctl] e817ef9b89: kernel BUG at mm/slub.c:3517!

2015-10-08 Thread Geliang Tang
On Thu, Oct 08, 2015 at 10:32:36AM +0800, kernel test robot wrote: > FYI, we noticed the below changes on > > https://git.kernel.org/pub/scm/linux/kernel/git/device-mapper/linux-dm.git > dm-4.4 > commit e817ef9b89e5bbfdb2a67ca9896909f414b2af91 ("dm ioctl: use kvfree() in > free_params()") > Hi

Re: 30c2a1faaeb3db94fc92f79553cc72634aa3b218 broke cryptsetup on my machine

2015-10-08 Thread Geliang Tang
On Wed, Oct 07, 2015 at 10:27:27PM +0300, Giedrius Statkevičius wrote: > Hello, > I have a LUKS on LVM setup: /boot is unencrypted and everything is "hidden" in > /dev/sda2. After booting on linux-next and just after entering my password > cryptsetup segfaults and a stack trace is printed from the

[PATCH] dm ioctl: bugfix for kvfree() in free_params()

2015-10-08 Thread Geliang Tang
The ioctl parameter block consists of two parts, a dm_ioctl struct followed by a data buffer. Don't free the parameter block when the data buffer is empty. Fixes: 30c2a1f('dm ioctl: use kvfree() in free_params()') Signed-off-by: Geliang Tang --- drivers/md/dm-ioctl.c | 5 -

[tip:sched/core] sched/core: Drop unlikely behind BUG_ON()

2015-10-06 Thread tip-bot for Geliang Tang
Commit-ID: ce03e4137bb22fc560ad7a07cf4138ae2cd59f65 Gitweb: http://git.kernel.org/tip/ce03e4137bb22fc560ad7a07cf4138ae2cd59f65 Author: Geliang Tang AuthorDate: Mon, 5 Oct 2015 21:26:05 +0800 Committer: Ingo Molnar CommitDate: Tue, 6 Oct 2015 17:08:22 +0200 sched/core: Drop unlikely

[PATCH v2 3/3] mm/nommu: drop unlikely behind BUG_ON()

2015-10-05 Thread Geliang Tang
) For CONFIG_BUG and !HAVE_ARCH_BUG_ON cases, BUG_ON() contains an unlikely compiler flag already. Hence, we can drop unlikely behind BUG_ON(). [1] http://lkml.iu.edu/hypermail/linux/kernel/1101.3/02289.html Signed-off-by: Geliang Tang Acked-by: Davidlohr Bueso --- Changes in v2: - Just rewrit

[PATCH v2 1/3] scsi: drop unlikely behind BUG_ON()

2015-10-05 Thread Geliang Tang
) For CONFIG_BUG and !HAVE_ARCH_BUG_ON cases, BUG_ON() contains an unlikely compiler flag already. Hence, we can drop unlikely behind BUG_ON(). [1] http://lkml.iu.edu/hypermail/linux/kernel/1101.3/02289.html Signed-off-by: Geliang Tang Reviewed-by: Bart Van Assche --- Changes in v2: - Just re

[PATCH v2 2/3] sched/core: drop unlikely behind BUG_ON()

2015-10-05 Thread Geliang Tang
) For CONFIG_BUG and !HAVE_ARCH_BUG_ON cases, BUG_ON() contains an unlikely compiler flag already. Hence, we can drop unlikely behind BUG_ON(). [1] http://lkml.iu.edu/hypermail/linux/kernel/1101.3/02289.html Signed-off-by: Geliang Tang --- Changes in v2: - Just rewrite the commit log. --- k

[PATCH v2] ath6kl: drop unlikely behind WARN_ON()

2015-10-05 Thread Geliang Tang
WARN_ON() already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang --- Changes in v2: - change subject prefix to ath6kl. --- drivers/net/wireless/ath/ath6kl/cfg80211.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath6kl

Re: [PATCH 3/3] mm/nommu: drop unlikely behind BUG_ON()

2015-10-04 Thread Geliang Tang
On Sun, Oct 04, 2015 at 06:50:55PM -0700, Davidlohr Bueso wrote: > On Sun, 04 Oct 2015, Geliang Tang wrote: > > >BUG_ON() already contain an unlikely compiler flag. Drop it. > > > >Signed-off-by: Geliang Tang > > Acked-by: Davidlohr Bueso > > ...

[PATCH v2] misc: mic: replace kfree with put_device

2015-10-04 Thread Geliang Tang
Handle a failed device_register(), replace kfree() with put_device(), which will call mbus_release_dev() or scif_release_dev(). Signed-off-by: Geliang Tang --- Changes in v2: - Just refresh this patch. --- drivers/misc/mic/bus/mic_bus.c | 2 +- drivers/misc/mic/bus/scif_bus.c | 2 +- 2 files

[PATCH] btrfs: fix a comment typo

2015-10-04 Thread Geliang Tang
Just fix a typo in the code comment. Signed-off-by: Geliang Tang --- fs/btrfs/free-space-cache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/btrfs/free-space-cache.c b/fs/btrfs/free-space-cache.c index abe3a66..7332995 100644 --- a/fs/btrfs/free-space-cache.c +++ b/fs

[PATCH] IB/iser: fix a comment typo

2015-10-04 Thread Geliang Tang
Just fix a typo in the code comment. Signed-off-by: Geliang Tang --- drivers/infiniband/ulp/iser/iscsi_iser.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/infiniband/ulp/iser/iscsi_iser.c b/drivers/infiniband/ulp/iser/iscsi_iser.c index f58ff96..8f2f105 100644

[PATCH] isci: fix two comment typos

2015-10-04 Thread Geliang Tang
Just fix two typos in the code comment. Signed-off-by: Geliang Tang --- drivers/scsi/isci/request.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/isci/request.c b/drivers/scsi/isci/request.c index cfd0084..8fe106f 100644 --- a/drivers/scsi/isci/request.c

[PATCH] mwifiex: fix a comment typo

2015-10-04 Thread Geliang Tang
Just fix a typo in the code comment. Signed-off-by: Geliang Tang --- drivers/net/wireless/mwifiex/cfg80211.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/wireless/mwifiex/cfg80211.c b/drivers/net/wireless/mwifiex/cfg80211.c index 30cbafb..b7ac45f 100644 --- a

[PATCH] cfg80211: drop unlikely behind WARN_ON()

2015-10-04 Thread Geliang Tang
WARN_ON() already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang --- drivers/net/wireless/ath/ath6kl/cfg80211.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/wireless/ath/ath6kl/cfg80211.c b/drivers/net/wireless/ath/ath6kl/cfg80211

[PATCH] lockdep: drop unlikely behind DEBUG_LOCKS_WARN_ON()

2015-10-04 Thread Geliang Tang
DEBUG_LOCKS_WARN_ON() already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang --- kernel/locking/lockdep.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/locking/lockdep.c b/kernel/locking/lockdep.c index deae390..9d8cbb1 100644 --- a/kernel

[PATCH 1/3] scsi: drop unlikely behind BUG_ON()

2015-10-03 Thread Geliang Tang
BUG_ON() already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang --- drivers/scsi/scsi_lib.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/scsi_lib.c b/drivers/scsi/scsi_lib.c index f570b48..3b5faab 100644 --- a/drivers/scsi

[PATCH 2/3] sched/core: drop unlikely behind BUG_ON()

2015-10-03 Thread Geliang Tang
BUG_ON() already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang --- kernel/sched/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/sched/core.c b/kernel/sched/core.c index fe81929..a3518789 100644 --- a/kernel/sched/core.c +++ b/kernel

[PATCH 3/3] mm/nommu: drop unlikely behind BUG_ON()

2015-10-03 Thread Geliang Tang
BUG_ON() already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang --- mm/nommu.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/mm/nommu.c b/mm/nommu.c index 1e0f168..92be862 100644 --- a/mm/nommu.c +++ b/mm/nommu.c @@ -578,16 +578,16

[PATCH] RDMA/amso1100: use offset_in_page macro

2015-10-02 Thread Geliang Tang
Use offset_in_page macro instead of (addr & ~PAGE_MASK). Signed-off-by: Geliang Tang --- drivers/staging/rdma/amso1100/c2_alloc.c| 2 +- drivers/staging/rdma/amso1100/c2_provider.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rdma/amso

[PATCH] IB/ipath: use offset_in_page macro

2015-10-02 Thread Geliang Tang
Use offset_in_page macro instead of (addr & ~PAGE_MASK). Signed-off-by: Geliang Tang --- drivers/staging/rdma/ipath/ipath_user_sdma.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/rdma/ipath/ipath_user_sdma.c b/drivers/staging/rdma/i

[PATCH] IB/hfi1: use offset_in_page macro

2015-10-02 Thread Geliang Tang
Use offset_in_page macro instead of (addr & ~PAGE_MASK). Signed-off-by: Geliang Tang --- drivers/staging/rdma/hfi1/file_ops.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/staging/rdma/hfi1/file_ops.c b/drivers/staging/rdma/hfi1/file_ops.c index 9a7

[PATCH] KEYS: use kvfree() in add_key

2015-10-02 Thread Geliang Tang
There is no need to make a flag to tell that this memory is allocated by kmalloc or vmalloc. Just use kvfree to free the memory. Signed-off-by: Geliang Tang --- security/keys/keyctl.c | 8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/security/keys/keyctl.c b/security

[PATCH] dm ioctl: use kvfree() in free_params()

2015-10-02 Thread Geliang Tang
There is no need to make a flag to tell that this memory is allocated by kmalloc or vmalloc. Just use kvfree to free the memory. Signed-off-by: Geliang Tang --- drivers/md/dm-ioctl.c | 11 +-- 1 file changed, 1 insertion(+), 10 deletions(-) diff --git a/drivers/md/dm-ioctl.c b/drivers

[PATCH] misc: mic: use kstrdup() in mic_sysfs

2015-10-01 Thread Geliang Tang
Use kstrdup instead of kmalloc and strncpy. Signed-off-by: Geliang Tang --- drivers/misc/mic/host/mic_sysfs.c | 23 --- 1 file changed, 4 insertions(+), 19 deletions(-) diff --git a/drivers/misc/mic/host/mic_sysfs.c b/drivers/misc/mic/host/mic_sysfs.c index 6dd864e

[PATCH] pcmcia: use kstrdup() in pcmcia_device_query()

2015-10-01 Thread Geliang Tang
Use kstrdup instead of kmalloc and strncpy. Signed-off-by: Geliang Tang --- drivers/pcmcia/ds.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/pcmcia/ds.c b/drivers/pcmcia/ds.c index 0decee6..489ea10 100644 --- a/drivers/pcmcia/ds.c +++ b/drivers/pcmcia/ds.c

[PATCH v2] hsi: fix double kfree

2015-10-01 Thread Geliang Tang
When device_register() fails, kfree() is called in hsi_client_release(), hence there is no need to call kfree in err3 again. Signed-off-by: Geliang Tang --- Changes in v2: - Just to make myself clearer that what I did in this patch. --- drivers/hsi/hsi.c | 1 - 1 file changed, 1 deletion

[PATCH] PM / devfreq: fix double kfree

2015-10-01 Thread Geliang Tang
When device_register() fails, kfree(devfreq) is called already in devfreq_dev_release(), hence there is no need to call kfree(devfreq) in err_dev again. Signed-off-by: Geliang Tang --- drivers/devfreq/devfreq.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers

[tip:x86/apic] x86/irq: Drop unlikely before IS_ERR_OR_NULL

2015-10-01 Thread tip-bot for Geliang Tang
Commit-ID: a7e705af524d165fe7bc303aee82225c66734885 Gitweb: http://git.kernel.org/tip/a7e705af524d165fe7bc303aee82225c66734885 Author: Geliang Tang AuthorDate: Thu, 1 Oct 2015 10:55:29 +0800 Committer: Thomas Gleixner CommitDate: Thu, 1 Oct 2015 11:08:56 +0200 x86/irq: Drop unlikely

[PATCH 3/3] cxlflash: drop unlikely before IS_ERR_OR_NULL

2015-09-30 Thread Geliang Tang
IS_ERR_OR_NULL already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang --- drivers/scsi/cxlflash/superpipe.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/cxlflash/superpipe.c b/drivers/scsi/cxlflash/superpipe.c index f1b62ce

[PATCH 1/3] x86/irq: drop unlikely before IS_ERR_OR_NULL

2015-09-30 Thread Geliang Tang
IS_ERR_OR_NULL already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang --- arch/x86/kernel/irq_64.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kernel/irq_64.c b/arch/x86/kernel/irq_64.c index c767cf2..206d0b9 100644 --- a/arch/x86/kernel

[PATCH 2/3] Input: alps: drop unlikely before IS_ERR_OR_NULL

2015-09-30 Thread Geliang Tang
IS_ERR_OR_NULL already contain an unlikely compiler flag. Drop it. Signed-off-by: Geliang Tang --- drivers/input/mouse/alps.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/input/mouse/alps.c b/drivers/input/mouse/alps.c index 4d24686..b4f146a 100644 --- a/drivers

[PATCH] drm/i915: fix compiler warnings in intel_audio.c

2015-09-30 Thread Geliang Tang
audio_rate_need_prog(struct intel_crtc *crtc, ^ Signed-off-by: Geliang Tang --- drivers/gpu/drm/i915/intel_audio.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/i915/intel_audio.c b/drivers/gpu/drm/i915/intel_audio.c index baadd90..72d696b 100644 --- a/drivers

[PATCH] amd-xgbe: fix potential memory leak in xgbe-debugfs

2015-09-30 Thread Geliang Tang
Added kfree() to avoid the memory leak when debugfs_create_dir() fails. Signed-off-by: Geliang Tang --- drivers/net/ethernet/amd/xgbe/xgbe-debugfs.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/net/ethernet/amd/xgbe/xgbe-debugfs.c b/drivers/net/ethernet/amd/xgbe/xgbe-debugfs.c

[PATCH 1/2] misc: genwqe: fix a comment typo

2015-09-29 Thread Geliang Tang
Just fix a typo in the code comment. Signed-off-by: Geliang Tang --- drivers/misc/genwqe/card_base.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/misc/genwqe/card_base.h b/drivers/misc/genwqe/card_base.h index e735344..cb851c1 100644 --- a/drivers/misc/genwqe

[PATCH 2/2] libxfs: fix two comment typos

2015-09-29 Thread Geliang Tang
Just fix two typos in code comments. Signed-off-by: Geliang Tang --- fs/xfs/libxfs/xfs_btree.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/fs/xfs/libxfs/xfs_btree.c b/fs/xfs/libxfs/xfs_btree.c index f7d7ee7..1bd3532 100644 --- a/fs/xfs/libxfs/xfs_btree.c +++ b/fs/xfs

[PATCH] PM/hibernate: fix a comment typo

2015-09-29 Thread Geliang Tang
Just fix a typo in a function name in kerneldoc comments. Signed-off-by: Geliang Tang --- kernel/power/hibernate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/power/hibernate.c b/kernel/power/hibernate.c index 690f78f..b7342a2 100644 --- a/kernel/power/hibernate.c

[PATCH] ceph: fix a comment typo

2015-09-29 Thread Geliang Tang
Just fix a typo in the code comment. Signed-off-by: Geliang Tang --- fs/ceph/cache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/ceph/cache.c b/fs/ceph/cache.c index 834f9f3..a4766de 100644 --- a/fs/ceph/cache.c +++ b/fs/ceph/cache.c @@ -88,7 +88,7 @@ static uint16_t

[PATCH] net: macb: fix two typos

2015-09-29 Thread Geliang Tang
Just fix two typos in code comments. Signed-off-by: Geliang Tang --- drivers/net/ethernet/cadence/macb.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/net/ethernet/cadence/macb.h b/drivers/net/ethernet/cadence/macb.h index 6e1faea..866b128 100644 --- a/drivers

[PATCH] misc: mic: replace kfree with put_device

2015-09-29 Thread Geliang Tang
Handle a failed device_register(), replace kfree() with put_device(), which will call mbus_release_dev() or scif_release_dev(). Signed-off-by: Geliang Tang --- drivers/misc/mic/bus/mic_bus.c | 2 +- drivers/misc/mic/bus/scif_bus.c | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff

[PATCH] hsi: fix double kfree

2015-09-29 Thread Geliang Tang
Already 'kfree' in hsi_client_release(), no need to 'kfree' in err3. Signed-off-by: Geliang Tang --- drivers/hsi/hsi.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/hsi/hsi.c b/drivers/hsi/hsi.c index fe93712..35d631e 100644 --- a/drivers/hsi/hsi.c +++ b/drivers

[tip:perf/core] perf/core, perf/x86: Change needlessly global functions and a variable to static

2015-09-28 Thread tip-bot for Geliang Tang
Commit-ID: 18ab2cd3ee9d52dc64c5ae984146a261a328c4e8 Gitweb: http://git.kernel.org/tip/18ab2cd3ee9d52dc64c5ae984146a261a328c4e8 Author: Geliang Tang AuthorDate: Sun, 27 Sep 2015 23:25:50 +0800 Committer: Ingo Molnar CommitDate: Mon, 28 Sep 2015 08:09:52 +0200 perf/core, perf/x86

[PATCH 2/2] x86/cpu/cacheinfo: amd_get/set_l3_disable_slot should be static

2015-09-27 Thread Geliang Tang
Should it be static? Signed-off-by: Geliang Tang --- arch/x86/kernel/cpu/intel_cacheinfo.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/x86/kernel/cpu/intel_cacheinfo.c b/arch/x86/kernel/cpu/intel_cacheinfo.c index 6245a3c..e38d338 100644 --- a/arch/x86/kernel/c

[PATCH 1/2] x86/cpu/cacheinfo: num_cache_leaves should be static

2015-09-27 Thread Geliang Tang
Fixes the following sparse warning: arch/x86/kernel/cpu/intel_cacheinfo.c:160:33: warning: symbol 'num_cache_leaves' was not declared. Should it be static? Signed-off-by: Geliang Tang --- arch/x86/kernel/cpu/intel_cacheinfo.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) di

[PATCH 2/2] perf/core: nop_txn_flags should be static

2015-09-27 Thread Geliang Tang
Fixes the following sparse warning: kernel/events/core.c:7393:1: warning: symbol 'nop_txn_flags' was not declared. Should it be static? Signed-off-by: Geliang Tang --- kernel/events/core.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/kernel/events/core.

[PATCH 1/2] perf/core: fix should it be static warnings

2015-09-27 Thread Geliang Tang
e.c:7753:12: warning: symbol 'perf_init_event' was not declared. Should it be static? Signed-off-by: Geliang Tang --- kernel/events/core.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/kernel/events/core.c b/kernel/events/core.c index f87b434..f88e4ad 100644 --

[PATCH 1/2] selinux: ioctl_has_perm should be static

2015-09-27 Thread Geliang Tang
Fixes the following sparse warning: security/selinux/hooks.c:3242:5: warning: symbol 'ioctl_has_perm' was not declared. Should it be static? Signed-off-by: Geliang Tang --- security/selinux/hooks.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/security/selinux

[PATCH 2/2] smack: smk_ipv6_port_list should be static

2015-09-27 Thread Geliang Tang
Fixes the following sparse warning: security/smack/smack_lsm.c:55:1: warning: symbol 'smk_ipv6_port_list' was not declared. Should it be static? Signed-off-by: Geliang Tang --- security/smack/smack_lsm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/secu

[PATCH] crypto: blkcipher and ablkcipher should it be static

2015-09-27 Thread Geliang Tang
atic? Signed-off-by: Geliang Tang --- crypto/skcipher.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crypto/skcipher.c b/crypto/skcipher.c index dd5fc1b..7591928 100644 --- a/crypto/skcipher.c +++ b/crypto/skcipher.c @@ -91,7 +91,7 @@ static void crypto_exit_skcipher_

[PATCH] acpi/apei: apei_resources_all should be static

2015-09-27 Thread Geliang Tang
Fixes the following sparse warning: drivers/acpi/apei/apei-base.c:298:23: warning: symbol 'apei_resources_all' was not declared. Should it be static? Signed-off-by: Geliang Tang --- drivers/acpi/apei/apei-base.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dr

[PATCH] fs: namespace: to_mnt_ns should be static

2015-09-27 Thread Geliang Tang
Fixes the following sparse warning: fs/namespace.c:1654:22: warning: symbol 'to_mnt_ns' was not declared. Should it be static? Signed-off-by: Geliang Tang --- fs/namespace.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/namespace.c b/fs/namespace.c ind

[PATCH] kvm: x86: kvm_emulate_wbinvd_noskip should be static

2015-09-27 Thread Geliang Tang
Fixes the following sparse warning: arch/x86/kvm/x86.c:4492:5: warning: symbol 'kvm_emulate_wbinvd_noskip' was not declared. Should it be static? Signed-off-by: Geliang Tang --- arch/x86/kvm/x86.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/x86/kvm/x86.c

[tip:perf/urgent] perf/x86: Change test_aperfmperf() and test_intel() to static

2015-09-25 Thread tip-bot for Geliang Tang
Commit-ID: 7e5560a5648ab2bce7199c73b9c2a51b846f5541 Gitweb: http://git.kernel.org/tip/7e5560a5648ab2bce7199c73b9c2a51b846f5541 Author: Geliang Tang AuthorDate: Thu, 24 Sep 2015 04:48:53 -0700 Committer: Ingo Molnar CommitDate: Fri, 25 Sep 2015 09:42:40 +0200 perf/x86: Change

[PATCH] drm/i915: fix task reference leak in i915_debugfs.c

2015-09-25 Thread Geliang Tang
Leak a task reference in i915_ppgtt_info(), add put_task_struct() to fix it. Signed-off-by: Geliang Tang --- drivers/gpu/drm/i915/i915_debugfs.c | 8 ++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c

[PATCH] perf/x86: test_aperfmperf and test_intel should be static

2015-09-24 Thread Geliang Tang
Fixes the following sparse warnings: arch/x86/kernel/cpu/perf_event_msr.c:13:6: warning: symbol 'test_aperfmperf' was not declared. Should it be static? arch/x86/kernel/cpu/perf_event_msr.c:18:6: warning: symbol 'test_intel' was not declared. Should it be static? Signed-

[PATCH] drm: drm_atomic_crtc_get_property should be static

2015-09-24 Thread Geliang Tang
Fixes the following sparse warning: drivers/gpu/drm/drm_atomic.c:442:5: warning: symbol 'drm_atomic_crtc_get_property' was not declared. Should it be static? Signed-off-by: Geliang Tang --- drivers/gpu/drm/drm_atomic.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) di

[PATCH] staging: rdma: add a blank line after function

2015-09-21 Thread Geliang Tang
/verbs.c:1202: Signed-off-by: Geliang Tang --- drivers/staging/rdma/amso1100/c2_mq.c | 1 + drivers/staging/rdma/hfi1/file_ops.c | 1 + drivers/staging/rdma/hfi1/sdma.c | 1 + drivers/staging/rdma/hfi1/verbs.c | 1 + 4 files changed, 4 insertions(+) diff --git a/drivers/staging/rdma/amso1100

[PATCH] IB/hfi1: class_name_user() should be static

2015-09-21 Thread Geliang Tang
Fixes the following sparse warning: drivers/staging/rdma/hfi1/device.c:127:12: warning: symbol 'class_name_user' was not declared. Should it be static? Signed-off-by: Geliang Tang --- drivers/staging/rdma/hfi1/device.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) di

[PATCH] IB/hfi1: use kvfree() in sdma.c

2015-09-21 Thread Geliang Tang
Use kvfree() instead of open-coding it. Signed-off-by: Geliang Tang --- drivers/staging/rdma/hfi1/sdma.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/drivers/staging/rdma/hfi1/sdma.c b/drivers/staging/rdma/hfi1/sdma.c index aecd1a7..9c02a3d 100644 --- a/drivers

[PATCH 2/3] drm: fix kernel-doc warnings in drm_crtc.h

2015-09-17 Thread Geliang Tang
Fix the following 'make htmldocs' warning: .//include/drm/drm_crtc.h:929: warning: Excess struct/union/enum/typedef member 'base' description in 'drm_bridge' Signed-off-by: Geliang Tang --- include/drm/drm_crtc.h | 1 - 1 file changed, 1 deletion(-) diff --

[PATCH 3/3] hsi: fix kernel-doc warnings in hsi.h

2015-09-17 Thread Geliang Tang
claimed' description in 'hsi_client' .//include/linux/hsi/hsi.h:150: warning: Excess struct/union/enum/typedef member 'nb' description in 'hsi_client' Signed-off-by: Geliang Tang --- include/linux/hsi/hsi.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

[PATCH 1/3] media: dvb-core: fix kernel-doc warnings in dvbdev.h

2015-09-17 Thread Geliang Tang
x27; description in 'dvb_register_device' Signed-off-by: Geliang Tang --- drivers/media/dvb-core/dvbdev.h | 4 1 file changed, 4 deletions(-) diff --git a/drivers/media/dvb-core/dvbdev.h b/drivers/media/dvb-core/dvbdev.h index c61a4f0..1069a77 100644 --- a/drivers/media/dvb-c

[PATCH] drm/i915: fix kernel-doc warnings in intel_audio.c

2015-09-15 Thread Geliang Tang
l_audio_codec_disable' .//drivers/gpu/drm/i915/intel_audio.c:439: warning: No description found for parameter 'intel_encoder' .//drivers/gpu/drm/i915/intel_audio.c:439: warning: Excess function parameter 'encoder' description in 'intel_audio_codec_disable' Si

[PATCH] drm/i915: fix kernel-doc warnings in i915_gem.c

2015-09-15 Thread Geliang Tang
5/i915_gem.c:4962: warning: No description found for parameter 'old' .//drivers/gpu/drm/i915/i915_gem.c:4962: warning: No description found for parameter 'new' .//drivers/gpu/drm/i915/i915_gem.c:4962: warning: No description found for parameter 'frontbuffer_bits' Signe

[PATCH] drm/i915: fix kernel-doc warnings in i915_gem_gtt.c

2015-09-15 Thread Geliang Tang
27;length' .//drivers/gpu/drm/i915/i915_gem_gtt.c:818: warning: No description found for parameter 'new_pds' Signed-off-by: Geliang Tang --- drivers/gpu/drm/i915/i915_gem_gtt.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.

[PATCH] media: fix kernel-doc warnings in v4l2-dv-timings.h

2015-09-15 Thread Geliang Tang
imings.h:171: warning: No description found for parameter 'fmt' .//include/media/v4l2-dv-timings.h:184: warning: No description found for parameter 'hor_landscape' .//include/media/v4l2-dv-timings.h:184: warning: No description found for parameter 'vert_portrait' Signed

[PATCH] drm/i915: fix kernel-doc warnings in intel_lrc.c

2015-09-15 Thread Geliang Tang
ng_begin' .//drivers/gpu/drm/i915/intel_lrc.c:780: warning: Excess function parameter 'ctx' description in 'intel_logical_ring_begin' Signed-off-by: Geliang Tang --- drivers/gpu/drm/i915/intel_lrc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --g

[PATCH] spi: fix kernel-doc warnings in spi.h

2015-09-15 Thread Geliang Tang
Fix the following 'make htmldocs' warnings: .//include/linux/spi/spi.h:71: warning: No description found for parameter 'lock' .//include/linux/spi/spi.h:71: warning: Excess struct/union/enum/typedef member 'clock' description in 'spi_statistics' S

[PATCH] block: Fix compiler warnings in blk-merge.c

2015-09-06 Thread Geliang Tang
^ Signed-off-by: Geliang Tang --- block/blk-merge.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/block/blk-merge.c b/block/blk-merge.c index d088cff..5bd84c8 100644 --- a/block/blk-merge.c +++ b/block/blk-merge.c @@ -67,7 +67,7 @@ static struct bio *blk_bio_

[PATCH] staging: comedi: make "unsigned int __force *" shorter

2015-07-26 Thread Geliang Tang
Use macro "uip" instead of "unsigned int __force *", like "vuip" in arch/alpha/include/asm/core_cia.h. Signed-off-by: Geliang Tang --- drivers/staging/comedi/comedi_compat32.c | 3 +-- drivers/staging/comedi/comedi_compat32.h | 2 ++ drivers/staging/comedi/comed

[PATCH] staging: lustre: lnet: fix type warning in lib-socket.c

2015-06-27 Thread Geliang Tang
This patch fixes the following sparse warning: drivers/staging/lustre/lnet/lnet/lib-socket.c:175:29: warning: incorrect type in assignment (different address spaces) expected char [noderef] *ifcu_buf got char * Signed-off-by: Geliang Tang --- drivers/staging/lustre/lnet/lnet/lib-socket.c

<    2   3   4   5   6   7   8   >