On 11/02/2015 10:42 PM, KY Srinivasan wrote:
-Original Message-
From: Denis V. Lunev [mailto:d...@openvz.org]
Sent: Monday, November 2, 2015 3:34 AM
Cc: rka...@virtuozzo.com; de...@linuxdriverproject.org; linux-
ker...@vger.kernel.org; Andrey Smetanin ; KY
Srinivasan ; Haiyang Zhang
; V
On Wed, Nov 11, 2015 at 3:39 PM, Dennis Dalessandro
wrote:
> On Wed, Nov 11, 2015 at 08:25:35AM +0200, Leon Romanovsky wrote:
>> On Wed, Nov 11, 2015 at 12:34:37AM -0500, ira.we...@intel.com wrote:
>>> From: Dennis Dalessandro
>>> +int snoop_send_dma_handler(struct hfi1_qp *qp, struct hfi1_pkt_st
On Sat, Oct 31, 2015 at 12:41 AM, wrote:
> From: Mitko Haralanov
>
> Add mmu notify helper functions and TID caching function stubs in preparation
> for the TID caching implementation.
>
> TID caching makes use of the MMU notifier to allow the driver to respond to
> the
> user freeing memory wh
sdio_init and wilc_spi_init always return error, so it fails everytime.
Fix this by removing else statement.
This fixes c1af9db78950a778ec18343c5c5a6d4cfbf58a61,
bacd388547f55eb415ce266c8723cc6f1281a25d
Signed-off-by: Glen Lee
---
drivers/staging/wilc1000/wilc_sdio.c | 2 --
drivers/staging/wil
From: Dexuan Cui
We want to simplify vmbus_onoffer_rescind() by not invoking
hv_process_channel_removal(NULL, ...).
Signed-off-by: Dexuan Cui
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/channel_mgmt.c | 21 +++--
1 files changed, 15 insertions(+), 6 deletions(-)
diff --g
From: Dexuan Cui
This fixes an incorrect assumption of channel state in the function.
Signed-off-by: Dexuan Cui
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/channel.c | 12
1 files changed, 12 insertions(+), 0 deletions(-)
diff --git a/drivers/hv/channel.c b/drivers/hv/chann
From: Kamal Mostafa
Use the local uapi headers to keep in sync with "recently" added #define's
(e.g. VSS_OP_REGISTER1).
Fixes: 3eb2094c59e89db2bedd401e23c7a870081c9edb
Signed-off-by: Kamal Mostafa
Signed-off-by: K. Y. Srinivasan
---
tools/hv/Makefile |2 ++
1 files changed, 2 insertions(+
The irq we extract from ACPI is not used - we deliver hypervisor
interrupts on a special vector. Make the necessary adjustments.
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/vmbus_drv.c | 16 +++-
1 files changed, 3 insertions(+), 13 deletions(-)
diff --git a/drivers/hv/vmbus_dr
From: Olaf Hering
The Backup integration service on WS2012 has appearently trouble to
negotiate with a guest which does not support the provided util version.
Currently the VSS driver supports only version 5/0. A WS2012 offers only
version 1/x and 3/x, and vmbus_prep_negotiate_resp correctly retu
From: Dexuan Cui
spinlock is unnecessary here.
mutex is enough.
Signed-off-by: Dexuan Cui
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/channel_mgmt.c | 12 ++--
drivers/hv/connection.c |7 +++
drivers/hv/hyperv_vmbus.h |2 +-
3 files changed, 10 insertions(+), 11 del
From: Dexuan Cui
In the path vmbus_onoffer_rescind() -> vmbus_device_unregister() ->
device_unregister() -> ... -> __device_release_driver(), we can see for a
device without a driver loaded: dev->driver is NULL, so
dev->bus->remove(dev), namely vmbus_remove(), isn't invoked.
As a result, vmbus_
Use uuid_le_cmp() for comparing GUIDs.
Signed-off-by: K. Y. Srinivasan
---
V2: Fixed the typo in both the commit log and Subject line.
drivers/hv/channel_mgmt.c |3 +--
drivers/hv/vmbus_drv.c|4 ++--
2 files changed, 3 insertions(+), 4 deletions(-)
diff --git a/drivers/hv/c
From: Dexuan Cui
process_chn_event(), running in the tasklet, can race with
vmbus_close_internal() in the case of SMP guest, e.g., when the former is
accessing channel->inbound.ring_buffer, the latter could be freeing the
ring_buffer pages.
To resolve the race, we can serialize them by disabling
Consistently use uuid_le type in the Hyper-V driver code.
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/channel_mgmt.c |2 +-
drivers/hv/vmbus_drv.c | 10 ++--
include/linux/hyperv.h | 92 +-
include/linux/mod_devicetable.h |
From: Jake Oshins
This patch makes 16GB GPUs work in Hyper-V VMs, since, for
compatibility reasons, the Hyper-V BIOS lists MMIO ranges in 2GB
chunks in its root bus's _CRS object.
Signed-off-by: Jake Oshins
Signed-off-by: K. Y. Srinivasan
---
drivers/hv/vmbus_drv.c | 16
1
The macro VMBUS_DEVICE() is unused; get rid of it.
Signed-off-by: K. Y. Srinivasan
---
include/linux/hyperv.h | 13 -
1 files changed, 0 insertions(+), 13 deletions(-)
diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h
index ee243a7..437c9c8 100644
--- a/include/linux/hy
Miscellaneous fixes and cleanup.
Changes from V1:
Fixed the typo in patch number 3.
Included the patch tools/hv: Use include/uapi with __EXPORTED_HEADERS__
Dexuan Cui (5):
Drivers: hv: vmbus: serialize process_chn_event() and
vmbus_close_internal()
Drivers: hv: vmbus: do sanity check of c
Good day,
I wish to contact you personally for an important proposal that might be
of interest to you. I am sending this mail just to know if this email address
is functional.
I have something absolutely essential to discuss with you. Contact me for
details through my private email: shu...@qq.c
Use the local uapi headers to keep in sync with "recently" added #define's
(e.g. VSS_OP_REGISTER1).
Fixes: 3eb2094c59e89db2bedd401e23c7a870081c9edb
Signed-off-by: Kamal Mostafa
---
tools/hv/Makefile | 2 ++
1 file changed, 2 insertions(+)
diff --git a/tools/hv/Makefile b/tools/hv/Makefile
index
On Wed, Nov 11, 2015 at 9:48 AM, Marcos Oviedo wrote:
> Fixing code style space issues at rtw_mcs_rate23a()
>
> Signed-off-by: Marcos Oviedo
> ---
> drivers/staging/rtl8723au/core/rtw_ieee80211.c | 80
> ++--
> 1 file changed, 48 insertions(+), 32 deletions(-)
>
> diff --g
Fixing code style space issues at rtw_mcs_rate23a()
Signed-off-by: Marcos Oviedo
---
drivers/staging/rtl8723au/core/rtw_ieee80211.c | 80 ++--
1 file changed, 48 insertions(+), 32 deletions(-)
diff --git a/drivers/staging/rtl8723au/core/rtw_ieee80211.c
b/drivers/staging/r
On Wednesday 11 November 2015 15:14:48 Mauro Carvalho Chehab wrote:
> rename include/media/{ => platform}/exynos-fimc.h (100%)
> rename include/media/{ => platform}/mmp-camera.h (100%)
> rename include/media/{ => platform}/omap1_camera.h (100%)
> rename include/media/{ => platform}/omap4iss.h (
Let's not mix headers used by the core with those headers that
are needed by some specific platform drivers.
This patch was made via this script:
mkdir include/media/platform
for i in include/media/*.h; do n=`basename $i`; (for j in $(git grep
-l $n); do dirname $j; done)|sort|u
> > [also build test ERROR on v4.3 next-2015]
>> >
>> > url:
>> > https://github.com/0day-ci/linux/commits/Shivani-Bhardwaj/staging-lustre-cl_lock-Remove-cl_lock_lockdep_init-wrapper/2015-182452
>> > config: m68k-allyesconfig (attached as .
> -Original Message-
> From: Andy Shevchenko [mailto:andy.shevche...@gmail.com]
> Sent: Wednesday, November 11, 2015 12:35 AM
> To: KY Srinivasan
> Cc: Greg Kroah-Hartman ; linux-
> ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de;
> Robo Bot ; Vitaly Kuznetsov ;
> ja
On Wed, Nov 11, 2015 at 10:51:36PM +0530, Shivani Bhardwaj wrote:
> On Wed, Nov 11, 2015 at 4:24 PM, kbuild test robot wrote:
> > Hi Shivani,
> >
> > [auto build test ERROR on staging/staging-testing]
> > [also build test ERROR on v4.3 next-2015]
> >
> &g
On Wed, Nov 11, 2015 at 03:03:12PM +, Luick, Dean wrote:
> > -Original Message-
> > From: Dan Carpenter [mailto:dan.carpen...@oracle.com]
> > Sent: Wednesday, November 11, 2015 8:39 AM
> > To: Luick, Dean
> > Cc: John, Jubin ; de...@driverdev.osuosl.org;
> > gre...@linuxfoundation.org;
On Wed, Nov 11, 2015 at 4:24 PM, kbuild test robot wrote:
> Hi Shivani,
>
> [auto build test ERROR on staging/staging-testing]
> [also build test ERROR on v4.3 next-2015]
>
> url:
> https://github.com/0day-ci/linux/commits/Shivani-Bhardwaj/staging-
BIT macro is used for defining BIT location instead of
shifting operator - coding style issue
Signed-off-by: Ranjith Thangavel
---
drivers/staging/comedi/drivers/cb_pcidda.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/staging/comedi/drivers/cb_pcidda.c
b
BIT macro is used for defining BIT location instead of
shifting operator - coding style issue
Signed-off-by: Ranjith Thangavel
---
drivers/staging/comedi/drivers/ni_65xx.c | 54 +++---
1 file changed, 27 insertions(+), 27 deletions(-)
diff --git a/drivers/staging/comed
BIT macro is used for defining BIT location instead of
shifting operator - coding style issue
Signed-off-by: Ranjith Thangavel
---
drivers/staging/comedi/drivers/ni_6527.c | 24
1 file changed, 12 insertions(+), 12 deletions(-)
diff --git a/drivers/staging/comedi/driv
BIT macro is used for defining BIT location instead of
shifting operator - coding style issue
Signed-off-by: Ranjith Thangavel
---
drivers/staging/comedi/drivers/comedi_parport.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/comedi/drivers/comedi_parp
BIT macro is used for defining BIT location instead of
shifting operator - coding style issue
Signed-off-by: Ranjith Thangavel
---
drivers/staging/comedi/drivers/pcmmio.c | 44 +++
1 file changed, 22 insertions(+), 22 deletions(-)
diff --git a/drivers/staging/comed
A: No.
Q: Should I include quotations after my reply?
http://daringfireball.net/2007/07/on_top
On Wed, Nov 11, 2015 at 07:15:28PM +0530, Ranjith T wrote:
> Is this patch is fine?..
You sent it a few days ago, please wait at least 2 weeks before worrying
if a patch is accepted or not, we all are
On Wed, Nov 11, 2015 at 08:39:08AM -0500, Dennis Dalessandro wrote:
> On Wed, Nov 11, 2015 at 08:25:35AM +0200, Leon Romanovsky wrote:
> >On Wed, Nov 11, 2015 at 12:34:37AM -0500, ira.we...@intel.com wrote:
> >>From: Dennis Dalessandro
> >>
> >>+int snoop_send_dma_handler(struct hfi1_qp *qp, struc
On Wed, Nov 11, 2015 at 12:01:08PM +0300, Dan Carpenter wrote:
> On Wed, Nov 11, 2015 at 12:43:05AM -0500, ira.we...@intel.com wrote:
> > From: Ira Weiny
> >
> > This goto done is followed by an if (ret) break in the outer switch clause.
> > It
> > is unnecessary.
> >
> > Signed-off-by: Dennis
> -Original Message-
> From: Dan Carpenter [mailto:dan.carpen...@oracle.com]
> Sent: Wednesday, November 11, 2015 8:39 AM
> To: Luick, Dean
> Cc: John, Jubin ; de...@driverdev.osuosl.org;
> gre...@linuxfoundation.org; dledf...@redhat.com; linux-
> r...@vger.kernel.org
> Subject: Re: [PATCH
On Wed, Nov 11, 2015 at 02:03:02PM +, Luick, Dean wrote:
>
>
> > -Original Message-
> > From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma-
> > ow...@vger.kernel.org] On Behalf Of Dan Carpenter
> > Sent: Wednesday, November 11, 2015 2:45 AM
> > To: John, Jubin
> > Cc: gre...@l
On 11/11/15 14:18, Ranjith T wrote:
Actually my name is Ranjith and father name is Thangavel. So I used to
mention my name as Ranjith T.
How do you write your name on legal documents? That is what you should
use on the patch's "Signed-off-by:" line, and on the email's "From:" line.
Thanks.
Actually my name is Ranjith and father name is Thangavel. So I used to
mention my name as Ranjith T.
Thanks,
Ranjith T
On Wed, Nov 11, 2015 at 7:41 PM, Ian Abbott wrote:
> On 11/11/2015 13:44, Ranjith T wrote:
>>
>> Is this patch is fine?.
>>
>> Thanks,
>> Ranjith T.
>
>
> Fine apart from the "
On 11/11/2015 13:44, Ranjith T wrote:
Is this patch is fine?.
Thanks,
Ranjith T.
Fine apart from the "Signed-off-by" line. I'm guessing your last name
isn't really "T".
On Mon, Nov 9, 2015 at 11:18 PM, Ranjith T wrote:
BIT macro is used for defining BIT location instead of
shifting oper
On 10/11/2015 14:41, Geliang Tang wrote:
Use kmalloc_array instead of kmalloc to allocate memory for an array.
Signed-off-by: Geliang Tang
---
Changes in v2:
- preserve the existing whitespace style.
---
drivers/staging/comedi/drivers/amplc_pci224.c | 11 +++
drivers/staging/comed
> -Original Message-
> From: linux-rdma-ow...@vger.kernel.org [mailto:linux-rdma-
> ow...@vger.kernel.org] On Behalf Of Dan Carpenter
> Sent: Wednesday, November 11, 2015 2:45 AM
> To: John, Jubin
> Cc: gre...@linuxfoundation.org; de...@driverdev.osuosl.org; linux-
> r...@vger.kernel.org
On Wed, Nov 11, 2015 at 12:03:36PM +0300, Dan Carpenter wrote:
On Wed, Nov 11, 2015 at 12:43:05AM -0500, ira.we...@intel.com wrote:
From: Ira Weiny
This goto done is followed by an if (ret) break in the outer switch clause. It
is unnecessary.
Signed-off-by: Dennis Dalessandro
Signed-off-by:
Is this patch is fine?..
Thanks,
Ranjith T.
On Mon, Nov 9, 2015 at 11:14 PM, Ranjith T wrote:
> BIT macro is used for defining BIT location instead of
> shifting operator - coding style issue
>
> Signed-off-by: Ranjith T
> ---
> drivers/staging/comedi/drivers/pcmmio.c | 44
> +++
Is this patch is fine?.
Thanks,
Ranjith T.
On Mon, Nov 9, 2015 at 11:18 PM, Ranjith T wrote:
> BIT macro is used for defining BIT location instead of
> shifting operator - coding style issue
>
> Signed-off-by: Ranjith T
> ---
> drivers/staging/comedi/drivers/comedi_parport.c |4 ++--
> 1 f
On Wed, Nov 11, 2015 at 08:25:35AM +0200, Leon Romanovsky wrote:
On Wed, Nov 11, 2015 at 12:34:37AM -0500, ira.we...@intel.com wrote:
From: Dennis Dalessandro
+int snoop_send_dma_handler(struct hfi1_qp *qp, struct hfi1_pkt_state *ps,
+ u64 pbc)
{
- pr_alert("Sno
Hi Arnd,
[auto build test ERROR on staging/staging-testing]
[cannot apply to v4.3 next-2015]
url:
https://github.com/0day-ci/linux/commits/Arnd-Bergmann/staging-wilc1000-cleanups-once-again/2015-074846
config: i386-allmodconfig (attached as .config)
reproduce:
# save the
Hi Shivani,
[auto build test ERROR on staging/staging-testing]
[also build test ERROR on v4.3 next-2015]
url:
https://github.com/0day-ci/linux/commits/Shivani-Bhardwaj/staging-lustre-cl_lock-Remove-cl_lock_lockdep_init-wrapper/2015-182452
config: m68k-allyesconfig (attached as
Remove the wrapper function cl_lock_lockdep_init() and replace its only
call with the function it wrapped.
Signed-off-by: Shivani Bhardwaj
---
drivers/staging/lustre/lustre/obdclass/cl_lock.c | 7 +--
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/drivers/staging/lustre/lustre/
On 11/11/2015 04:43 PM, Dan Carpenter wrote:
On Sun, Nov 08, 2015 at 02:11:08PM +0700, Ivan Safonov wrote:
_rtl88e_fill_dummy fills the array elemets with zeros if necessary.
_rtl88e_fill_dummy() fills the last sizeof(u32) or whatever so the code
is aligned but it doesn't fill up to the end of
Remove the wrapper function cl_2queue_add() and replace all its calls in
different files with the function it wrapped. Also, comments are added
wherever necessary to make the working of function clear. Prototype of
the function is also removed from the header file as it is no longer
needed.
Signed
On Sun, Nov 08, 2015 at 02:11:08PM +0700, Ivan Safonov wrote:
> _rtl88e_fill_dummy fills the array elemets with zeros if necessary.
>
_rtl88e_fill_dummy() fills the last sizeof(u32) or whatever so the code
is aligned but it doesn't fill up to the end of FW_8188E_SIZE. Why do
we even allocate tha
On Wed, Nov 11, 2015 at 01:10:40AM -0500, ira.weiny wrote:
> The original author and I have been going through the code to see what we can
> do. We have identified a couple of other pieces which can be split.
>
> One question. Is it ok to have functionality which is added which is unused
> in
>
On Wed, Nov 11, 2015 at 12:43:06AM -0500, ira.we...@intel.com wrote:
> From: Ira Weiny
>
> Set link state was not supported and so we can return early in the parameter
> checks rather than falling through the switch clause.
>
> Signed-off-by: Dennis Dalessandro
> Signed-off-by: Ira Weiny
> ---
On Wed, Nov 11, 2015 at 12:43:05AM -0500, ira.we...@intel.com wrote:
> From: Ira Weiny
>
> This goto done is followed by an if (ret) break in the outer switch clause.
> It
> is unnecessary.
>
> Signed-off-by: Dennis Dalessandro
> Signed-off-by: Ira Weiny
Also these sign offs don't really ma
On Wed, Nov 11, 2015 at 12:43:05AM -0500, ira.we...@intel.com wrote:
> From: Ira Weiny
>
> This goto done is followed by an if (ret) break in the outer switch clause.
> It
> is unnecessary.
>
> Signed-off-by: Dennis Dalessandro
> Signed-off-by: Ira Weiny
> ---
> drivers/staging/rdma/hfi1/di
Remove unnecessary wrapper function cl_lock_descr_fid() and replace all
its calls with the function it wrapped.
Signed-off-by: Shivani Bhardwaj
---
drivers/staging/lustre/lustre/obdclass/cl_io.c | 12
1 file changed, 4 insertions(+), 8 deletions(-)
diff --git a/drivers/staging/lust
On Wednesday 11 November 2015 09:54:57 Andy Shevchenko wrote:
> > diff --git a/drivers/staging/wilc1000/linux_wlan_common.h
> > b/drivers/staging/wilc1000/linux_wlan_common.h
> > index f2ea8280b8f8..72b524a98cba 100644
> > --- a/drivers/staging/wilc1000/linux_wlan_common.h
> > +++ b/drivers/stagin
On Wed, Nov 11, 2015 at 02:33:32AM -0500, Jubin John wrote:
> +static int read_efi_var(const char *name, unsigned long *size,
> + void **return_data)
> +{
> + int ret;
> +
> + /* set failure return values */
> + *size = 0;
> + *return_data = NULL;
> +
> + /*
On Wed, Nov 11, 2015 at 3:59 AM, K. Y. Srinivasan wrote:
> Use the uuid_le_cmp() macro for comparing GUIDs.
It's not a macro as I see.
>
> Signed-off-by: K. Y. Srinivasan
> ---
> drivers/hv/channel_mgmt.c |3 +--
> drivers/hv/vmbus_drv.c|4 ++--
> 2 files changed, 3 insertions(+),
On Wed, Nov 11, 2015 at 02:33:30AM -0500, Jubin John wrote:
> @@ -8288,6 +8367,21 @@ static int init_cntrs(struct hfi1_devdata *dd)
> dd->ndevcntrs++;
> index++;
> }
> + } else if (dev_cntrs[i].flags & CNT
On Wed, 2015-11-11 at 00:42 +0100, Arnd Bergmann wrote:
> A recent cleanup added a reference to struct net_device, but
> that structure is not always visible in the context of the
> declaration, so we may get a compile-time error:
>
> In file included from wilc1000/host_interface.c:5:0:
> wilc1000
On Tue, Nov 10, 2015 at 07:35:30PM +0100, Daniel H. Hemmingsen wrote:
> Fixed a spacing after cast coding style issue.
>
> Signed-off-by: Daniel H. Hemmingsen
Acked-by: Willy Tarreau
Thanks,
Willy
___
devel mailing list
de...@linuxdriverproject.org
64 matches
Mail list logo
