[PATCH] x86/hotplug: Remove incorrect comment about mwait_play_dead()

2023-01-27 Thread Srivatsa S. Bhat
From: "Srivatsa S. Bhat (VMware)" The comment that says mwait_play_dead() returns only on failure is a bit misleading because mwait_play_dead() could actually return for valid reasons (such as mwait not being supported by the platform) that do not indicate a failure of the CPU offline operation.

[PATCH v3 2/2] virtio_net: notify MAC address change on device initialization

2023-01-27 Thread Laurent Vivier
In virtnet_probe(), if the device doesn't provide a MAC address the driver assigns a random one. As we modify the MAC address we need to notify the device to allow it to update all the related information. The problem can be seen with vDPA and mlx5_vdpa driver as it doesn't assign a MAC address

[PATCH v3 0/2] virtio_net: vdpa: update MAC address when it is generated by virtio-net

2023-01-27 Thread Laurent Vivier
When the MAC address is not provided by the vdpa device virtio_net driver assigns a random one without notifying the device. The consequence, in the case of mlx5_vdpa, is the internal routing tables of the device are not updated and this can block the communication between two namespaces. To fix

[PATCH v3 1/2] virtio_net: disable VIRTIO_NET_F_STANDBY if VIRTIO_NET_F_MAC is not set

2023-01-27 Thread Laurent Vivier
failover relies on the MAC address to pair the primary and the standby devices: "[...] the hypervisor needs to enable VIRTIO_NET_F_STANDBY feature on the virtio-net interface and assign the same MAC address to both virtio-net and VF interfaces."

Re: [PATCH] drm/virtio: exbuf->fence_fd unmodified on interrupted wait

2023-01-27 Thread Rob Clark
On Fri, Jan 27, 2023 at 12:31 AM Ryan Neph wrote: > > An interrupted dma_fence_wait() becomes an -ERESTARTSYS returned > to userspace ioctl(DRM_IOCTL_VIRTGPU_EXECBUFFER) calls, prompting to > retry the ioctl(), but the passed exbuf->fence_fd has been reset to -1, > making the retry attempt fail

Re: [PATCH v2 1/1] virtio_net: notify MAC address change on device initialization

2023-01-27 Thread Michael S. Tsirkin
On Fri, Jan 27, 2023 at 01:28:01PM +0100, Laurent Vivier wrote: > On 1/27/23 12:08, Michael S. Tsirkin wrote: > > On Tue, Jan 24, 2023 at 12:04:24PM +0100, Laurent Vivier wrote: > > > On 1/24/23 11:15, Michael S. Tsirkin wrote: > > > > On Mon, Jan 23, 2023 at 01:00:22PM +0100, Laurent Vivier

Re: [PATCH v1 2/6] virtio console: Harden port adding

2023-01-27 Thread Michael S. Tsirkin
On Fri, Jan 27, 2023 at 04:17:46PM +0200, Alexander Shishkin wrote: > Greg Kroah-Hartman writes: > > > On Fri, Jan 27, 2023 at 02:47:55PM +0200, Alexander Shishkin wrote: > >> "Michael S. Tsirkin" writes: > >> > >> > On Fri, Jan 27, 2023 at 01:55:43PM +0200, Alexander Shishkin wrote: > >> >>

Re: [PATCH v1 2/6] virtio console: Harden port adding

2023-01-27 Thread Greg Kroah-Hartman
On Fri, Jan 27, 2023 at 04:17:46PM +0200, Alexander Shishkin wrote: > Greg Kroah-Hartman writes: > > > On Fri, Jan 27, 2023 at 02:47:55PM +0200, Alexander Shishkin wrote: > >> "Michael S. Tsirkin" writes: > >> > >> > On Fri, Jan 27, 2023 at 01:55:43PM +0200, Alexander Shishkin wrote: > >> >>

Re: [PATCH v1 2/6] virtio console: Harden port adding

2023-01-27 Thread Michael S. Tsirkin
On Fri, Jan 27, 2023 at 02:47:55PM +0200, Alexander Shishkin wrote: > "Michael S. Tsirkin" writes: > > > On Fri, Jan 27, 2023 at 01:55:43PM +0200, Alexander Shishkin wrote: > >> "Michael S. Tsirkin" writes: > >> > >> > On Thu, Jan 19, 2023 at 10:13:18PM +0200, Alexander Shishkin wrote: > >> >>

Re: [PATCH v1 2/6] virtio console: Harden port adding

2023-01-27 Thread Greg Kroah-Hartman
On Fri, Jan 27, 2023 at 02:47:55PM +0200, Alexander Shishkin wrote: > "Michael S. Tsirkin" writes: > > > On Fri, Jan 27, 2023 at 01:55:43PM +0200, Alexander Shishkin wrote: > >> "Michael S. Tsirkin" writes: > >> > >> > On Thu, Jan 19, 2023 at 10:13:18PM +0200, Alexander Shishkin wrote: > >> >>

Re: [PATCH v2 1/1] virtio_net: notify MAC address change on device initialization

2023-01-27 Thread Laurent Vivier
On 1/27/23 12:08, Michael S. Tsirkin wrote: On Tue, Jan 24, 2023 at 12:04:24PM +0100, Laurent Vivier wrote: On 1/24/23 11:15, Michael S. Tsirkin wrote: On Mon, Jan 23, 2023 at 01:00:22PM +0100, Laurent Vivier wrote: In virtnet_probe(), if the device doesn't provide a MAC address the driver

Re: [PATCH v1 2/6] virtio console: Harden port adding

2023-01-27 Thread Michael S. Tsirkin
On Fri, Jan 27, 2023 at 01:55:43PM +0200, Alexander Shishkin wrote: > "Michael S. Tsirkin" writes: > > > On Thu, Jan 19, 2023 at 10:13:18PM +0200, Alexander Shishkin wrote: > >> When handling control messages, instead of peeking at the device memory > >> to obtain bits of the control structure,

Re: [PATCH 0/2] vhost: improve livepatch switching for heavily loaded vhost worker kthreads

2023-01-27 Thread Petr Mladek via Virtualization
On Fri 2023-01-27 11:37:02, Peter Zijlstra wrote: > On Thu, Jan 26, 2023 at 08:43:55PM -0800, Josh Poimboeuf wrote: > > On Thu, Jan 26, 2023 at 03:12:35PM -0600, Seth Forshee (DigitalOcean) wrote: > > > On Thu, Jan 26, 2023 at 06:03:16PM +0100, Petr Mladek wrote: > > > > On Fri 2023-01-20

Re: [PATCH 0/2] vhost: improve livepatch switching for heavily loaded vhost worker kthreads

2023-01-27 Thread Petr Mladek via Virtualization
On Thu 2023-01-26 15:12:35, Seth Forshee (DigitalOcean) wrote: > On Thu, Jan 26, 2023 at 06:03:16PM +0100, Petr Mladek wrote: > > On Fri 2023-01-20 16:12:20, Seth Forshee (DigitalOcean) wrote: > > > We've fairly regularaly seen liveptches which cannot transition within > > > kpatch's > > >

Re: [PATCH 0/4] Fix probe failed when modprobe modules

2023-01-27 Thread Michael S. Tsirkin
On Mon, Nov 28, 2022 at 05:14:44AM -0500, Michael S. Tsirkin wrote: > On Mon, Nov 28, 2022 at 10:10:01AM +0800, Li Zetao wrote: > > This patchset fixes similar issue, the root cause of the > > problem is that the virtqueues are not stopped on error > > handling path. > > I've been thinking about

Re: [PATCH v2 1/1] virtio_net: notify MAC address change on device initialization

2023-01-27 Thread Michael S. Tsirkin
On Tue, Jan 24, 2023 at 12:04:24PM +0100, Laurent Vivier wrote: > On 1/24/23 11:15, Michael S. Tsirkin wrote: > > On Mon, Jan 23, 2023 at 01:00:22PM +0100, Laurent Vivier wrote: > > > In virtnet_probe(), if the device doesn't provide a MAC address the > > > driver assigns a random one. > > > As we

Re: [PATCH v1 2/6] virtio console: Harden port adding

2023-01-27 Thread Michael S. Tsirkin
On Thu, Jan 19, 2023 at 10:13:18PM +0200, Alexander Shishkin wrote: > Greg Kroah-Hartman writes: > > > Then you need to copy it out once, and then only deal with the local > > copy. Otherwise you have an incomplete snapshot. > > Ok, would you be partial to something like this: > > >From

Re: [PATCH v1 4/6] virtio console: Harden control message handling

2023-01-27 Thread Michael S. Tsirkin
On Fri, Jan 20, 2023 at 06:41:27PM +0200, Alexander Shishkin wrote: > "Michael S. Tsirkin" writes: > > > On Thu, Jan 19, 2023 at 04:22:09PM +0100, Greg Kroah-Hartman wrote: > >> On Thu, Jan 19, 2023 at 03:57:19PM +0200, Alexander Shishkin wrote: > >> > In handle_control_message(), we look at the

Re: [PATCH 0/2] Fix expected set_vq_state behavior on vdpa_sim

2023-01-27 Thread Michael S. Tsirkin
On Wed, Jan 18, 2023 at 05:43:57PM +0100, Eugenio PĂ©rez wrote: > The use of set_vq_state is to indicate vdpa device the state of a virtqueue. > In the case of split, it means the avail_idx. This is mandatory for use > cases like live migration. > > However, vdpa_sim reset the vq state at

Re: [PATCH -next] vdpa/mlx5: fix check wrong pointer in mlx5_vdpa_add_mac_vlan_rules()

2023-01-27 Thread Michael S. Tsirkin
Were you going to do this? On Wed, Jan 04, 2023 at 02:31:52PM +, Eli Cohen wrote: > Sure, thanks! > > > -Original Message- > > From: Michael S. Tsirkin > > Sent: Wednesday, 4 January 2023 15:58 > > To: Eli Cohen > > Cc: Yang Yingliang ; virtualization@lists.linux- > >

Re: [PATCH] vdpa: ifcvf: Do proper cleanup if IFCVF init fails

2023-01-27 Thread Michael S. Tsirkin
On Tue, Dec 27, 2022 at 10:15:25PM +0100, Tanmay Bhushan wrote: > >From 7eae04667ddaac8baa4812d48ef2c942cedef946 Mon Sep 17 00:00:00 2001 > From: Tanmay Bhushan <0070472...@gmail.com> > Date: Tue, 27 Dec 2022 22:02:16 +0100 > Subject: [PATCH] vdpa: ifcvf: Do proper cleanup if IFCVF init fails > >

Re: [PATCH 0/2] vhost: improve livepatch switching for heavily loaded vhost worker kthreads

2023-01-27 Thread Peter Zijlstra
On Thu, Jan 26, 2023 at 08:43:55PM -0800, Josh Poimboeuf wrote: > On Thu, Jan 26, 2023 at 03:12:35PM -0600, Seth Forshee (DigitalOcean) wrote: > > On Thu, Jan 26, 2023 at 06:03:16PM +0100, Petr Mladek wrote: > > > On Fri 2023-01-20 16:12:20, Seth Forshee (DigitalOcean) wrote: > > > > We've fairly

Re: [PATCH 3/4] virtio_ring: introduce a per virtqueue waitqueue

2023-01-27 Thread Michael S. Tsirkin
On Fri, Dec 30, 2022 at 11:43:08AM +0800, Jason Wang wrote: > On Thu, Dec 29, 2022 at 4:10 PM Michael S. Tsirkin wrote: > > > > On Thu, Dec 29, 2022 at 04:04:13PM +0800, Jason Wang wrote: > > > On Thu, Dec 29, 2022 at 3:07 PM Michael S. Tsirkin > > > wrote: > > > > > > > > On Wed, Dec 28, 2022

Re: [PATCH] vdpa_sim: get rid of DMA ops

2023-01-27 Thread Michael S. Tsirkin
On Mon, Dec 26, 2022 at 12:12:42PM +0800, Jason Wang wrote: > > >@@ -682,6 +553,11 @@ static int vdpasim_dma_unmap(struct vdpa_device > > >*vdpa, unsigned int asid, > > > if (asid >= vdpasim->dev_attr.nas) > > > return -EINVAL; > > > > > >+ if (vdpasim->iommu_pt[asid]) {

Re: [PATCH v2 01/11] genirq/affinity:: Export irq_create_affinity_masks()

2023-01-27 Thread Michael S. Tsirkin
On Mon, Dec 19, 2022 at 05:36:02PM +0800, Yongji Xie wrote: > On Mon, Dec 19, 2022 at 3:33 PM Michael S. Tsirkin wrote: > > > > On Mon, Dec 05, 2022 at 04:41:17PM +0800, Xie Yongji wrote: > > > Export irq_create_affinity_masks() so that some modules > > > can make use of it to implement interrupt

Re: [PATCH] vhost_net: revert upend_idx only on retriable error

2023-01-27 Thread Michael S. Tsirkin
On Mon, Dec 19, 2022 at 11:24:26AM +0300, Andrey Smetanin wrote: > Sorry for the delay. > I will send update on this week after some tests. > 19.12.2022, 10:39, "Michael S. Tsirkin" : Do you still plan to send something? Dropping this for now. ___

Re: [PATCH v3 0/4] vDPA: initial config export via "vdpa dev show"

2023-01-27 Thread Michael S. Tsirkin
Did you say you are going to post v4 of this? I'm dropping this from review for now. -- MST ___ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization

Re: [PATCH v10 00/11] Add generic memory shrinker to VirtIO-GPU and Panfrost DRM drivers

2023-01-27 Thread Gerd Hoffmann
On Thu, Jan 26, 2023 at 01:55:09AM +0300, Dmitry Osipenko wrote: > Hello Thomas and Gerd, > > On 1/9/23 00:04, Dmitry Osipenko wrote: > > This series: > > > > 1. Makes minor fixes for drm_gem_lru and Panfrost > > 2. Brings refactoring for older code > > 3. Adds common drm-shmem memory

Re: [PATCH v10 06/11] drm/shmem-helper: Don't use vmap_use_count for dma-bufs

2023-01-27 Thread Gerd Hoffmann
On Thu, Jan 26, 2023 at 03:24:30PM +0300, Dmitry Osipenko wrote: > On 1/26/23 15:17, Gerd Hoffmann wrote: > > On Mon, Jan 09, 2023 at 12:04:40AM +0300, Dmitry Osipenko wrote: > >> its own refcounting of vmaps, use it instead of drm-shmem > >> counting. This change prepares drm-shmem for addition

Re: [PATCH v10 10/11] drm/virtio: Support memory shrinking

2023-01-27 Thread Gerd Hoffmann
On Mon, Jan 09, 2023 at 12:04:44AM +0300, Dmitry Osipenko wrote: > Support generic drm-shmem memory shrinker and add new madvise IOCTL to > the VirtIO-GPU driver. BO cache manager of Mesa driver will mark BOs as > "don't need" using the new IOCTL to let shrinker purge the marked BOs on > OOM, the