On 20/12/16 20:49, Jason Hrycay wrote:
> Add sanity checks for cport_quiesce and cport_clear before invoking the
> callbacks as these function pointers are not required during the host
> device registration. This follows the logic implemented elsewhere for
> various other function pointers.
>
> Si
Hi Roman,
[auto build test ERROR on linus/master]
[also build test ERROR on v4.9 next-20161221]
[if your patch is applied to the wrong git tree, please drop us a note to help
improve the system]
url:
https://github.com/0day-ci/linux/commits/Roman-Kagan/hyperv-more-stuff-to-uapi-cleanup
On Tue, Dec 20, Roman Kagan wrote:
Reverting commit 22356585712d ("staging: hv: use sync_bitops when
interacting with the hypervisor") is save because ...
> - sync_set_bit(channel->monitor_bit,
> + set_bit(channel->monitor_bit,
Olaf
signature.asc
Description: PGP s
On Wed, Dec 21, 2016 at 09:29:39AM +0300, Roman Kagan wrote:
> QEMU in particular. We're planning to implement VMBus devices in QEMU
> and would like to have the definitions shared with the Linux guest
> drivers for Hyper-V.
And that's everything but a userspace API. The way to go for protocol
c
On Wed, Dec 21, 2016 at 04:18:58AM -0800, Christoph Hellwig wrote:
> On Wed, Dec 21, 2016 at 09:29:39AM +0300, Roman Kagan wrote:
> > QEMU in particular. We're planning to implement VMBus devices in QEMU
> > and would like to have the definitions shared with the Linux guest
> > drivers for Hyper-V
On Wed, Dec 21, 2016 at 01:00:44PM +0100, Olaf Hering wrote:
> On Tue, Dec 20, Roman Kagan wrote:
>
> Reverting commit 22356585712d ("staging: hv: use sync_bitops when
> interacting with the hypervisor") is save because ...
>
> > - sync_set_bit(channel->monitor_bit,
> > +
On Wed, Dec 21, 2016 at 03:59:20PM +0300, Roman Kagan wrote:
> That's fine by me.
>
> I guess the series should then start with a complete move
> arch/x86/include/uapi/asm/hyperv.h ->
> arch/x86/include/asm/hyperv_proto.h, and the remaining patches have to
> change the latter instead of the former
Larry Finger wrote:
> With commit e49656147359 {"rtlwifi: Use dev_kfree_skb_irq instead of
> kfree_skb"), the method used to free an skb was changed because the
> kfree_skb() was inside a spinlock. What was forgotten is that kfree_skb()
> guards against a NULL value for the argument. Routine dev_k
On Wed, Dec 21, 2016 at 06:26:54AM -0800, Christoph Hellwig wrote:
> On Wed, Dec 21, 2016 at 03:59:20PM +0300, Roman Kagan wrote:
> > That's fine by me.
> >
> > I guess the series should then start with a complete move
> > arch/x86/include/uapi/asm/hyperv.h ->
> > arch/x86/include/asm/hyperv_proto
This is a patch to the wcmd.h file that fixes up two identifier
name warnings found by the checkpatch.pl tool at lines 54 and 56
Signed-off-by: Tobias Heineken
Signed-off-by: Florian Schleicher
CC: linux-ker...@i4.cs.fau.de
---
drivers/staging/vt6656/wcmd.h | 4 ++--
1 file changed, 2 insertion
???.xls
Description: application/msexcel
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Fix coding style issue caught by checkpatch.pl related to the following
warning:
- "WARNING: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then
dev_err(dev, ... then pr_err(... to printk(KERN_ERR ."
Signed-off-by: Emmanuil Chatzipetru
---
drivers/staging/greybus/svc_watchdog.c |
Fix coding style issue caught by checkpatch.pl related to the following
warning:
- "WARNING: char * array declaration might be better as static const"
Signed-off-by: Emmanuil Chatzipetru
---
drivers/staging/greybus/svc_watchdog.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Fix coding style issue caught by checkpatch.pl related to the following
warning:
- "CHECK: spaces preferred around that '*' (ctx:VxV) "
Signed-off-by: Emmanuil Chatzipetru
---
drivers/staging/greybus/svc_watchdog.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drive
On 21/12/2016 15:26, Christoph Hellwig wrote:
> On Wed, Dec 21, 2016 at 03:59:20PM +0300, Roman Kagan wrote:
>> That's fine by me.
>>
>> I guess the series should then start with a complete move
>> arch/x86/include/uapi/asm/hyperv.h ->
>> arch/x86/include/asm/hyperv_proto.h, and the remaining pat
On Wed, Dec 21, 2016 at 04:39:18PM +0100, Paolo Bonzini wrote:
> That said, there are precedents in using UAPI this way for PV
> interfaces. See for example include/uapi/linux/virtio*.h and
> arch/x86/include/uapi/asm/kvm_para.h.
We have all kinds of historical examples, but most of them turned
i
These drivers need to be able to reference "struct ieee80211_hw" from
the driver's private data, and vice versa. The USB driver failed to
store the address of ieee80211_hw in the private data. Although this
bug has been present for a long time, it was not exposed until
commit ba9f93f82aba ("rtlwifi
On 21/12/2016 16:43, Christoph Hellwig wrote:
> On Wed, Dec 21, 2016 at 04:39:18PM +0100, Paolo Bonzini wrote:
>> That said, there are precedents in using UAPI this way for PV
>> interfaces. See for example include/uapi/linux/virtio*.h and
>> arch/x86/include/uapi/asm/kvm_para.h.
>
> We have al
On Wed, 21 Dec 2016 07:43:48 -0800
Christoph Hellwig wrote:
> On Wed, Dec 21, 2016 at 04:39:18PM +0100, Paolo Bonzini wrote:
> > That said, there are precedents in using UAPI this way for PV
> > interfaces. See for example include/uapi/linux/virtio*.h and
> > arch/x86/include/uapi/asm/kvm_para.h
On 21/12/2016 18:50, Stephen Hemminger wrote:
> The other problem with the hyperv headers is they were initially done with
> only the Linux driver usage in mind. This made perfect sense at the time,
> the problem is that they mix internal state with protocol definitions.
Yes, and this was partly
On Wed, Dec 21, 2016 at 09:50:49AM -0800, Stephen Hemminger wrote:
> Lastly, there is licensing issues on headers. It would be good to have any
> userspace ABI headers licensed with a more liberal license so that BSD and
> DPDK drivers
> could use them directly. Right now each one reinvents.
Micr
> -Original Message-
> From: Roman Kagan [mailto:rka...@virtuozzo.com]
> Sent: Tuesday, December 20, 2016 7:56 AM
> To: Paolo Bonzini ; Radim Krčmář
> ; KY Srinivasan ; Vitaly
> Kuznetsov
> Cc: Thomas Gleixner ; Ingo Molnar
> ; H. Peter Anvin ; x...@kernel.org;
> Haiyang Zhang ; k...@vge
On Wed, 21 Dec 2016 09:58:36 -0800
Christoph Hellwig wrote:
> On Wed, Dec 21, 2016 at 09:50:49AM -0800, Stephen Hemminger wrote:
> > Lastly, there is licensing issues on headers. It would be good to have any
> > userspace ABI headers licensed with a more liberal license so that BSD and
> > DPDK
> -Original Message-
> From: Roman Kagan [mailto:rka...@virtuozzo.com]
> Sent: Tuesday, December 20, 2016 7:56 AM
> To: Paolo Bonzini ; Radim Krčmář
> ; KY Srinivasan ; Vitaly
> Kuznetsov
> Cc: Thomas Gleixner ; Ingo Molnar
> ; H. Peter Anvin ; x...@kernel.org;
> Haiyang Zhang ; k...@vge
According to Documentation/Changes, the minimum gcc version required
to compile the kernel is 3.2 (this is probably outdated too).
Signed-off-by: Cheah Kok Cheong
---
drivers/staging/comedi/comedidev.h | 8 +---
1 file changed, 1 insertion(+), 7 deletions(-)
diff --git a/drivers/staging/com
> -Original Message-
> From: Roman Kagan [mailto:rka...@virtuozzo.com]
> Sent: Tuesday, December 20, 2016 7:56 AM
> To: Paolo Bonzini ; Radim Krčmář
> ; KY Srinivasan ; Vitaly
> Kuznetsov
> Cc: Thomas Gleixner ; Ingo Molnar
> ; H. Peter Anvin ; x...@kernel.org;
> Haiyang Zhang ; k...@vge
> -Original Message-
> From: Roman Kagan [mailto:rka...@virtuozzo.com]
> Sent: Tuesday, December 20, 2016 7:56 AM
> To: Paolo Bonzini ; Radim Krčmář
> ; KY Srinivasan ; Vitaly
> Kuznetsov
> Cc: Thomas Gleixner ; Ingo Molnar
> ; H. Peter Anvin ; x...@kernel.org;
> Haiyang Zhang ; k...@vge
> -Original Message-
> From: Roman Kagan [mailto:rka...@virtuozzo.com]
> Sent: Tuesday, December 20, 2016 7:56 AM
> To: Paolo Bonzini ; Radim Krčmář
> ; KY Srinivasan ; Vitaly
> Kuznetsov
> Cc: Thomas Gleixner ; Ingo Molnar
> ; H. Peter Anvin ; x...@kernel.org;
> Haiyang Zhang ; k...@vge
From: Anna-Maria Gleixner
Install the callbacks via the state machine. No functional change.
Signed-off-by: Anna-Maria Gleixner
Signed-off-by: Sebastian Andrzej Siewior
Cc: de...@driverdev.osuosl.org
Cc: Andreas Dilger
Cc: Greg Kroah-Hartman
Cc: Oleg Drokin
Cc: r...@linutronix.de
Cc: lustre
> -Original Message-
> From: Stephen Hemminger [mailto:step...@networkplumber.org]
> Sent: Wednesday, December 21, 2016 10:03 AM
> To: Christoph Hellwig
> Cc: Paolo Bonzini ; Roman Kagan
> ; Radim Krčmář ; KY
> Srinivasan ; Vitaly Kuznetsov
> ; k...@vger.kernel.org; Denis V . Lunev
> ; H
Currently the greybus-loopback thread logic spins around waiting for
send_count == iteration_max which on real hardware doesn't make a
difference to us but in simulation is excruciatingly slow, anti-social and
bad manners. Use the existing gb_loopback_async_wait_all() function to gate
continuing wh
commit 9250c0ee2626 ("greybus: Loopback_test: use poll instead of
inotify") changes the flow of determining when to break out of a loop
polling for loopback test completion.
The clause is_complete() which determines if all tests are complete - as
used is subject to a race condition where one of th
Two simple patches here resulting from using greybus on gbsim and
developing support for async. I found a bug in the user-space tool and
while doing that decided to update the kernel thread to be better behaved
when waiting for test completions.
V2:
Use available bundle pointer for dev_dbg printou
On 12/20/2016 09:23 PM, Joe Perches wrote:
On Tue, 2016-12-20 at 20:58 -0600, Scott Matheina wrote:
Fixed indention and space issues to align the code with the linux style guide.
[]
diff --git a/drivers/staging/iio/addac/adt7316.c
b/drivers/staging/iio/addac/adt7316.c
[]
@@ -176,16 +176,1
fixed a missing letter in the TODO file 'unneeded'
Signed-off-by: Scott Matheina
---
drivers/staging/dgnc/TODO | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/dgnc/TODO b/drivers/staging/dgnc/TODO
index 0e0825b..6c91bbd 100644
--- a/drivers/staging/dgnc/TODO
35 matches
Mail list logo
