[Virtio-fs] Kernel patches on latest linux kernel
Vivek Goyal
vgoyal at redhat.com
Fri Jul 10 20:49:52 UTC 2020
Hi Sebastien,
I pushed new patches with the fix here.
https://gitlab.com/virtio-fs/linux/-/commits/vivek-july-10-2020
Give it a try.
Thanks
Vivek
On Fri, Jul 10, 2020 at 04:38:51PM -0400, Vivek Goyal wrote:
> Hi Sebastien,
>
> I can't seem to remove the device. I get error saying pci bus does not
> support hot removal. But I tried "rmmod virtiofs" and that ran into
> similar issue. I think following patch should fix it. I will merge
> it one of the existing patches.
>
> Thanks
> Vivek
>
>
> ---
> fs/fuse/virtio_fs.c | 9 +++++----
> 1 file changed, 5 insertions(+), 4 deletions(-)
>
> Index: redhat-linux/fs/fuse/virtio_fs.c
> ===================================================================
> --- redhat-linux.orig/fs/fuse/virtio_fs.c 2020-07-10 20:25:25.679624830 +0000
> +++ redhat-linux/fs/fuse/virtio_fs.c 2020-07-10 20:28:02.169624830 +0000
> @@ -794,10 +794,10 @@ static const struct dax_operations virti
>
> static void virtio_fs_cleanup_dax(void *data)
> {
> - struct virtio_fs *fs = data;
> + struct dax_device *dax_dev = data;
>
> - kill_dax(fs->dax_dev);
> - put_dax(fs->dax_dev);
> + kill_dax(dax_dev);
> + put_dax(dax_dev);
> }
>
> static int virtio_fs_setup_dax(struct virtio_device *vdev, struct virtio_fs *fs)
> @@ -852,7 +852,8 @@ static int virtio_fs_setup_dax(struct vi
> if (!fs->dax_dev)
> return -ENOMEM;
>
> - return devm_add_action_or_reset(&vdev->dev, virtio_fs_cleanup_dax, fs);
> + return devm_add_action_or_reset(&vdev->dev, virtio_fs_cleanup_dax,
> + fs->dax_dev);
> }
>
> static int virtio_fs_probe(struct virtio_device *vdev)
>
>
>
>
> On Fri, Jul 10, 2020 at 09:51:23AM +0000, Boeuf, Sebastien wrote:
> > Huuh it seems I'm running into some issue wrt PCI hotplug.
> > If I remove my virtio-fs-pci device from my VM, I get the
> > following error from the guest kernel:
> >
> >
> > [ 135.347517] general protection fault, probably for non-canonical address 0xa37366424f344963: 0000 [#1] SMP PTI
> > [ 135.353330] CPU: 0 PID: 21 Comm: kworker/u2:2 Not tainted 5.8.0-rc4+ #317
> > [ 135.357176] Hardware name: Cloud Hypervisor cloud-hypervisor, BIOS 0
> > [ 135.360724] Workqueue: kacpi_hotplug acpi_hotplug_work_fn
> > [ 135.363578] RIP: 0010:kill_dax+0xd/0x70
> > [ 135.365642] Code: d0 02 00 00 04 c3 0f 1f 80 00 00 00 00 48 8b 87 d0 02 00 00 83 e0 01 c3 0f 1f 44 00 00 48 85 ff 74 60 55 48 89 e5 53 48 89 fb <3e> 80 a7 d0 02 00 00 fe 48 c7 c7 80 b2 07 ae e8 df 69 bd ff 48 c7
> > [ 135.375052] RSP: 0018:ffffb464000b7b10 EFLAGS: 00010286
> > [ 135.377791] RAX: ffffffffad3cdcf0 RBX: a37366424f344963 RCX: 0000000000000282
> > [ 135.380378] RDX: ffff8e6c7976ec40 RSI: ffff8e6c7976ea98 RDI: a37366424f344963
> > [ 135.382880] RBP: ffffb464000b7b18 R08: ffffb464000b7b78 R09: ffffb464000b7b78
> > [ 135.384876] R10: ffff8e6c7c1e9e00 R11: 0000000000000001 R12: ffffb464000b7b78
> > [ 135.386864] R13: ffff8e6c7976ea80 R14: ffff8e6c7976ea80 R15: ffff8e6c79744ab0
> > [ 135.388623] FS: 0000000000000000(0000) GS:ffff8e6c7ec00000(0000) knlGS:0000000000000000
> > [ 135.390417] CS: 0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> > [ 135.391741] CR2: 00007fcf64d08db0 CR3: 000000003c112001 CR4: 00000000003606b0
> > [ 135.393248] Call Trace:
> > [ 135.393808] virtio_fs_cleanup_dax+0x11/0x20
> > [ 135.394666] devm_action_release+0x10/0x20
> > [ 135.395468] release_nodes+0x22c/0x270
> > [ 135.396216] devres_release_all+0x3a/0x50
> > [ 135.397016] device_release_driver_internal+0xfb/0x1c0
> > [ 135.397974] device_release_driver+0xd/0x10
> > [ 135.398681] bus_remove_device+0xdc/0x150
> > [ 135.399369] device_del+0x16f/0x3f0
> > [ 135.399981] device_unregister+0x16/0x60
> > [ 135.400662] unregister_virtio_device+0x13/0x30
> > [ 135.401446] virtio_pci_remove+0x28/0x60
> > [ 135.402119] pci_device_remove+0x39/0xa0
> > [ 135.402778] device_release_driver_internal+0xeb/0x1c0
> > [ 135.403535] device_release_driver+0xd/0x10
> > [ 135.404182] pci_stop_bus_device+0x6b/0x90
> > [ 135.404803] pci_stop_and_remove_bus_device+0xe/0x20
> > [ 135.405573] disable_slot+0x4a/0x90
> > [ 135.406123] acpiphp_disable_and_eject_slot+0x15/0x80
> > [ 135.406877] acpiphp_hotplug_notify+0x184/0x280
> > [ 135.407581] ? free_bridge+0x130/0x130
> > [ 135.408165] acpi_device_hotplug+0xdd/0x390
> > [ 135.408734] acpi_hotplug_work_fn+0x1d/0x30
> > [ 135.409313] process_one_work+0x192/0x2e0
> > [ 135.409856] worker_thread+0x4b/0x3b0
> > [ 135.410359] kthread+0x10a/0x140
> > [ 135.410823] ? process_one_work+0x2e0/0x2e0
> > [ 135.411390] ? __kthread_parkme+0x60/0x60
> > [ 135.411962] ret_from_fork+0x22/0x30
> > [ 135.412501] ---[ end trace bbfed4f67497ceb8 ]---
> >
> > Any chance you already ran into this problem?
> >
> > Thanks,
> > Sebastien
> > ________________________________
> > From: virtio-fs-bounces at redhat.com <virtio-fs-bounces at redhat.com> on behalf of Boeuf, Sebastien <sebastien.boeuf at intel.com>
> > Sent: Friday, July 10, 2020 8:47 AM
> > To: Vivek Goyal <vgoyal at redhat.com>
> > Cc: virtio-fs at redhat.com <virtio-fs at redhat.com>
> > Subject: Re: [Virtio-fs] Kernel patches on latest linux kernel
> >
> > Oh cool! Thanks Vivek, I'll keep you posted if I run into any issue.
> >
> > Sebastien
> > ________________________________
> > From: Vivek Goyal <vgoyal at redhat.com>
> > Sent: Thursday, July 9, 2020 9:44 PM
> > To: Boeuf, Sebastien <sebastien.boeuf at intel.com>
> > Cc: virtio-fs at redhat.com <virtio-fs at redhat.com>
> > Subject: Re: Kernel patches on latest linux kernel
> >
> > Hi Sebastien,
> >
> > I rebased virtiofs dax patches on top of 5.8-rc4. Give it a try.
> >
> > https://gitlab.com/virtio-fs/linux/-/commits/vivek-july-09-2020
> >
> > Thanks
> > Vivek
> >
> > On Wed, Jun 17, 2020 at 01:58:54PM +0000, Boeuf, Sebastien wrote:
> > > Sounds good, thank you Vivek for the heads up 🙂
> > >
> > > Sebastien
> > > ________________________________
> > > From: Vivek Goyal <vgoyal at redhat.com>
> > > Sent: Wednesday, June 17, 2020 3:52 PM
> > > To: Boeuf, Sebastien <sebastien.boeuf at intel.com>
> > > Cc: virtio-fs at redhat.com <virtio-fs at redhat.com>
> > > Subject: Re: Kernel patches on latest linux kernel
> > >
> > > On Fri, Jun 12, 2020 at 09:00:50AM +0000, Boeuf, Sebastien wrote:
> > > > Hi folks,
> > > >
> > > > Just wondering when the kernel patch series will be rebased on top of latest kernel?
> > >
> > > Hi Sebastien,
> > >
> > > I will try to take care of this week or next week and rebase the patches
> > > on top of 5.8-rc1
> > >
> > > Thanks
> > > Vivek
> > >
> > > ---------------------------------------------------------------------
> > > Intel Corporation SAS (French simplified joint stock company)
> > > Registered headquarters: "Les Montalets"- 2, rue de Paris,
> > > 92196 Meudon Cedex, France
> > > Registration Number: 302 456 199 R.C.S. NANTERRE
> > > Capital: 4,572,000 Euros
> > >
> > > This e-mail and any attachments may contain confidential material for
> > > the sole use of the intended recipient(s). Any review or distribution
> > > by others is strictly prohibited. If you are not the intended
> > > recipient, please contact the sender and delete all copies.
> >
> >
> > ---------------------------------------------------------------------
> > Intel Corporation SAS (French simplified joint stock company)
> > Registered headquarters: "Les Montalets"- 2, rue de Paris,
> > 92196 Meudon Cedex, France
> > Registration Number: 302 456 199 R.C.S. NANTERRE
> > Capital: 4,572,000 Euros
> >
> > This e-mail and any attachments may contain confidential material for
> > the sole use of the intended recipient(s). Any review or distribution
> > by others is strictly prohibited. If you are not the intended
> > recipient, please contact the sender and delete all copies.
> > ---------------------------------------------------------------------
> > Intel Corporation SAS (French simplified joint stock company)
> > Registered headquarters: "Les Montalets"- 2, rue de Paris,
> > 92196 Meudon Cedex, France
> > Registration Number: 302 456 199 R.C.S. NANTERRE
> > Capital: 4,572,000 Euros
> >
> > This e-mail and any attachments may contain confidential material for
> > the sole use of the intended recipient(s). Any review or distribution
> > by others is strictly prohibited. If you are not the intended
> > recipient, please contact the sender and delete all copies.
More information about the Virtio-fs
mailing list