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

2015-06-28 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] asn:1*ifcu_buf got char *noident Signed-off-by: Geliang Tang geliangt...@163.com ---

LIBCFS_ALLOC

2015-06-28 Thread Julia Lawall
It is not clear that all of the uses of LIBCFS_ALLOC really risk needing vmalloc. For example: lnet/klnds/socklnd/socklnd.c, function ksocknal_accept: ksock_connreq_t *cr; ... LIBCFS_ALLOC(cr, sizeof(*cr)); The definition of ksock_connreq_t is: typedef struct ksock_connreq { struct

New Order

2015-06-28 Thread soumiya
Dear Sir/Madam, Kindly find the attached reversed order quantity, technical details and full list of our demand which we intend to supply on urgent bases to our end buyer. After study please give me feedback promptly so we can finalize discussion on that, because we need the products

Re: LIBCFS_ALLOC

2015-06-28 Thread Dan Carpenter
Yeah. You're right. Doing a vmalloc() when kmalloc() doesn't have even a tiny sliver of RAM isn't going to work. It's easier to use libcfs_kvzalloc() everywhere, but it's probably the wrong thing. regards, dan carpenter ___ devel mailing list

[PATCH RESEND 04/10] Drivers: hv: don't do hypercalls when hypercall_page is NULL

2015-06-28 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov vkuzn...@redhat.com At the very late stage of kexec a driver (which are not being unloaded) can try to post a message or signal an event. This will crash the kernel as we already did hv_cleanup() and the hypercall page is NULL. Move all common (between 32 and 64 bit code)

[PATCH REESEND 06/10] Drivers: hv: vmbus: prefer 'die' notification chain to 'panic'

2015-06-28 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov vkuzn...@redhat.com current_pt_regs() sometimes returns regs of the userspace process and in case of a kernel crash this is not what we need to report. E.g. when we trigger crash with sysrq we see the following: ... RIP: 0010:[815b8696] [815b8696]

[PATCH RESEND 05/10] Drivers: hv: vmbus: add special crash handler

2015-06-28 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov vkuzn...@redhat.com Full kernel hang is observed when kdump kernel starts after a crash. This hang happens in vmbus_negotiate_version() function on wait_for_completion() as Hyper-V host (Win2012R2 in my testing) never responds to CHANNELMSG_INITIATE_CONTACT as it thinks the

[PATCH RESEND 01/10] Drivers: hv: vmbus: remove hv_synic_free_cpu() call from hv_synic_cleanup()

2015-06-28 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov vkuzn...@redhat.com We already have hv_synic_free() which frees all per-cpu pages for all CPUs, let's remove the hv_synic_free_cpu() call from hv_synic_cleanup() so it will be possible to do separate cleanup (writing to MSRs) and final freeing. This is going to be used to

[PATCH RESEND 10/10] Drivers: hv: vmbus: Permit sending of packets without payload

2015-06-28 Thread K. Y. Srinivasan
The guest may have to send a completion packet back to the host. To support this usage, permit sending a packet without a payload - we would be only sending the descriptor in this case. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/hv/channel.c |4 +++- 1 files changed, 3

[PATCH RESEND 02/10] kexec: define kexec_in_progress in !CONFIG_KEXEC case

2015-06-28 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov vkuzn...@redhat.com If some piece of code wants to check kexec_in_progress it has to be put in #ifdef CONFIG_KEXEC block to not break the build in !CONFIG_KEXEC case. Overcome this limitation by defining kexec_in_progress to false. Signed-off-by: Vitaly Kuznetsov

[PATCH RESEND 08/10] Drivers: hv: fcopy: dynamically allocate smsg_out in fcopy_send_data()

2015-06-28 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov vkuzn...@redhat.com struct hv_start_fcopy is too big to be on stack on i386, the following warning is reported: drivers/hv/hv_fcopy.c:159:1: warning: the frame size of 1088 bytes is larger than 1024 bytes [-Wframe-larger-than=] Reported-by: kbuild test robot

[PATCH RESEND 07/10] Drivers: hv: kvp: check kzalloc return value

2015-06-28 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov vkuzn...@redhat.com kzalloc() return value check was accidentally lost in 11bc3a5fa91f: Drivers: hv: kvp: convert to hv_utils_transport commit. We don't need to reset kvp_transaction.state here as we have the kvp_timeout_func() timeout function and in case we're in OOM

[PATCH RESEND 09/10] Drivers: hv: balloon: Enable dynamic memory protocol negotiation with Windows 10 hosts

2015-06-28 Thread K. Y. Srinivasan
From: Alex Ng ale...@microsoft.com Support Win10 protocol for Dynamic Memory. Thia patch allows guests on Win10 hosts to hot-add memory even when dynamic memory is not enabled on the guest. Signed-off-by: Alex Ng ale...@microsoft.com Signed-off-by: K. Y. Srinivasan k...@microsoft.com ---

[PATCH RESEND 03/10] Drivers: hv: vmbus: add special kexec handler

2015-06-28 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov vkuzn...@redhat.com When general-purpose kexec (not kdump) is being performed in Hyper-V guest the newly booted kernel fails with an MCE error coming from the host. It is the same error which was fixed in the Drivers: hv: vmbus: Implement the protocol for tearing down vmbus

[PATCH]drivers:staging:visorbus:Fix checkpatch warnings

2015-06-28 Thread Ravi Teja
This patch fixes checkpatch warnings Signed-off-by: Ravi Teja Darbha rav...@gmail.com --- drivers/staging/unisys/visorbus/visorchipset.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/staging/unisys/visorbus/visorchipset.c

[PATCH RESEND 00/10] Drivers: hv: vmbus: Enable kexec and other misc cleanup

2015-06-28 Thread K. Y. Srinivasan
In addition to enabling kexec, this patch-set has a bunch of miscellaneous fixes. I have been having mail issues and hence resending the series. Alex Ng (1): Drivers: hv: balloon: Enable dynamic memory protocol negotiation with Windows 10 hosts K. Y. Srinivasan (1): Drivers: hv: vmbus:

[PATCH 07/11] staging: sm750fb: fix indentation in 'else' statements

2015-06-28 Thread Anatoly Stepanov
Fix indentation according to checkpatch.pl message: ERROR: space required after that close brace '}' Signed-off-by: Anatoly Stepanov drivengro...@gmail.com --- drivers/staging/sm750fb/sm750_hw.c | 16 1 file changed, 8 insertions(+), 8 deletions(-) diff --git

[PATCH 08/11] staging: sm750fb: fix 'switch and case' indentation

2015-06-28 Thread Anatoly Stepanov
Fix according to checkpatch.pl message: ERROR: switch and case should be at the same indent Signed-off-by: Anatoly Stepanov drivengro...@gmail.com --- drivers/staging/sm750fb/sm750_hw.c | 172 ++--- 1 file changed, 86 insertions(+), 86 deletions(-) diff --git

[PATCH 06/11] staging: sm750fb: put open brace on the previous line

2015-06-28 Thread Anatoly Stepanov
Fix open braces according to checkpatch.pl message: ERROR: that open brace { should be on the previous line Signed-off-by: Anatoly Stepanov drivengro...@gmail.com --- drivers/staging/sm750fb/sm750_hw.c | 24 1 file changed, 8 insertions(+), 16 deletions(-) diff --git

[PATCH 03/11] staging: sm750fb: insert space between ')' and '{'

2015-06-28 Thread Anatoly Stepanov
Insert spaces according to checkpatch.pl message: ERROR: space required before the open brace '{' Signed-off-by: Anatoly Stepanov drivengro...@gmail.com --- drivers/staging/sm750fb/sm750_hw.c | 40 +++--- 1 file changed, 20 insertions(+), 20 deletions(-) diff

[PATCH 05/11] staging: sm750fb: fix indentation of pointer operator

2015-06-28 Thread Anatoly Stepanov
Fix indentation of pointer operator '*' according to checkpatch.pl message: ERROR: 'foo* bar' should be 'foo *bar' Signed-off-by: Anatoly Stepanov drivengro...@gmail.com --- drivers/staging/sm750fb/sm750_hw.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-)

[PATCH 00/11] staging: sm750fb: fix checkpatch issues

2015-06-28 Thread Anatoly Stepanov
This patch series fixes chekpatch.pl errors in sm750fb. Anatoly Stepanov (11): staging: sm750fb: replace spaces with tabs staging: sm750fb: remove trailing whitespaces staging: sm750fb: insert space between ')' and '{' staging: sm750fb: insert space before open parenthesis staging:

[PATCH 02/11] staging: sm750fb: remove trailing whitespaces

2015-06-28 Thread Anatoly Stepanov
Remove trailing whitespaces according to checkpatch.pl error message: ERROR: trailing whitespace Signed-off-by: Anatoly Stepanov drivengro...@gmail.com --- drivers/staging/sm750fb/sm750_hw.c | 25 - 1 file changed, 12 insertions(+), 13 deletions(-) diff --git

[PATCH 01/11] staging: sm750fb: replace spaces with tabs

2015-06-28 Thread Anatoly Stepanov
Replace spaces with tabs according to checkpatch.pl error message: ERROR: code indent should use tabs where possible Signed-off-by: Anatoly Stepanov drivengro...@gmail.com --- drivers/staging/sm750fb/sm750_hw.c | 70 +++--- 1 file changed, 35 insertions(+), 35

[RESEND] mfd: rtsx: add support for rts522A

2015-06-28 Thread micky_ching
From: Micky Ching micky_ch...@realsil.com.cn rts522a(rts5227s) is derived from rts5227, and mainly same with rts5227. Add it to file mfd/rts5227.c to support this chip. Signed-off-by: Micky Ching micky_ch...@realsil.com.cn --- drivers/mfd/Kconfig | 7 ++-- drivers/mfd/rts5227.c

[PATCH 11/11] staging: sm750fb: add required spaces around C operators

2015-06-28 Thread Anatoly Stepanov
Add spaces according to checkpatch.pl messages: ERROR: spaces required around that '=' (ctx:VxV) ERROR: need consistent spacing around '-' (ctx:WxV) ERROR: spaces required around that '?' (ctx:VxE) ERROR: need consistent spacing around '' (ctx:VxW) Signed-off-by: Anatoly Stepanov

[PATCH 04/11] staging: sm750fb: insert space before open parenthesis

2015-06-28 Thread Anatoly Stepanov
Insert spaces before '(' according to checkpatch.pl message: ERROR: space required before the open parenthesis Signed-off-by: Anatoly Stepanov drivengro...@gmail.com --- drivers/staging/sm750fb/sm750_hw.c | 62 +++--- 1 file changed, 31 insertions(+), 31

[PATCH 10/11] staging: sm750fb: fix open brace in function declaration

2015-06-28 Thread Anatoly Stepanov
Fix according to checkpatch.pl message: ERROR: open brace '{' following function declarations go on the next line Signed-off-by: Anatoly Stepanov drivengro...@gmail.com --- drivers/staging/sm750fb/sm750_hw.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH 09/11] staging: sm750fb: remove assignment from if condition

2015-06-28 Thread Anatoly Stepanov
Remove assignment from if condition according to checkpatch.pl message: ERROR: do not use assignment in if condition Signed-off-by: Anatoly Stepanov drivengro...@gmail.com --- drivers/staging/sm750fb/sm750_hw.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

Re: [PATCH]drivers:staging:visorbus:Fix checkpatch warnings

2015-06-28 Thread Greg KH
On Sun, Jun 28, 2015 at 10:24:49PM +0530, Ravi Teja wrote: This patch fixes checkpatch warnings What warnings? Be specific please. ___ devel mailing list de...@linuxdriverproject.org