Re: staging: lustre: One function call less in class_register_type() after error detection

2016-07-27 Thread SF Markus Elfring
> This function is called several times during lustre module insert. > Namely it's called 5 times for 5 types: > osc, mdc, lov, lmv, mgc. Will any extra memory accesses matter for the successful execution in this use case? > It's not called any more than that, so it's not exactly a super hot-pat

RE: [PATCH 1/1] Drivers: infiniband: hw: vmbus-nd: NetworkDirect driver for Linux

2016-07-27 Thread KY Srinivasan
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Wednesday, July 27, 2016 2:26 PM > To: KY Srinivasan > Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; linux- > r...@vger.kernel.org; yish...@mellanox.com; sean.he...@intel.com; > dledf...@re

RE: [PATCH 1/1] Drivers: infiniband: hw: vmbus-nd: NetworkDirect driver for Linux

2016-07-27 Thread KY Srinivasan
> -Original Message- > From: Leon Romanovsky [mailto:l...@kernel.org] > Sent: Tuesday, July 26, 2016 9:25 PM > To: KY Srinivasan > Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; > de...@linuxdriverproject.org; linux-r...@vger.kernel.org; > yish...@mellanox.com; sean.he...@

Re: [PATCH 1/1] Drivers: infiniband: hw: vmbus-nd: NetworkDirect driver for Linux

2016-07-27 Thread Greg KH
On Wed, Jul 27, 2016 at 09:09:08PM +, KY Srinivasan wrote: > > > > -Original Message- > > From: Greg KH [mailto:gre...@linuxfoundation.org] > > Sent: Tuesday, July 26, 2016 9:41 PM > > To: KY Srinivasan > > Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; linux- > > r.

RE: [PATCH 1/1] Drivers: infiniband: hw: vmbus-nd: NetworkDirect driver for Linux

2016-07-27 Thread KY Srinivasan
> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Tuesday, July 26, 2016 9:41 PM > To: KY Srinivasan > Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; linux- > r...@vger.kernel.org; yish...@mellanox.com; sean.he...@intel.com; > dledf...@redh

RE: [PATCH RESEND 00/28] staging: unisys: fix visorbus & visorinput issues raised by tglx

2016-07-27 Thread Kershner, David A
> -Original Message- > From: gre...@linuxfoundation.org [mailto:gre...@linuxfoundation.org] > Sent: Thursday, June 16, 2016 3:03 PM > To: Kershner, David A > Cc: Neil Horman ; driverdev- > de...@linuxdriverproject.org; *S-Par-Maintainer > ; jes.soren...@redhat.com; > t...@linutronix.de; Bi

Re: [PATCH RESEND 00/28] staging: unisys: fix visorbus & visorinput issues raised by tglx

2016-07-27 Thread gre...@linuxfoundation.org
On Wed, Jul 27, 2016 at 05:43:27PM +, Kershner, David A wrote: > > -Original Message- > > From: gre...@linuxfoundation.org [mailto:gre...@linuxfoundation.org] > > Sent: Thursday, June 16, 2016 3:03 PM > > To: Kershner, David A > > Cc: Neil Horman ; driverdev- > > de...@linuxdriverproje

completion issues in ks7010

2016-07-27 Thread Nicholas Mc Guire
Hi ! staging ks7010 has a a few completion calls, some of which are using wait_for_completion_interruptible_timeout(). The key difference betweeen wait_for_completion_interruptible_timeout() and wait_for_completion_timeout() is, if I got it right, only that the former will return on any pen

[PATCH 2/2 RFC] staging: ks7010: drop unused empty function

2016-07-27 Thread Nicholas Mc Guire
declared. Should it be static? but ks_wlan_reset is unused and effectively an empty function so it can be dropped. Compile tested with: x86_64_defconfig + CONFIG_STAGING=y CONFIG_MMC=m, CONFIG_KS7010=m Patch is against 4.7-rc7 (localversion-next -next-20160727) drivers/staging/ks7010/ks_wla

[PATCH 1/2 RFC] staging: ks7010: fix location of declaration

2016-07-27 Thread Nicholas Mc Guire
ng: symbol 'send_packet_complete' was not declared. Should it be static? was caused by build dependencies being resolved locally rather than at device level. Compile tested with: x86_64_defconfig + CONFIG_STAGING=y CONFIG_MMC=m, CONFIG_KS7010=m Patch is against 4.7-rc7 (localversion-next -next

[PATCH 0/2 RFC] staging: ks7010: fix sparse warnings

2016-07-27 Thread Nicholas Mc Guire
Two proposed fixes for sparse warnings drivers/staging/ks7010/ks_wlan_net.c:3392:6: warning: symbol 'send_packet_complete' was not declared. Should it be static? looking at how it is used I guess it should not be static but the extern declaration moved into ks_wlan.h but all the functi

RE: [PATCH v18 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-07-27 Thread Dexuan Cui
> From: David Miller [mailto:da...@davemloft.net] > Sent: Wednesday, July 27, 2016 1:45 > To: Dexuan Cui > > From: Dexuan Cui > Date: Tue, 26 Jul 2016 07:09:41 + > > > I googled "S390 hypervisor socket" but didn't find anything related (I > > think). > > That would be net/iucv/ Thanks for

RE: [PATCH v18 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-07-27 Thread Dexuan Cui
> From: netdev-ow...@vger.kernel.org [mailto:netdev- > ow...@vger.kernel.org] On Behalf Of Dexuan Cui > Sent: Tuesday, July 26, 2016 21:22 > ... > This is because, the design of AF_HYPERV in the Hyper-V host side is > suboptimal IMHO (the current host side design requires the least > change in the