[PATCH v7] Revert "PCI: hv: Use device serial number as PCI domain"

2018-04-11 Thread Sridhar Pitchai
When Linux runs as a guest VM in Hyper-V and Hyper-V adds the virtual PCI bus to the guest, Hyper-V always provides unique PCI domain. commit 4a9b0933bdfc ("PCI: hv: Use device serial number as PCI domain") overrode unique domain with the serial number of the first device added to the virtual PCI

Re: [PATCH v6] PCI: hv: Make sure the bus domain is really unique

2018-04-11 Thread Bjorn Helgaas
I would make the subject: Revert "PCI: hv: Use device serial number as PCI domain" so it matches the original commit. Otherwise you really need quite a lot of driver internal knowledge to connect them. On Wed, Apr 11, 2018 at 12:36:25AM +, Sridhar Pitchai wrote: > When Linux runs as a

Re: [PATCH] staging: Android: Add 'vsoc' driver for cuttlefish.

2018-04-11 Thread Dan Carpenter
On Wed, Apr 11, 2018 at 10:49:47AM -0700, Greg Hartman wrote: > > + /* Ensure that the offset is within shared memory */ > > + if (((uint64_t)arg->offset) + region_p->region_begin_offset + > > + sizeof(uint32_t) > region_p->region_end_offset) > > I'm not certain if I understand

[PATCH v3 char-misc 1/1] x86/hyperv: Add interrupt handler annotations

2018-04-11 Thread mhkelley58
From: Michael Kelley Add standard interrupt handler annotations to hyperv_vector_handler(). This does not fix any observed bug, but avoids potential removal of the code by link time optimization and makes it consistent with hv_stimer0_vector_handler in the same source

Re: [PATCH 1/2] staging: irda: Replace mdelay with usleep_range in stir421x_fw_upload

2018-04-11 Thread Jia-Ju Bai
On 2018/4/11 22:26, David Miller wrote: From: Jia-Ju Bai Date: Wed, 11 Apr 2018 16:20:22 +0800 Okay, I now know why many of my patches were not replied. Many of your patches are not responded to because you handle patch feedback poorly sometimes. Okay, thanks for

Re: [PATCH 1/2] staging: irda: Replace mdelay with usleep_range in stir421x_fw_upload

2018-04-11 Thread David Miller
From: Jia-Ju Bai Date: Wed, 11 Apr 2018 16:20:22 +0800 > Okay, I now know why many of my patches were not replied. Many of your patches are not responded to because you handle patch feedback poorly sometimes. Also, all of your networking submissions have been dropped

Re: [PATCH] staging: comedi: cb_pcidas64: fix alignment of function parameters

2018-04-11 Thread Ian Abbott
On 10/04/18 23:50, Gabriel Francisco Mandaji wrote: Fix most checkpatch.pl issues of type: CHECK: Alignment should match open parenthesis Signed-off-by: Gabriel Francisco Mandaji --- drivers/staging/comedi/drivers/cb_pcidas64.c | 6 +++--- 1 file changed, 3

Re: [PATCH] staging: comedi: cb_pcidas64: fix alignment of function parameters

2018-04-11 Thread Ian Abbott
On 11/04/18 06:07, Julia Lawall wrote: On Tue, 10 Apr 2018, Gabriel Francisco Mandaji wrote: [snip] diff --git a/drivers/staging/comedi/drivers/cb_pcidas64.c b/drivers/staging/comedi/drivers/cb_pcidas64.c index fdd81c3..631a703 100644 --- a/drivers/staging/comedi/drivers/cb_pcidas64.c +++

Re: [PATCH 1/2] staging: irda: Replace mdelay with usleep_range in stir421x_fw_upload

2018-04-11 Thread Jia-Ju Bai
On 2018/4/11 16:17, Greg KH wrote: On Wed, Apr 11, 2018 at 04:11:00PM +0800, Jia-Ju Bai wrote: On 2018/4/11 16:03, Greg KH wrote: On Wed, Apr 11, 2018 at 03:17:10PM +0800, Jia-Ju Bai wrote: On 2018/4/11 14:41, Greg KH wrote: On Wed, Apr 11, 2018 at 09:29:34AM +0800, Jia-Ju Bai wrote:

Re: [PATCH 1/2] staging: irda: Replace mdelay with usleep_range in stir421x_fw_upload

2018-04-11 Thread Greg KH
On Wed, Apr 11, 2018 at 04:11:00PM +0800, Jia-Ju Bai wrote: > > > On 2018/4/11 16:03, Greg KH wrote: > > On Wed, Apr 11, 2018 at 03:17:10PM +0800, Jia-Ju Bai wrote: > > > > > > On 2018/4/11 14:41, Greg KH wrote: > > > > On Wed, Apr 11, 2018 at 09:29:34AM +0800, Jia-Ju Bai wrote: > > > > >

Re: [PATCH 1/2] staging: irda: Replace mdelay with usleep_range in stir421x_fw_upload

2018-04-11 Thread Jia-Ju Bai
On 2018/4/11 16:03, Greg KH wrote: On Wed, Apr 11, 2018 at 03:17:10PM +0800, Jia-Ju Bai wrote: On 2018/4/11 14:41, Greg KH wrote: On Wed, Apr 11, 2018 at 09:29:34AM +0800, Jia-Ju Bai wrote: stir421x_fw_upload() is never called in atomic context. The call chain ending up at

Re: [PATCH] staging: Android: Add 'vsoc' driver for cuttlefish.

2018-04-11 Thread Dan Carpenter
On Tue, Apr 10, 2018 at 12:06:47PM -0700, Alistair Strachan wrote: > +static int do_create_fd_scoped_permission( > + struct vsoc_device_region *region_p, > + struct fd_scoped_permission_node *np, > + struct fd_scoped_permission_arg *__user arg) > +{ > + struct file *managed_filp; >

Re: [PATCH 1/2] staging: irda: Replace mdelay with usleep_range in stir421x_fw_upload

2018-04-11 Thread Greg KH
On Wed, Apr 11, 2018 at 03:17:10PM +0800, Jia-Ju Bai wrote: > > > On 2018/4/11 14:41, Greg KH wrote: > > On Wed, Apr 11, 2018 at 09:29:34AM +0800, Jia-Ju Bai wrote: > > > stir421x_fw_upload() is never called in atomic context. > > > > > > The call chain ending up at stir421x_fw_upload() is: > >

Re: [PATCH 1/2] staging: irda: Replace mdelay with usleep_range in stir421x_fw_upload

2018-04-11 Thread Jia-Ju Bai
On 2018/4/11 14:41, Greg KH wrote: On Wed, Apr 11, 2018 at 09:29:34AM +0800, Jia-Ju Bai wrote: stir421x_fw_upload() is never called in atomic context. The call chain ending up at stir421x_fw_upload() is: [1] stir421x_fw_upload() <- stir421x_patch_device() <- irda_usb_probe()

Re: [PATCH 1/2] staging: irda: Replace mdelay with usleep_range in stir421x_fw_upload

2018-04-11 Thread Greg KH
On Wed, Apr 11, 2018 at 09:29:34AM +0800, Jia-Ju Bai wrote: > stir421x_fw_upload() is never called in atomic context. > > The call chain ending up at stir421x_fw_upload() is: > [1] stir421x_fw_upload() <- stir421x_patch_device() <- irda_usb_probe() > > irda_usb_probe() is set as ".probe" in

Re: [PATCH 2/2] staging: irda: Replace mdelay with usleep_range in irda_usb_probe

2018-04-11 Thread Johan Hovold
On Wed, Apr 11, 2018 at 09:33:55AM +0800, Jia-Ju Bai wrote: > irda_usb_probe() is never called in atomic context. > > irda_usb_probe() is only set as ".probe" in struct usb_driver. > > Despite never getting called from atomic context, irda_usb_probe() > calls mdelay() to busily wait. > This is

Re: [PATCH 1/2] staging: irda: Replace mdelay with usleep_range in stir421x_fw_upload

2018-04-11 Thread Johan Hovold
On Wed, Apr 11, 2018 at 09:29:34AM +0800, Jia-Ju Bai wrote: > stir421x_fw_upload() is never called in atomic context. > > The call chain ending up at stir421x_fw_upload() is: > [1] stir421x_fw_upload() <- stir421x_patch_device() <- irda_usb_probe() > > irda_usb_probe() is set as ".probe" in

Re: [PATCH 1/2] uio_hv_generic: set size of ring buffer attribute

2018-04-11 Thread Frans Klaver
On Wed, Apr 11, 2018 at 2:04 AM, Stephen Hemminger wrote: > The development version had ring size as a module parameter, but > then it was made a fixed value. The code to set the size of > the ring buffer binary file was lost in the transistion. transition Cheers,