Re: [PATCH 3/7] Drivers: hv: vmbus: Fix a bug in hv_need_to_signal_on_read()

2016-03-31 Thread Greg KH
On Wed, Mar 23, 2016 at 05:53:53PM -0700, K. Y. Srinivasan wrote: > We need to issue a full memory barrier prior making a signalling decision. > > Signed-off-by: K. Y. Srinivasan > Cc: sta...@vger.kernel.org > --- > drivers/hv/ring_buffer.c |1 + > 1 files changed, 1

Re: [PATCH 2/7] Drivers: hv: vmbus: Use READ_ONCE() to read variables that are volatile

2016-03-31 Thread Greg KH
On Wed, Mar 23, 2016 at 05:53:52PM -0700, K. Y. Srinivasan wrote: > Use the READ_ONCE macro to access variabes that can change asynchronously. Why? What is this "fixing"? thanks, greg k-h ___ devel mailing list de...@linuxdriverproject.org

Re: [PATCH 3/7] Drivers: hv: vmbus: Fix a bug in hv_need_to_signal_on_read()

2016-03-31 Thread Greg KH
On Wed, Mar 23, 2016 at 05:53:53PM -0700, K. Y. Srinivasan wrote: > We need to issue a full memory barrier prior making a signalling decision. > > Signed-off-by: K. Y. Srinivasan > Cc: sta...@vger.kernel.org > --- > drivers/hv/ring_buffer.c |1 + > 1 files changed, 1

[PATCH] staging/rdma/hfi1: select CRC32

2016-03-31 Thread Markus Böhme
The function parse_platform_config in firmware.c calls crc32_le. Building without CRC32 selected causes a link error: drivers/built-in.o: In function `parse_platform_config': (.text+0x92ffa): undefined reference to `crc32_le' Signed-off-by: Markus Böhme ---

[PATCH] staging: vme: fix bare use of 'unsigned'

2016-03-31 Thread Clifton Barnes
fix checkpatch.pl warning about 'Prefer 'unsigned int' to bare use of 'unsigned'' Signed-off-by: Clifton Barnes --- drivers/staging/vme/devices/vme_pio2_gpio.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git

RE: [lustre-devel] [PATCH 2/6] staging: lustre: libcfs: move memory_pressure functions to libcfs_prim.h

2016-03-31 Thread Simmons, James A.
>On Thu, Mar 31, 2016 at 10:18:36AM -0400, James Simmons wrote: >> Long ago libcfs_prim.h was used for userland code which is why >> memory_pressure_*() handling is in both libcfs_prim.h and >> linux-mem.h headers. So lets just move the memory_pressure_*() >> to libcfs_prim.h. >> >>

Re: [PATCH 1/6] Drivers: hv: kvp: fix IP Failover

2016-03-31 Thread Greg KH
On Tue, Mar 29, 2016 at 02:27:16PM +0200, Vitaly Kuznetsov wrote: > "K. Y. Srinivasan" writes: > > > From: Vitaly Kuznetsov > > > > Hyper-V VMs can be replicated to another hosts and there is a feature to > > set different IP for replicas, it is called

Re: [PATCH 2/6] staging: lustre: libcfs: move memory_pressure functions to libcfs_prim.h

2016-03-31 Thread Greg Kroah-Hartman
On Thu, Mar 31, 2016 at 10:18:36AM -0400, James Simmons wrote: > Long ago libcfs_prim.h was used for userland code which is why > memory_pressure_*() handling is in both libcfs_prim.h and > linux-mem.h headers. So lets just move the memory_pressure_*() > to libcfs_prim.h. > > Signed-off-by: James

[PATCH 0/6] cleanup libcfs memory handling

2016-03-31 Thread James Simmons
The libcfs module contains memory handling which needs to be cleaned up. First cleanup is to merge linux-mem.h and libcfs_prim.h. This is left over for when libcfs was both a kernel module and a userspace library. Second cleanup is remove some wrappers that are not needed. Just use kernel API's

[PATCH 5/6] staging: lustre: libcfs: move NUM_CACHEPAGES to libcfs_prim.h

2016-03-31 Thread James Simmons
We don't really need linux specific headers anymore so move NUM_CACHEPAGES macro to libcfs_prim.h. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/13841 Reviewed-by: frank zago

[PATCH 4/6] staging: lustre: libcfs: remove MMSPACE macros

2016-03-31 Thread James Simmons
Another abstraction that is not needed. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/13841 Reviewed-by: frank zago Reviewed-by: Dmitry Eremin

[PATCH 6/6] staging: lustre: libcfs: delete linux-mem.h

2016-03-31 Thread James Simmons
The header linux-mem.h is no longer needed. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/13841 Reviewed-by: frank zago Reviewed-by: Dmitry Eremin

[PATCH 3/6] staging: lustre: libcfs: remove page_index() macro

2016-03-31 Thread James Simmons
Just use the index field directly for struct page. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/13841 Reviewed-by: frank zago Reviewed-by: Dmitry Eremin

[PATCH 2/6] staging: lustre: libcfs: move memory_pressure functions to libcfs_prim.h

2016-03-31 Thread James Simmons
Long ago libcfs_prim.h was used for userland code which is why memory_pressure_*() handling is in both libcfs_prim.h and linux-mem.h headers. So lets just move the memory_pressure_*() to libcfs_prim.h. Signed-off-by: James Simmons ntel-bug-id:

[PATCH 1/6] staging: lustre: libcfs: move add_wait_queue_exclusive_head to lustre layer

2016-03-31 Thread James Simmons
Only lustre client uses add_wait_queue_exclusive_head() so move it from libcfs layer to lustre_lib.h where it is needed. Signed-off-by: James Simmons Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6245 Reviewed-on: http://review.whamcloud.com/13874 Reviewed-by: Dmitry

[PATCH 2/2] staging: dgnc: clean up dgnc_input function

2016-03-31 Thread Daeseok Youn
This is for fixing checkpatch.pl warning about "Alignment should match open parenthesis" but if that is fixed, code line is over 80 characters. I think "ch->ch_rqueue + tail + i" could be declared once in the begining of loop. Signed-off-by: Daeseok Youn ---

[PATCH 1/2] staging: dgnc: remove useless variables for saving tty's

2016-03-31 Thread Daeseok Youn
It doesn't need to save major number with variable. And there are no use of these variables(dgnc_serial_major and dgnc_transparent_print_major) Signed-off-by: Daeseok Youn --- drivers/staging/dgnc/dgnc_driver.h | 3 --- drivers/staging/dgnc/dgnc_tty.c| 4 2