All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Daniel P. Berrangé" <berrange@redhat.com>
To: Christian Schoenebeck <qemu_oss@crudebyte.com>
Cc: qemu-devel@nongnu.org, Greg Kurz <groug@kaod.org>,
	Jia Jia <physicalmtea@gmail.com>,
	Igor Mammedov <imammedo@redhat.com>
Subject: Re: [PATCH 2/3] hw/9pfs/virtio: disable hotpluggable property of virtio-9p device
Date: Fri, 17 Jul 2026 15:16:14 +0100	[thread overview]
Message-ID: <alo5LiqOfOVx0Yxu@redhat.com> (raw)
In-Reply-To: <3767624.R56niFO833@weasel>

On Fri, Jul 17, 2026 at 04:05:42PM +0200, Christian Schoenebeck wrote:
> On Monday, 13 July 2026 09:49:35 CEST Igor Mammedov wrote:
> > On Fri, 10 Jul 2026 15:40:04 +0100
> > 
> > Daniel P. Berrangé <berrange@redhat.com> wrote:
> > > On Fri, Jul 10, 2026 at 04:31:51PM +0200, Christian Schoenebeck wrote:
> > > > On Friday, 10 July 2026 14:51:49 CEST Igor Mammedov wrote:
> > > > > On Fri, 10 Jul 2026 12:49:06 +0200
> > > > > 
> > > > > Christian Schoenebeck <qemu_oss@crudebyte.com> wrote:
> > > > > > On Friday, 10 July 2026 12:23:45 CEST Igor Mammedov wrote:
> > > > [...]
> > > > 
> > > > > note: I'm looking from pov of hotpluggable PCI device and generic
> > > > > hotplug
> > > > > infra, only.
> > > > 
> > > > That's okay, but so far I don't see the relevance for this particular 9p
> > > > device.
> > > > 
> > > > > it's not guest users directly, it's how hotplug flow works for various
> > > > > guest OSes:
> > > > > 
> > > > > 1. host plugs device in (-device or device_add)
> > > > > 2. guest OS get's notified one way or another and does what ever guest
> > > > > side
> > > > > 
> > > > >    init needed (incl. mounting share in 9pfs case)
> > > > 
> > > > That's not affected by this patch, right?
> > > > 
> > > > > opposite flow:
> > > > > 1. host does device_del (basically notify guest to remove device)
> > > > > 2. guest OS frees resources and tells qemu to delete device
> > > > > 3. qemu process remove event (which incl. unrealize as part of
> > > > > destroying
> > > > > device)
> > > > 
> > > > And that's not affected by this patch either, right?
> > 
> > the patch would break unplug flow by effectively removing 'eject' knob
> > from guest side, which is part of unplug flow.
> 
> These are two different things: device_del would not be affected by this. 
> 
> 1. Ejecting the device from host side e.g. via QMP would still work.
> 
> vs.
> 
> 2. Ejecting from guest side OTOH would be disabled.
> 
> It is also different from regular block devices where you have convenient ways
> to eject a block device on guest OSes. For a virtio-9p device it is not that
> easy.

Igor can correct me if I'm wrong, but IIUC there's no distinction
to those 2 scenarios from the guest POV.

A "device_del" in QMP does not actually delete a device from the
QEMU POV.  Rather it triggers an ACPI notification to the geust
OS, that the operator wants to unplug the device.  The guest OS
then honours (or not) that by initiating a guest side "eject".
IOW disabling guest initiated eject would break device_del too
AFAIK.

With regards,
Daniel
-- 
|: https://berrange.com       ~~        https://hachyderm.io/@berrange :|
|: https://libvirt.org          ~~          https://entangle-photo.org :|
|: https://pixelfed.art/berrange   ~~    https://fstop138.berrange.com :|



  reply	other threads:[~2026-07-17 14:16 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <cover.1783604079.git.qemu_oss@crudebyte.com>
2026-07-09 13:50 ` [PATCH 2/3] hw/9pfs/virtio: disable hotpluggable property of virtio-9p device Christian Schoenebeck
2026-07-10  7:37   ` Igor Mammedov
2026-07-10  8:06     ` Christian Schoenebeck
2026-07-10 10:23       ` Igor Mammedov
2026-07-10 10:49         ` Christian Schoenebeck
2026-07-10 12:51           ` Igor Mammedov
2026-07-10 14:31             ` Christian Schoenebeck
2026-07-10 14:40               ` Daniel P. Berrangé
2026-07-13  7:49                 ` Igor Mammedov
2026-07-17 14:05                   ` Christian Schoenebeck
2026-07-17 14:16                     ` Daniel P. Berrangé [this message]
2026-07-09 13:50 ` [PATCH 1/3] hw/9pfs/virtio: drain in-flight PDUs before virtio-9p unrealize Christian Schoenebeck
2026-07-09 14:20   ` Christian Schoenebeck
2026-07-10  1:39     ` m'te'a physical
2026-07-09 13:50 ` [PATCH 3/3] hw/9pfs/xen: drain in-flight PDUs before xen-9p disconnect Christian Schoenebeck
2026-07-16 12:32   ` Christian Schoenebeck

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=alo5LiqOfOVx0Yxu@redhat.com \
    --to=berrange@redhat.com \
    --cc=groug@kaod.org \
    --cc=imammedo@redhat.com \
    --cc=physicalmtea@gmail.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu_oss@crudebyte.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.