[PATCH 1/1] staging: rtl8723au: Remove unnecessary OOM message

2015-03-06 Thread Quentin Lambert
This patch reduces the kernel size by removing error messages that duplicate the normal OOM message. A simplified version of the semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr) @@ identifier f,print,l; expression e; constant char[] c; @@ e = \(kzalloc\|kmalloc\|

[PATCH v2] staging:lustre:libcfs: Merge linux-proc.c into module.c

2015-03-06 Thread Matt Tyler
module.c was previously the sole exporter of symbols from linux-proc.c This patch removes the global symbols by merging the two files Signed-off-by: Matthew Tyler --- drivers/staging/lustre/lustre/libcfs/Makefile | 2 +- .../lustre/lustre/libcfs/linux/linux-proc.c| 577 ---

[PATCH v2] staging:lustre:libcfs: Merge linux-proc.c into module.c

2015-03-06 Thread Matthew Tyler
module.c was previously the sole exporter of symbols from linux-proc.c This patch removes the global symbols by merging the two files Signed-off-by: Matthew Tyler --- drivers/staging/lustre/lustre/libcfs/Makefile | 2 +- .../lustre/lustre/libcfs/linux/linux-proc.c| 577 ---

Re: [PATCH v3 0/1] drivers/bus: fsl-mc object allocator driver

2015-03-06 Thread Alexander Graf
On 06.03.15 02:35, J. German Rivera wrote: > This patch series introduces the object allocator driver for the > Freescale Management Complex (fsl-mc) of QorIQ Ls2 SoCs This patch > series is dependent on the patch series drivers/bus: "Freescale > Management Complex bus driver patch series Besides

Re: [PATCH v9 0/3] staging: fsl-mc: Freescale Management Complex bus driver patch series

2015-03-06 Thread Alexander Graf
On 06.03.15 02:29, J. German Rivera wrote: > This patch series introduces Linux support for the Freescale > Management Complex (fsl-mc) hardware. This patch series is dependent > on the patch series "ARM64: Add support for FSL's LS2085A SoC" > (http://thread.gmane.org/gmane.linux.ports.arm.kernel

Re: [PATCH v3 1/1] staging: fsl-mc: fsl-mc object allocator driver

2015-03-06 Thread Alexander Graf
On 06.03.15 02:35, J. German Rivera wrote: > The fsl-mc object allocator driver manages "allocatable" fsl-mc > objects such as DPBPs, DPMCPs and DPCONs. It provides services to > other fsl-mc drivers to allocate/deallocate these types of objects. > > Signed-off-by: J. German Rivera > Signed-off

Re: [PATCH 00/15] staging: comedi: cleanup NULL pointer checks

2015-03-06 Thread Ian Abbott
On 04/03/15 19:15, H Hartley Sweeten wrote: For aesthetics, change all the NULL comparisons from (foo == NULL) to (!foo) and (foo != NULL) to simply (foo). In addition, remove the NULL pointer checks of the 'cmd' in the s626 driver. The 'cmd' is the address of a member variable in a struct. It w

Re: [PATCH 15/15] staging: comedi: s626: remove unnecessary 'cmd' pointer checks

2015-03-06 Thread Ian Abbott
On 04/03/15 19:15, H Hartley Sweeten wrote: The local variable 'cmd' is a pointer to the address of a member variable of a struct. It will always be valid. Remove the unnecessary checks. Signed-off-by: H Hartley Sweeten Cc: Ian Abbott Cc: Greg Kroah-Hartman --- drivers/staging/comedi/driver

Re: [PATCH 0/4] staging: comedi: miscellaneous cleanup

2015-03-06 Thread Ian Abbott
On 05/03/15 20:21, H Hartley Sweeten wrote: Following are some miscellaneous cleanup patches I have been sitting on. If necessary I can break them up but they are pretty straight forward. H Hartley Sweeten (4): staging: comedi: drivers/*.c: fix common misspellings staging: comedi: drivers/

Re: [PATCH 1/1] staging: rtl8723au: Remove unnecessary OOM message

2015-03-06 Thread Jes Sorensen
Quentin Lambert writes: > This patch reduces the kernel size by removing error messages that duplicate > the normal OOM message. > > A simplified version of the semantic patch that finds this problem is as > follows: (http://coccinelle.lip6.fr) This patch removes useful warnings about what alloca

Re: [PATCH 1/1] staging: rtl8723au: Remove unnecessary OOM message

2015-03-06 Thread Julia Lawall
On Fri, 6 Mar 2015, Jes Sorensen wrote: > Quentin Lambert writes: > > This patch reduces the kernel size by removing error messages that duplicate > > the normal OOM message. > > > > A simplified version of the semantic patch that finds this problem is as > > follows: (http://coccinelle.lip6.fr

Re: [PATCH RESEND 0/3] memory_hotplug: hyperv: fix deadlock between memory adding and onlining

2015-03-06 Thread Michal Hocko
[Sorry for the late response] This is basically the same code posted by KY Srinivasan posted late last year (http://marc.info/?l=linux-mm&m=141782228129426&w=2). I had objections to the implementation http://marc.info/?l=linux-mm&m=141805109216700&w=2 On Thu 12-02-15 11:23:51, Vitaly Kuznetsov wr

Re: [PATCH 1/1] staging: rtl8723au: Remove unnecessary OOM message

2015-03-06 Thread Jes Sorensen
Julia Lawall writes: > On Fri, 6 Mar 2015, Jes Sorensen wrote: > >> Quentin Lambert writes: >> > This patch reduces the kernel size by removing error messages that >> > duplicate >> > the normal OOM message. >> > >> > A simplified version of the semantic patch that finds this problem is as >> >

Re: [PATCH 1/1] staging: rtl8723au: Remove unnecessary OOM message

2015-03-06 Thread Joe Perches
On Fri, 2015-03-06 at 11:08 -0500, Jes Sorensen wrote: > Julia Lawall writes: > > On Fri, 6 Mar 2015, Jes Sorensen wrote: > >> Quentin Lambert writes: > >> > This patch reduces the kernel size by removing error messages that > >> > duplicate > >> > the normal OOM message. > >> > A simplified ver

Re: [PATCH 1/1] staging: rtl8723au: Remove unnecessary OOM message

2015-03-06 Thread Jes Sorensen
Joe Perches writes: > On Fri, 2015-03-06 at 11:08 -0500, Jes Sorensen wrote: >> Julia Lawall writes: >> > On Fri, 6 Mar 2015, Jes Sorensen wrote: >> >> Quentin Lambert writes: >> >> > This patch reduces the kernel size by removing error messages >> >> > that duplicate >> >> > the normal OOM mess

Re: [PATCH 1/1] staging: rtl8723au: Remove unnecessary OOM message

2015-03-06 Thread Joe Perches
On Fri, 2015-03-06 at 14:43 -0500, Jes Sorensen wrote: > Joe Perches writes: > > On Fri, 2015-03-06 at 11:08 -0500, Jes Sorensen wrote: > >> Julia Lawall writes: > >> > On Fri, 6 Mar 2015, Jes Sorensen wrote: > >> >> Quentin Lambert writes: > >> >> > This patch reduces the kernel size by removin

[PATCH] iee80211:Remove no longer required strutucre defintion, _STA_QOS in rtl819x_Qos.h

2015-03-06 Thread Nicholas Krause
Removes the no longer required structure definition,_STA_QOS due to no functions using any structures of this type anymore. Signed-off-by: Nicholas Krause --- drivers/staging/rtl8192u/ieee80211/rtl819x_Qos.h | 48 1 file changed, 48 deletions(-) diff --git a/drivers/sta

Re: [PATCH] iee80211:Remove no longer required strutucre defintion, _STA_QOS in rtl819x_Qos.h

2015-03-06 Thread Giedrius Statkevičius
On 2015.03.06 23:20, Nicholas Krause wrote: > Removes the no longer required structure definition,_STA_QOS due to > no functions using any structures of this type anymore. > > Signed-off-by: Nicholas Krause > --- > drivers/staging/rtl8192u/ieee80211/rtl819x_Qos.h | 48 >

Re: [PATCH] iee80211:Remove no longer required strutucre defintion, _STA_QOS in rtl819x_Qos.h

2015-03-06 Thread Nicholas Krause
# On March 6, 2015 5:13:09 PM EST, "Giedrius Statkevičius" wrote: >On 2015.03.06 23:20, Nicholas Krause wrote: >> Removes the no longer required structure definition,_STA_QOS due to >> no functions using any structures of this type anymore. >> >> Signed-off-by: Nicholas Krause >> --- >> drive

RE: [PATCH] Drivers: hv: hv_balloon: keep locks balanced on add_memory() failure

2015-03-06 Thread KY Srinivasan
> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Monday, March 2, 2015 2:34 AM > To: KY Srinivasan > Cc: de...@linuxdriverproject.org; Haiyang Zhang; linux- > ker...@vger.kernel.org; Dexuan Cui > Subject: Re: [PATCH] Drivers: hv: hv_balloon: keep locks ba

[PATCH v3] Staging: dgnc: Remove sysfs file

2015-03-06 Thread Salah Triki
This sysfs file is not useful for anything. Signed-off-by: Salah Triki --- drivers/staging/dgnc/dgnc_sysfs.c | 25 - 1 file changed, 25 deletions(-) diff --git a/drivers/staging/dgnc/dgnc_sysfs.c b/drivers/staging/dgnc/dgnc_sysfs.c index a72e353..b5738dc 100644 --- a/dr

CONGRATULATION YOUR EMAIL WON

2015-03-06 Thread leelavathiraj
CHEVROLET CAR COMPANY, 45 ,Manchester, Linc's, BE9 8WB, London United Kingdom. Tel: +447010032898 5th mar 2015 We wish to congratulate you once again on this note, for being one of the lucky winners selected on this year CHEVROLET MOTORS PROMOTION 2015. We felicitate with you and your family an

Attention: Claim Your USD$ 3,500,000.00 Million,

2015-03-06 Thread Aba Justice
Attention:Claim Your USD$ 3,500,000.00 Million, RE-YOUR LONG OVER DUE PAYMENT APPROVED FOR PAYMENT IN YOUR FAVOR. This is to officially bring to your notice that you have been issued a payment notice of total sum of ( $3,500,000.00 Million ) Following the Board of Trustees and the Executive Cou

[PATCH 0/6] Drivers: hv: Miscellaneous fixes

2015-03-06 Thread K. Y. Srinivasan
This patch-set has miscellaneous fixes for both the VMBUS as well as the balloon driver. Dan Carpenter (1): hv: vmbus: missing curly braces in vmbus_process_offer() K. Y. Srinivasan (2): Drivers: hv: vmbus: Export the vmbus_sendpacket_pagebuffer_ctl() Drivers: hv: vmbus: Perform device regi

[PATCHv2 1/2] iee80211:Remove no longer required strutucre defintion, _STA_QOS in rtl819x_Qos.h

2015-03-06 Thread Nicholas Krause
Removes the no longer required structure definition,_STA_QOS due to no functions using any structures of this type anymore. Signed-off-by: Nicholas Krause --- drivers/staging/rtl8192u/ieee80211/rtl819x_Qos.h | 48 1 file changed, 48 deletions(-) diff --git a/drivers/sta

[PATCH 2/6] Drivers: hv: vmbus: Perform device register in the per-channel work element

2015-03-06 Thread K. Y. Srinivasan
This patch is a continuation of the rescind handling cleanup work. We cannot block in the global message handling work context especially if we are blocking waiting for the host to wake us up. I would like to thank Dexuan Cui for observing this problem. Signed-off-by: K. Y. Srinivasan --- drive

[PATCH 6/6] hv: vmbus: missing curly braces in vmbus_process_offer()

2015-03-06 Thread K. Y. Srinivasan
From: Dan Carpenter The indenting makes it clear that there were curly braces intended here. Fixes: 2dd37cb81580 ('Drivers: hv: vmbus: Handle both rescind and offer messages in the same context') Signed-off-by: Dan Carpenter Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel_mgmt.c |

[PATCH 1/6] Drivers: hv: vmbus: Export the vmbus_sendpacket_pagebuffer_ctl()

2015-03-06 Thread K. Y. Srinivasan
Export the vmbus_sendpacket_pagebuffer_ctl() interface. Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/hv/channel.c b/drivers/hv/channel.c index da53180..e58cdb7 100644 --- a/drivers/hv/channel.c +++ b/dri

[PATCHv2 2/2] rtl8192u:Remove commented declartions of the structure type, _STA_QOS in ieee80211.h and r8192U_dm.c

2015-03-06 Thread Nicholas Krause
This removes the commented declarations of the structure type, _STA_QOS due to us no longer having this structure declartion having now been removed from the header file,rtl819x_Qos.h due to no declartions of this type present anymore. Signed-off-by: Nicholas Krause --- drivers/staging/rtl8192u/

[PATCH 4/6] Drivers: hv: hv_balloon: don't lose memory when onlining order is not natural

2015-03-06 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov Memory blocks can be onlined in random order. When this order is not natural some memory pages are not onlined because of the redundant check in hv_online_page(). Here is a real world scenario: 1) Host tries to hot-add the following (process_hot_add): pg_start=rg_start=0

[PATCH 3/6] Drivers: hv: hv_balloon: keep locks balanced on add_memory() failure

2015-03-06 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov When add_memory() fails the following BUG is observed: [ 743.646107] hv_balloon: hot_add memory failed error is -17 [ 743.679973] [ 743.680930] = [ 743.680930] [ BUG: bad unlock balance detected! ] [ 743.680930] 3.19.0-rc5_bug1131426

[PATCH 5/6] Correcting truncation error for constant HV_CRASH_CTL_CRASH_NOTIFY

2015-03-06 Thread K. Y. Srinivasan
From: Nick Meier HV_CRASH_CTL_CRASH_NOTIFY is a 64 bit number. Depending on the usage context, the value may be truncated. This patch is in response from the following email from Intel: [char-misc:char-misc-testing 25/45] drivers/hv/vmbus_drv.c:67:9: sparse: constant 0x80

RE: [PATCH 0/6] Drivers: hv: Miscellaneous fixes

2015-03-06 Thread KY Srinivasan
> -Original Message- > From: K. Y. Srinivasan [mailto:k...@microsoft.com] > Sent: Friday, March 6, 2015 9:10 PM > To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; > de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; > vkuzn...@redhat.com > Cc: KY Srinivasan > S

[PATCH 2/2] staging: ft1000: remove code indention

2015-03-06 Thread Sudip Mukherjee
modified the code to keep the logic same but removed some indention. Signed-off-by: Sudip Mukherjee --- this patch will generate checkpatch warning about line more than 80char, and too many use of tab. but unless the total function is rewrtten it will be difficult to fix that. drivers/staging/

[PATCH 1/2] staging: ft1000: remove unused variables

2015-03-06 Thread Sudip Mukherjee
these variables were assigned some values but they were never being reused again. Signed-off-by: Sudip Mukherjee --- this patch will generate some checkpatch warning about line being above 80char, but the code is so much indented that it is difficult to break the line. drivers/staging/ft1000/f

[PATCH V2 0/7] Drivers: hv: Miscellaneous fixes

2015-03-06 Thread K. Y. Srinivasan
This patch-set has miscellaneous fixes for both the VMBUS as well as the balloon driver. There is also a fix for the hv tools makefile. In this version of the patchset, I have included a patch from Dexuan. Furthermore, I have addressed a memory leak issue in the patch: Drivers: hv: vmbus: Perform

[PATCH V2 4/7] Drivers: hv: hv_balloon: don't lose memory when onlining order is not natural

2015-03-06 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov Memory blocks can be onlined in random order. When this order is not natural some memory pages are not onlined because of the redundant check in hv_online_page(). Here is a real world scenario: 1) Host tries to hot-add the following (process_hot_add): pg_start=rg_start=0

[PATCH V2 3/7] Drivers: hv: hv_balloon: keep locks balanced on add_memory() failure

2015-03-06 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov When add_memory() fails the following BUG is observed: [ 743.646107] hv_balloon: hot_add memory failed error is -17 [ 743.679973] [ 743.680930] = [ 743.680930] [ BUG: bad unlock balance detected! ] [ 743.680930] 3.19.0-rc5_bug1131426

[PATCH V2 5/7] Correcting truncation error for constant HV_CRASH_CTL_CRASH_NOTIFY

2015-03-06 Thread K. Y. Srinivasan
From: Nick Meier HV_CRASH_CTL_CRASH_NOTIFY is a 64 bit number. Depending on the usage context, the value may be truncated. This patch is in response from the following email from Intel: [char-misc:char-misc-testing 25/45] drivers/hv/vmbus_drv.c:67:9: sparse: constant 0x80

[PATCH V2 1/7] Drivers: hv: vmbus: Export the vmbus_sendpacket_pagebuffer_ctl()

2015-03-06 Thread K. Y. Srinivasan
Export the vmbus_sendpacket_pagebuffer_ctl() interface. Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/drivers/hv/channel.c b/drivers/hv/channel.c index da53180..e58cdb7 100644 --- a/drivers/hv/channel.c +++ b/dri

[PATCH V2 6/7] hv: vmbus: missing curly braces in vmbus_process_offer()

2015-03-06 Thread K. Y. Srinivasan
From: Dan Carpenter The indenting makes it clear that there were curly braces intended here. Fixes: 2dd37cb81580 ('Drivers: hv: vmbus: Handle both rescind and offer messages in the same context') Signed-off-by: Dan Carpenter Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel_mgmt.c |

[PATCH V2 7/7] tools: hv: fcopy_daemon: support >2GB files for x86_32 guest

2015-03-06 Thread K. Y. Srinivasan
From: Dexuan Cui Without this patch, hv_fcopy_daemon's hv_copy_data() -> pwrite() will fail for >2GB file offset. Signed-off-by: Alex Ng Signed-off-by: Dexuan Cui Cc: K. Y. Srinivasan Signed-off-by: K. Y. Srinivasan --- tools/hv/Makefile |2 +- 1 files changed, 1 insertions(+), 1 deleti

[PATCH V2 2/7] Drivers: hv: vmbus: Perform device register in the per-channel work element

2015-03-06 Thread K. Y. Srinivasan
This patch is a continuation of the rescind handling cleanup work. We cannot block in the global message handling work context especially if we are blocking waiting for the host to wake us up. I would like to thank Dexuan Cui for observing this problem. Signed-off-by: K. Y. Srinivasan --- - Fre