qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* Re: [EXT] Re: [PATCH] hw/cxl: Fix background completion percentage calculation
       [not found] ` <svy4snogtc2k5423b4vtmzaoaryetmxzatjfiqfeiywepkr7us@dcslpmazfuw5>
@ 2024-08-27 16:01   ` ajay.opensrc
  2024-08-27 16:38     ` Jonathan Cameron via
  0 siblings, 1 reply; 2+ messages in thread
From: ajay.opensrc @ 2024-08-27 16:01 UTC (permalink / raw)
  To: Jonathan.Cameron@huawei.com
  Cc: linux-cxl@vger.kernel.org, john@jagalactic.com, Eishan Mirakhur,
	Ajay Joshi, Srinivasulu Thanneeru, Ravis OpenSrc, Aravind Ramesh,
	Davidlohr Bueso, qemu-devel@nongnu.org

> From: Davidlohr Bueso <dave@stgolabs.net>
> 
> On Mon, 29 Jul 2024, ajay.opensrc@micron.com wrote:\n
> >From: Ajay Joshi <ajayjoshi@micron.com>
> >
> >The current completion percentage calculation does not account for the
> >relative time since the start of the background activity, this leads to
> >showing incorrect start percentage vs what has actually been completed.
> >
> >This patch calculates the percentage based on the actual elapsed time
> >since the start of the operation.
> >
> >Fixes: 221d2cfbdb ("hw/cxl/mbox: Add support for background
> >operations")
> >
> >Signed-off-by: Ajay Joshi <ajay.opensrc@micron.com>
> 
> Reviewed-by: Davidlohr Bueso <dave@stgolabs.net>

Ping.
Jonathan, this patch was reviewed by Dave already.
Do you have any comments on this patch?

Ajay

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [EXT] Re: [PATCH] hw/cxl: Fix background completion percentage calculation
  2024-08-27 16:01   ` [EXT] Re: [PATCH] hw/cxl: Fix background completion percentage calculation ajay.opensrc
@ 2024-08-27 16:38     ` Jonathan Cameron via
  0 siblings, 0 replies; 2+ messages in thread
From: Jonathan Cameron via @ 2024-08-27 16:38 UTC (permalink / raw)
  To: ajay.opensrc
  Cc: linux-cxl@vger.kernel.org, john@jagalactic.com, Eishan Mirakhur,
	Ajay Joshi, Srinivasulu Thanneeru, Ravis OpenSrc, Aravind Ramesh,
	Davidlohr Bueso, qemu-devel@nongnu.org, Michael S. Tsirkin

On Tue, 27 Aug 2024 16:01:54 +0000
ajay.opensrc <ajay.opensrc@micron.com> wrote:

> > From: Davidlohr Bueso <dave@stgolabs.net>
> > 
> > On Mon, 29 Jul 2024, ajay.opensrc@micron.com wrote:\n  
> > >From: Ajay Joshi <ajayjoshi@micron.com>
> > >
> > >The current completion percentage calculation does not account for the
> > >relative time since the start of the background activity, this leads to
> > >showing incorrect start percentage vs what has actually been completed.
> > >
> > >This patch calculates the percentage based on the actual elapsed time
> > >since the start of the operation.
> > >
> > >Fixes: 221d2cfbdb ("hw/cxl/mbox: Add support for background
> > >operations")
> > >
> > >Signed-off-by: Ajay Joshi <ajay.opensrc@micron.com>  
> > 
> > Reviewed-by: Davidlohr Bueso <dave@stgolabs.net>  
> 
> Ping.
> Jonathan, this patch was reviewed by Dave already.
> Do you have any comments on this patch?

Sorry bit behind. Seems fine to me.   I'll queue it up, but not sure
when I'll get a fixes series out (hopefully soon!)

+CC Michael in meantime.

Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
in case Michael wants to pick it up directly.

Thanks,

Jonathan

> 
> Ajay


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-08-27 17:03 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20240729102338.22337-1-ajay.opensrc@micron.com>
     [not found] ` <svy4snogtc2k5423b4vtmzaoaryetmxzatjfiqfeiywepkr7us@dcslpmazfuw5>
2024-08-27 16:01   ` [EXT] Re: [PATCH] hw/cxl: Fix background completion percentage calculation ajay.opensrc
2024-08-27 16:38     ` Jonathan Cameron via

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).