[PATCH v2 0/2] misc: vcpu_stall_detector: Add a PPI interrupt

Sebastian Ene posted 2 patches 1 year, 8 months ago
There is a newer version of this series
.../misc/qemu,vcpu-stall-detector.yaml        |  6 ++++
drivers/misc/vcpu_stall_detector.c            | 31 +++++++++++++++++--
2 files changed, 35 insertions(+), 2 deletions(-)
[PATCH v2 0/2] misc: vcpu_stall_detector: Add a PPI interrupt
Posted by Sebastian Ene 1 year, 8 months ago
Hello,

I respined the v2 version to address an issue previously found here:
https://lore.kernel.org/all/202406132132.FBKSWFav-lkp@intel.com/
 
This is a small update of the previously introduced vcpu stall detector
which adds an interrupt to the virtual device to notify the guest VM in
case it stalls. This lets the guest VM to handle the reboot and to
panic in case it expires. 

Changelog from v1:

* 1/2 : collected the Ack from Conor Dooley, thank you Conor !

* 2/2 : applied the feedback received from Conor and used
	platform_get_irq_optional. Removed the error messages during
	probe
   

Thanks, 

Sebastian Ene (2):
  dt-bindings: vcpu_stall_detector: Add a PPI interrupt to the virtual
    device
  misc: Register a PPI for the vcpu stall detection virtual device

 .../misc/qemu,vcpu-stall-detector.yaml        |  6 ++++
 drivers/misc/vcpu_stall_detector.c            | 31 +++++++++++++++++--
 2 files changed, 35 insertions(+), 2 deletions(-)

-- 
2.45.2.505.gda0bf45e8d-goog
Re: [PATCH v2 0/2] misc: vcpu_stall_detector: Add a PPI interrupt
Posted by Marc Zyngier 1 year, 8 months ago
On Thu, 13 Jun 2024 15:13:33 +0100,
Sebastian Ene <sebastianene@google.com> wrote:
> 
> Hello,
> 
> I respined the v2 version to address an issue previously found here:
> https://lore.kernel.org/all/202406132132.FBKSWFav-lkp@intel.com/

So is this v2 or v3? Having two v2s on the list is... confusing.

	M.

-- 
Without deviation from the norm, progress is not possible.
Re: [PATCH v2 0/2] misc: vcpu_stall_detector: Add a PPI interrupt
Posted by Sebastian Ene 1 year, 8 months ago
On Thu, Jun 13, 2024 at 03:52:08PM +0100, Marc Zyngier wrote:
> On Thu, 13 Jun 2024 15:13:33 +0100,
> Sebastian Ene <sebastianene@google.com> wrote:
> > 
> > Hello,
> > 
> > I respined the v2 version to address an issue previously found here:
> > https://lore.kernel.org/all/202406132132.FBKSWFav-lkp@intel.com/
> 
> So is this v2 or v3? Having two v2s on the list is... confusing.
> 
> 	M.
> 

There is a small change in the patch 2/2 so you are right it should be v3,
sorry for the confusion.

> -- 
> Without deviation from the norm, progress is not possible.

Thanks,
Seb
Re: [PATCH v2 0/2] misc: vcpu_stall_detector: Add a PPI interrupt
Posted by Greg KH 1 year, 7 months ago
On Thu, Jun 13, 2024 at 04:44:23PM +0000, Sebastian Ene wrote:
> On Thu, Jun 13, 2024 at 03:52:08PM +0100, Marc Zyngier wrote:
> > On Thu, 13 Jun 2024 15:13:33 +0100,
> > Sebastian Ene <sebastianene@google.com> wrote:
> > > 
> > > Hello,
> > > 
> > > I respined the v2 version to address an issue previously found here:
> > > https://lore.kernel.org/all/202406132132.FBKSWFav-lkp@intel.com/
> > 
> > So is this v2 or v3? Having two v2s on the list is... confusing.
> > 
> > 	M.
> > 
> 
> There is a small change in the patch 2/2 so you are right it should be v3,
> sorry for the confusion.

Please send a new version, nothing we can do with duplicate series on
the lists...
Re: [PATCH v2 0/2] misc: vcpu_stall_detector: Add a PPI interrupt
Posted by Sebastian Ene 1 year, 7 months ago
On Wed, Jul 03, 2024 at 04:34:08PM +0200, Greg KH wrote:
> On Thu, Jun 13, 2024 at 04:44:23PM +0000, Sebastian Ene wrote:
> > On Thu, Jun 13, 2024 at 03:52:08PM +0100, Marc Zyngier wrote:
> > > On Thu, 13 Jun 2024 15:13:33 +0100,
> > > Sebastian Ene <sebastianene@google.com> wrote:
> > > > 
> > > > Hello,
> > > > 
> > > > I respined the v2 version to address an issue previously found here:
> > > > https://lore.kernel.org/all/202406132132.FBKSWFav-lkp@intel.com/
> > > 
> > > So is this v2 or v3? Having two v2s on the list is... confusing.
> > > 
> > > 	M.
> > > 
> > 
> > There is a small change in the patch 2/2 so you are right it should be v3,
> > sorry for the confusion.

Hello,

> 
> Please send a new version, nothing we can do with duplicate series on
> the lists...


Thanks for letting me know, I will spin a new version.

Seb