LIBCFS_ALLOC

2015-06-27 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 li

[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

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

2015-06-27 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov 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 state" commit - mo

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

2015-06-27 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov 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 Signed-off-by: K. Y. Sriniva

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

2015-06-27 Thread K. Y. Srinivasan
From: Vitaly Kuznetsov 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 assist kexec. Signed

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

2015-06-27 Thread K. Y. Srinivasan
In addition to enabling kexec, this patch-set has a bunch of miscellaneous fixes. Alex Ng (1): Drivers: hv: balloon: Enable dynamic memory protocol negotiation with Windows 10 hosts K. Y. Srinivasan (1): Drivers: hv: vmbus: Permit sending of packets without payload Vitaly Kuznetsov (8):

RE: [PATCH] scsi: storvsc: be more picky about scmnd->sc_data_direction

2015-06-27 Thread KY Srinivasan
> -Original Message- > From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Thursday, June 25, 2015 9:12 AM > To: linux-s...@vger.kernel.org > Cc: Long Li; KY Srinivasan; Haiyang Zhang; James E.J. Bottomley; > de...@linuxdriverproject.org; linux-ker...@vger.kernel.org; Radim Krčmář

Re: [PATCH 2/2] staging: wilc1000: One function call less in mac_ioctl() after error detection

2015-06-27 Thread Julia Lawall
On Sat, 27 Jun 2015, SF Markus Elfring wrote: > From: Markus Elfring > Date: Sat, 27 Jun 2015 16:00:59 +0200 > > The kfree() function was called in two cases by the mac_ioctl() function > during error handling even if the passed variable did not contain a pointer > for a valid data item. > > *

Re: How to learn linux driver

2015-06-27 Thread Greg KH
On Sat, Jun 27, 2015 at 08:37:29PM +0800, fancier unity wrote: > I'am a beginner for linux, I do not know how to learn it, how should do if I > want to learn it? kernelnewbies.org ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linux

[PATCH 2/2] staging: wilc1000: One function call less in mac_ioctl() after error detection

2015-06-27 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 27 Jun 2015 16:00:59 +0200 The kfree() function was called in two cases by the mac_ioctl() function during error handling even if the passed variable did not contain a pointer for a valid data item. * This implementation detail could be improved by the introductio

[PATCH 1/2] staging: wilc1000: Delete unnecessary checks before two function calls

2015-06-27 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 27 Jun 2015 15:56:57 +0200 The functions kfree() and release_firmware() test whether their argument is NULL and then return immediately. Thus the test around the calls is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus

[PATCH 0/2] staging: wilc1000: Deletion of two unnecessary checks

2015-06-27 Thread SF Markus Elfring
From: Markus Elfring Further update suggestions were taken into account after a patch was applied from static source code analysis. Markus Elfring (2): Delete unnecessary checks before two function calls One function call less in mac_ioctl() after error detection drivers/staging/wilc1000/l

How to learn linux driver

2015-06-27 Thread fancier unity
I'am a beginner for linux, I do not know how to learn it, how should do if I want to learn it? ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

[PATCH] staging: comedi: adl_pci9118: Delete an unnecessary check before the function call "pci_dev_put"

2015-06-27 Thread SF Markus Elfring
From: Markus Elfring Date: Sat, 27 Jun 2015 13:50:43 +0200 The pci_dev_put() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- drive

Re: rename_rev.pl: review script for whitespace changes

2015-06-27 Thread Sudip Mukherjee
On Fri, Jun 26, 2015 at 04:15:24PM +0300, Dan Carpenter wrote: > I've sent my review script out a few times before but we have some new > reviewers in staging who maybe haven't tried them. Thanks Dan. It will be of great help. regards sudip ___ devel mai

Re: [PATCH] staging: board: Add dependency on CLKDEV_LOOKUP

2015-06-27 Thread Geert Uytterhoeven
On Sat, Jun 27, 2015 at 4:39 AM, Guenter Roeck wrote: > The code depends on CLKDEV_LOOKUP since commit 225d68d852f1 ("staging: > board: Add support for devices with complex dependencies"). > > Related build error (powerpc:allmodconfig): > > drivers/built-in.o: In function `.board_staging_register_