All of lore.kernel.org
 help / color / mirror / Atom feed
From: sashiko-bot@kernel.org
To: "Jose Ignacio Tornos Martinez" <jtornosm@redhat.com>
Cc: linux-pci@vger.kernel.org
Subject: Re: [PATCH v14 2/2] PCI: Add device-specific reset for Qualcomm WCN6855/WCN7850 WLAN
Date: Thu, 17 Sep 2026 07:26:52 +0000	[thread overview]
Message-ID: <20260917072653.26B081F000FF@smtp.kernel.org> (raw)
In-Reply-To: <20260917071651.14174-3-jtornosm@redhat.com>

> Qualcomm WCN6855 (17cb:1103) and WCN7850 (17cb:1107) WLAN devices lack
> working reset methods for VFIO passthrough scenarios. These devices have no
> FLR capability, advertise NoSoftRst+ (blocking PM reset), and have broken
> bus reset (addressed by quirk_no_bus_reset, merged for v7.2).
> 
> VFIO attempts to reset devices on every reassignment. Without a working
> reset method, the attempt fails. On clean VM shutdown, the guest driver
> properly deinitializes the device via .shutdown/.remove callbacks, leaving
> it in a usable state despite the failed reset. However, on unclean VM
> termination (crash, force-off), the guest driver callbacks are not
> triggered, the device remains in an undefined state (DMA active, interrupts
> enabled, etc.), and without a working reset it cannot be reused.
> 
> Add a device-specific reset method using BAR-space hardware reset registers
> that exist in these devices.
> [ ... ]
> to previous approaches that failed after ~30 cycles. Device-specific reset
> is position #1 in the reset hierarchy, so these Qualcomm devices will use
> hardware reset as their primary reset method.
> 
> Signed-off-by: Jose Ignacio Tornos Martinez <jtornosm@redhat.com>

Sashiko has reviewed this patch and found no issues. It looks great!

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260917071651.14174-1-jtornosm@redhat.com?part=2


  reply	other threads:[~2026-09-17  7:26 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-17  7:16 [PATCH v14 0/2] PCI: Add device-specific reset for Qualcomm devices Jose Ignacio Tornos Martinez
2026-09-17  7:16 ` [PATCH v14 1/2] PCI: Add device-specific reset for Qualcomm SDX62/SDX65 modems Jose Ignacio Tornos Martinez
2026-09-17  7:35   ` sashiko-bot
2026-09-17 12:45     ` Jose Ignacio Tornos Martinez
2026-09-17  7:16 ` [PATCH v14 2/2] PCI: Add device-specific reset for Qualcomm WCN6855/WCN7850 WLAN Jose Ignacio Tornos Martinez
2026-09-17  7:26   ` sashiko-bot [this message]
2026-10-02 21:40 ` [PATCH v14 0/2] PCI: Add device-specific reset for Qualcomm devices Bjorn Helgaas
2026-10-02 23:09   ` Alex Williamson
2026-10-02 23:47     ` Bjorn Helgaas
2026-10-05 10:48       ` Jose Ignacio Tornos Martinez

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=20260917072653.26B081F000FF@smtp.kernel.org \
    --to=sashiko-bot@kernel.org \
    --cc=jtornosm@redhat.com \
    --cc=linux-pci@vger.kernel.org \
    --cc=sashiko-reviews@lists.linux.dev \
    /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.