[PATCH 3/3] staging/lustre/quota: Get rid of compat quota code

2014-10-11 Thread Oleg Drokin
Nobody should be using any fo the old lustre 1.8 era tools anymore, so drop compatibility with those. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/dir.c | 58 -- .../staging/lustre/lustre/llite/llite_internal.h | 18 --- 2 files changed, 7

[PATCH 0/3] Lustre: remove old compat code

2014-10-11 Thread Oleg Drokin
These three patches remove support for code that is no longer needed. Mostly ability to talk to old version of lustre clients and servers. Oleg Drokin (3): staging/lustre/ptlrpc: get rid of lustre 1.8 compatible checksums staging/lustre: get rid of deprecaed acl mount option code staging/lus

[PATCH 2/3] staging/lustre: get rid of deprecaed acl mount option code

2014-10-11 Thread Oleg Drokin
This is long since unused code, so get rid of it. Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/llite/llite_lib.c | 19 --- 1 file changed, 19 deletions(-) diff --git a/drivers/staging/lustre/lustre/llite/llite_lib.c b/drivers/staging/lustre/lustre/llite/llite_li

[PATCH 1/3] staging/lustre/ptlrpc: get rid of lustre 1.8 compatible checksums

2014-10-11 Thread Oleg Drokin
Old version of lustre (1.8) calculated RPC checksums differently, but the compat code is no longer important since in-kernel version of lustre is nto compatible with 1.8 lustre servers (that are long deprecated too). Signed-off-by: Oleg Drokin --- drivers/staging/lustre/lustre/include/lustre_net

Re: [PATCH 1/1] drivers/staging: Fixed sparse error "directive in argument list"

2014-10-11 Thread Filipe Gonçalves
Hi, Oops.. sorry for the mess then. I am still trying to get to know the codebase. I submitted a new patch where I removed the check. Thanks for your patience :) Best, Filipe On Sat, Oct 11, 2014 at 10:49 PM, Drokin, Oleg wrote: > > Hello! > >No, it's not the way to test the kernel version

Re: [PATCH 1/1] drivers/staging: Fixed sparse error "directive in argument list"

2014-10-11 Thread Filipe Gonçalves
Ah .. right! I didn't know what OBD_OCD_VERSION() was. Now I see it's a way to test kernel version. I am going to submit a new patch shortly. Thanks, Filipe On Sat, Oct 11, 2014 at 9:15 PM, Greg KH wrote: > > On Sat, Oct 11, 2014 at 08:13:42PM +0100, Filipe Gonçalves wrote: >> This patch fixes a

RE: [PATCH 0/3] scsi: Add Hyper-V logical block provisioning quirks

2014-10-11 Thread KY Srinivasan
> -Original Message- > From: Jeff Leung [mailto:jle...@v10networks.ca] > Sent: Saturday, October 11, 2014 1:22 PM > To: KY Srinivasan; James Bottomley; Christoph Hellwig > Cc: Sitsofe Wheeler; Haiyang Zhang; Christoph Hellwig; Hannes Reinecke; > linux-s...@vger.kernel.org; linux-ker...@vg

Re: [PATCH 1/1] drivers/staging: Fixed sparse error "directive in argument list"

2014-10-11 Thread Drokin, Oleg
Hello! No, it's not the way to test the kernel version, it's the way to test internal lustre version. Either way maintaining compatibility with Lustre 1.8 and 2.0 servers should not be important anymore, so it's fine to drop this check indeed. Bye, Oleg On Oct 11, 2014, at 5:06

[PATCH 1/1] drivers/staging: Removed unnecessary check for LUSTRE_VERSION_CODE

2014-10-11 Thread Filipe Goncalves
From: Filipe Gonçalves Given that this code is now part of the kernel tree, the #ifdef directive comparing LUSTRE_VERSION_CODE to the kernel version is no longer needed. As a side effect, this will also remove the sparse error "directive in argument list". Signed-off-by: Filipe Gonçalves ---

RE: [PATCH 0/3] scsi: Add Hyper-V logical block provisioning quirks

2014-10-11 Thread Jeff Leung
> On the current release of Windows (windows 10), we are advertising SPC3 > compliance. > We are ok with declaring compliance to SPC3 in our drivers. If you are going to declare SPC3 compliance in the drivers, are you going to put in checks to ensure that SPC-3 compliance doesn't get accidentally

Re: [PATCH 1/1] drivers/staging: Fixed sparse error "directive in argument list"

2014-10-11 Thread Greg KH
On Sat, Oct 11, 2014 at 08:13:42PM +0100, Filipe Gonçalves wrote: > This patch fixes a sparse warning on layout.c (ptlrpc) that was caused by > having preprocessor directives in the arguments to a macro. > > Signed-off-by: Filipe Gonçalves > --- > drivers/staging/lustre/lustre/ptlrpc/layout.c |

RE: [PATCH 0/3] scsi: Add Hyper-V logical block provisioning quirks

2014-10-11 Thread KY Srinivasan
> -Original Message- > From: James Bottomley [mailto:james.bottom...@hansenpartnership.com] > Sent: Saturday, October 11, 2014 10:42 AM > To: Christoph Hellwig > Cc: Sitsofe Wheeler; KY Srinivasan; Haiyang Zhang; Christoph Hellwig; Hannes > Reinecke; linux-s...@vger.kernel.org; linux-ker.

[PATCH 1/1] drivers/staging: Fixed sparse error "directive in argument list"

2014-10-11 Thread Filipe Gonçalves
This patch fixes a sparse warning on layout.c (ptlrpc) that was caused by having preprocessor directives in the arguments to a macro. Signed-off-by: Filipe Gonçalves --- drivers/staging/lustre/lustre/ptlrpc/layout.c | 7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/driv

Re: [PATCH 0/3] scsi: Add Hyper-V logical block provisioning quirks

2014-10-11 Thread James Bottomley
On Sat, 2014-10-11 at 10:39 -0700, Christoph Hellwig wrote: > On Fri, Oct 10, 2014 at 08:49:01AM +0100, Sitsofe Wheeler wrote: > > Microsoft Hyper-V virtual disks currently only claim SPC-2 compliance > > even though they implement post SPC-2 features (such as thin > > provisioning) which means the

Re: [PATCH 0/3] scsi: Add Hyper-V logical block provisioning quirks

2014-10-11 Thread Christoph Hellwig
On Fri, Oct 10, 2014 at 08:49:01AM +0100, Sitsofe Wheeler wrote: > Microsoft Hyper-V virtual disks currently only claim SPC-2 compliance > even though they implement post SPC-2 features (such as thin > provisioning) which means the Linux kernel does not go on to test for > those features even thoug

Re: [PATCH 11/15] staging: rtl8723au: Remove a stack of write only counters

2014-10-11 Thread Greg KH
On Sat, Oct 11, 2014 at 05:32:49AM -0700, Greg KH wrote: > On Fri, Oct 10, 2014 at 09:41:34PM +0200, jes.soren...@redhat.com wrote: > > From: Jes Sorensen > > > > We don't do anything with these, so get rid of them > > > > Signed-off-by: Jes Sorensen > > --- > > drivers/staging/rtl8723au/core/

Re: [PATCH 11/15] staging: rtl8723au: Remove a stack of write only counters

2014-10-11 Thread Greg KH
On Fri, Oct 10, 2014 at 09:41:34PM +0200, jes.soren...@redhat.com wrote: > From: Jes Sorensen > > We don't do anything with these, so get rid of them > > Signed-off-by: Jes Sorensen > --- > drivers/staging/rtl8723au/core/rtw_xmit.c| 6 -- > drivers/staging/rtl8723au/include/rtw_xm

Re: [PATCH net 1/1] hyperv: Fix a bug in netvsc_send()

2014-10-11 Thread Sitsofe Wheeler
On Fri, Oct 10, 2014 at 11:39:00PM +, Long Li wrote: > Thanks Sitsofe. Can you provide more details on the test setup? > > The kernel trace shows that skb->mac_header=0x (which means not > yet set, it's in RCX: ). See reply below. > -Original Message- > From: deve

Re: [PATCH] Staging: lustre: llite: fix some sparse warnings about userspace pointer

2014-10-11 Thread Sudip Mukherjee
On Sat, Oct 11, 2014 at 01:31:17PM +0700, Anh Le wrote: > Add __user macro to the function declarations that accept userspace pointers > as > arguments. > if you check your patch with --strict option of checkpatch.pl , you will see many check of : Alignment should match open parenthesis thanks s