Re: [PATCH 2/2] Drivers: scsi: storvsc: Force discovery of LUNs that may have been removed.

2014-08-28 Thread Hannes Reinecke
On 08/29/2014 04:42 AM, Mike Christie wrote: On 08/27/2014 09:31 AM, Hannes Reinecke wrote: On 08/19/2014 07:54 PM, Christoph Hellwig wrote: On Sat, Aug 16, 2014 at 08:09:48PM -0700, K. Y. Srinivasan wrote: The host asks the guest to scan when a LUN is removed or added. The only way a guest ca

[PATCH] staging: rts5208: Remove casting the return value which is a void pointer

2014-08-28 Thread Jingoo Han
Casting the return value which is a void pointer is redundant. The conversion from void pointer to any other pointer type is guaranteed by the C programming language. Signed-off-by: Jingoo Han --- drivers/staging/rts5208/rtsx.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff

RE: [PANIC, hyperv] BUG: unable to handle kernel paging request at ffff880077800004 (hv_ringbuffer_write)

2014-08-28 Thread Dexuan Cui
> -Original Message- > From: Sitsofe Wheeler > Sent: Thursday, August 28, 2014 20:49 PM > > > > The only issue seen on boot now is similar to > > > > https://lkml.org/lkml/2014/8/19/227 ... > > > I don't see this issue. Do you still see the issue for EVERY boot > > after you applied KY's

Re: [PATCH 2/2] Drivers: scsi: storvsc: Force discovery of LUNs that may have been removed.

2014-08-28 Thread Mike Christie
On 08/27/2014 09:31 AM, Hannes Reinecke wrote: > On 08/19/2014 07:54 PM, Christoph Hellwig wrote: >> On Sat, Aug 16, 2014 at 08:09:48PM -0700, K. Y. Srinivasan wrote: >>> The host asks the guest to scan when a LUN is removed or added. >>> The only way a guest can identify the removed LUN is when an

Re: [PATCH 2/2] Drivers: scsi: storvsc: Force discovery of LUNs that may have been removed.

2014-08-28 Thread Christoph Hellwig
On Tue, Aug 26, 2014 at 10:54:51PM +, KY Srinivasan wrote: > > This looks pretty reasonable to me, but I wonder if we should move this up > > to common code so that it happens for any host rescan triggered by sysfs or > > other drivers as well. > Ping. Any decision on if/when this patch may be

[PATCH 2/2] Drivers: hv: vmbus: Properly protect calls to smp_processor_id()

2014-08-28 Thread K. Y. Srinivasan
Disable preemption when sampling current processor ID when preemption is otherwise possible. Signed-off-by: K. Y. Srinivasan --- drivers/hv/channel.c |7 +-- drivers/hv/channel_mgmt.c | 21 +++-- 2 files changed, 20 insertions(+), 8 deletions(-) diff --git a/drive

[PATCH 1/2] Drivers: hv: vmbus: Cleanup hv_post_message()

2014-08-28 Thread K. Y. Srinivasan
Minimize failures in this function by pre-allocating the buffer for posting messages. The hypercall for posting the message can fail for a number of reasons: 1. Transient resource related issues 2. Buffer alignment 3. Buffer cannot span a page boundry We address issues 2 a

[PATCH 0/2] Drivers: hv: vmbus: Miscellaneous cleanup

2014-08-28 Thread K. Y. Srinivasan
Cleanup hv_post_message() to minimize failures. Also disable preemption when sampling CPU ID when preemption is otherwise possible. K. Y. Srinivasan (2): Drivers: hv: vmbus: Cleanup hv_post_message() Drivers: hv: vmbus: Properly protect calls to smp_processor_id() drivers/hv/channel.c |

Re: [PATCH v3 00/11] lib: introduce string_escape_mem and %*pE specifier

2014-08-28 Thread Andy Shevchenko
On Fri, Aug 29, 2014 at 12:06 AM, Andrew Morton wrote: > On Thu, 28 Aug 2014 23:58:45 +0300 Andy Shevchenko > wrote: [] >> For now (so far no more comments) it is only couple of trivia fixes >> (removing useless comments). Would you like to resend whole series or >> just fixup would be enough?

Re: [PATCH v3 00/11] lib: introduce string_escape_mem and %*pE specifier

2014-08-28 Thread Andrew Morton
On Thu, 28 Aug 2014 23:58:45 +0300 Andy Shevchenko wrote: > On Thu, Aug 28, 2014 at 10:08 PM, Andrew Morton > wrote: > > On Thu, 28 Aug 2014 14:33:30 -0400 "John W. Linville" > > wrote: > > > >> On Wed, Aug 20, 2014 at 12:42:41PM +0300, Andy Shevchenko wrote: > >> > The introduced function is

Re: [PATCH v3 00/11] lib: introduce string_escape_mem and %*pE specifier

2014-08-28 Thread Andy Shevchenko
On Thu, Aug 28, 2014 at 10:08 PM, Andrew Morton wrote: > On Thu, 28 Aug 2014 14:33:30 -0400 "John W. Linville" > wrote: > >> On Wed, Aug 20, 2014 at 12:42:41PM +0300, Andy Shevchenko wrote: >> > The introduced function is a kind of opposite to string_unescape. We have >> > several users of such

[PATCH] staging: lustre: osc: add blank line after declarations

2014-08-28 Thread Mario Schlegel
fixed a coding style issue Signed-off-by: Mario Schlegel --- drivers/staging/lustre/lustre/osc/osc_io.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/drivers/staging/lustre/lustre/osc/osc_io.c b/drivers/staging/lustre/lustre/osc/osc_io.c index 886b1ef..fad4c13 100644 --- a/drivers/stag

Re: [PATCH v3 00/11] lib: introduce string_escape_mem and %*pE specifier

2014-08-28 Thread John W. Linville
On Thu, Aug 28, 2014 at 12:08:51PM -0700, Andrew Morton wrote: > On Thu, 28 Aug 2014 14:33:30 -0400 "John W. Linville" > wrote: > > > On Wed, Aug 20, 2014 at 12:42:41PM +0300, Andy Shevchenko wrote: > > > The introduced function is a kind of opposite to string_unescape. We have > > > several use

Re: [PATCH v3 00/11] lib: introduce string_escape_mem and %*pE specifier

2014-08-28 Thread Andrew Morton
On Thu, 28 Aug 2014 14:33:30 -0400 "John W. Linville" wrote: > On Wed, Aug 20, 2014 at 12:42:41PM +0300, Andy Shevchenko wrote: > > The introduced function is a kind of opposite to string_unescape. We have > > several users of such functionality each of them created custom > > implementation. >

[PATCH 4/4] staging/lustre: remove unused request policies.

2014-08-28 Thread Oleg Drokin
Client policies do not make much sense to have in client-only code, so remove all references to everything but fifo. This also removes lustre/libcfs/heap.c and its header, since it was only used by certain request policies (crr and orr). Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lust

[PATCH 2/4] staging/lustre/obdclass: remove unused file llog_lvfs.c

2014-08-28 Thread Oleg Drokin
This contains server-side llog operations. Also remove declaration of public structure from lustre_log.h Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_log.h | 3 - drivers/staging/lustre/lustre/obdclass/llog_lvfs.c | 847 - 2 files changed, 850

[PATCH 0/4] Remove unused lustre files

2014-08-28 Thread Oleg Drokin
This batch of patches removes some lustre files that are not being built and are overall unneeded in the client only code. These include EA from disk swabbing code, various network request policies for server-site traffic prioritization, server-side llog code, lustre binary heap implementation. Pl

[PATCH 3/4] staging/lustre/obdclass: remove unused file linkea.c

2014-08-28 Thread Oleg Drokin
Also unused header file lustre_linkea.h Signed-off-by: Oleg Drokin --- .../staging/lustre/lustre/include/lustre_linkea.h | 57 -- drivers/staging/lustre/lustre/obdclass/linkea.c| 194 - 2 files changed, 251 deletions(-) delete mode 100644 drivers/staging/lustre/lus

[PATCH 1/4] staging/lustre/obdclass: remove unused file md_attrs.c

2014-08-28 Thread Oleg Drokin
It's only used by server side luster for on-disk structure swabbing. And also remove forward declarations for its functions from md_object.h and lustre_idl.h Signed-off-by: Oleg Drokin --- .../lustre/lustre/include/lustre/lustre_idl.h | 6 - drivers/staging/lustre/lustre/include/md_objec

Re: [PATCH v3 00/11] lib: introduce string_escape_mem and %*pE specifier

2014-08-28 Thread John W. Linville
Any objections? Do you (Andy) want me to merge this through the wireless tree? John On Wed, Aug 20, 2014 at 12:42:41PM +0300, Andy Shevchenko wrote: > The introduced function is a kind of opposite to string_unescape. We have > several users of such functionality each of them created custom > im

[PATCH] staging: lustre: mgc: expand the GOTO macro

2014-08-28 Thread Julia Lawall
From: Julia Lawall The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ identifier lbl; identifier rc; constant c; @@ - GOTO(lbl,\(rc\|c\)); + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); + rc; + goto lbl; // Signed-off-by: Julia Law

[PATCH] staging: lustre: mgc: remove unused file

2014-08-28 Thread Julia Lawall
From: Julia Lawall This file contains unresolved references and is not mentioned in the Makefile. Signed-off-by: Julia Lawall --- This should be applied after the return-parenthesis patch. diff --git a/drivers/staging/lustre/lustre/mgc/libmgc.c b/drivers/staging/lustre/lustre/mgc/libmgc.c de

[PATCH] staging: lustre: obdech: remove unused file

2014-08-28 Thread Julia Lawall
From: Julia Lawall This file contains unresolved references and is not mentioned in the Makefile. Signed-off-by: Julia Lawall --- This should be applied after the return-parenthesis patch. diff --git a/drivers/staging/lustre/lustre/obdecho/echo.c b/drivers/staging/lustre/lustre/obdecho/echo.

Re: Fwd: Re: [PATCH] staging:lustre:lnet: lib-md.c fix checkpath warnings and errors.

2014-08-28 Thread Greg KH
On Thu, Aug 28, 2014 at 10:23:05AM +0800, Janet Liu wrote: > On 08/28/2014 10:20 AM, Janet Liu wrote: > > > > > > > > Original Message > >Subject: Re: [PATCH] staging:lustre:lnet: lib-md.c fix checkpath > >warnings and errors. > >Date: Tue, 26 Aug 2014 18:22:45 -0700 > >From: Greg

Re: [PATCH 04/38] Staging: bcm: Bcmchar.c: Renamed variable: "uiBit" -> "bit"

2014-08-28 Thread Dan Carpenter
On Wed, Aug 27, 2014 at 12:57:41PM +0200, Matthias Beyer wrote: > Signed-off-by: Matthias Beyer This is a write-only variable. Just delete it (in a follow on patch is fine). regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org h

Re: [PANIC, hyperv] BUG: unable to handle kernel paging request at ffff880077800004 (hv_ringbuffer_write)

2014-08-28 Thread Sitsofe Wheeler
Hi Dexuan, On Thu, Aug 28, 2014 at 03:21:59AM +, Dexuan Cui wrote: > > > > First let me thank you guys for looking into this issue. Looking at Feel free to add Reported-by: Sitsofe Wheeler to your patches (it's useful too me because it makes it easier for me to show what I've been doing to

Re: [PATCH] staging: lustre: mdc: expand the GOTO macro

2014-08-28 Thread Julia Lawall
On Thu, 28 Aug 2014, Dan Carpenter wrote: > Thanks. These are nice. We need to get rid of the GOTO() macro > entirely. I will. The whole patch is 11,000 lines (including the context code, @@s etc), and I didn't have the patience to check all of that at once... julia

Re: [PATCH] staging: lustre: mdc: expand the GOTO macro

2014-08-28 Thread Dan Carpenter
Thanks. These are nice. We need to get rid of the GOTO() macro entirely. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Re: [PATCH 2/2] android: add sync_fence_create_dma

2014-08-28 Thread Dan Carpenter
On Thu, Aug 28, 2014 at 08:54:05AM +0200, Maarten Lankhorst wrote: > Hey, > > On 15-08-14 08:46, Greg Kroah-Hartman wrote: > > On Thu, Aug 14, 2014 at 11:54:52AM +0200, Maarten Lankhorst wrote: > >> This allows users of dma fences to create a android fence. > > > > Who is going to use these funct

Re: Fwd: Re: [PATCH] staging:lustre:lnet: lib-md.c fix checkpath warnings and errors.

2014-08-28 Thread Dan Carpenter
No worries. But Greg asked you to break the patch up into 4 patches: patch 1: long lines patch 2: spaces patch 3: comments patch 4: trailing statements. Please resend. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://

[PATCH] staging: lustre: mdc: expand the GOTO macro

2014-08-28 Thread Julia Lawall
From: Julia Lawall The semantic patch that makes this change is as follows: (http://coccinelle.lip6.fr/) // @@ identifier lbl; identifier rc; constant c; @@ - GOTO(lbl,\(rc\|c\)); + goto lbl; @@ identifier lbl; expression rc; @@ - GOTO(lbl,rc); + rc; + goto lbl; // Signed-off-by: Julia Law

Re: [PATCH 2/2] android: add sync_fence_create_dma

2014-08-28 Thread Maarten Lankhorst
Hey, On 15-08-14 08:46, Greg Kroah-Hartman wrote: > On Thu, Aug 14, 2014 at 11:54:52AM +0200, Maarten Lankhorst wrote: >> This allows users of dma fences to create a android fence. > > Who is going to use these functions? I need an in-kernel user before I > can add new api calls. So I found a i