[PATCH vhost 0/2] vdpa/mlx5: Iova mapping related fixes

Dragos Tatulea posted 2 patches 1 month ago
drivers/vdpa/mlx5/core/mr.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
[PATCH vhost 0/2] vdpa/mlx5: Iova mapping related fixes
Posted by Dragos Tatulea 1 month ago
Here are 2 fixes from Si-Wei:
- The first one is an important fix that has to be applied as far
  back as possible (hence CC'ing linux-stable).
- The second is more of an improvement. That's why it doesn't have the
  Fixes tag.

I'd like to thank Si-Wei for the effort of finding and fixing these
issues. Especially the first issue which was very well hidden and
was there since day 1.

Si-Wei Liu (2):
  vdpa/mlx5: Fix PA offset with unaligned starting iotlb map
  vdpa/mlx5: Fix suboptimal range on iotlb iteration

 drivers/vdpa/mlx5/core/mr.c | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

-- 
2.46.1
Re: [PATCH vhost 0/2] vdpa/mlx5: Iova mapping related fixes
Posted by Dragos Tatulea 2 weeks ago

On 21.10.24 15:40, Dragos Tatulea wrote:
> Here are 2 fixes from Si-Wei:
> - The first one is an important fix that has to be applied as far
>   back as possible (hence CC'ing linux-stable).
> - The second is more of an improvement. That's why it doesn't have the
>   Fixes tag.
> 
> I'd like to thank Si-Wei for the effort of finding and fixing these
> issues. Especially the first issue which was very well hidden and
> was there since day 1.
> 
> Si-Wei Liu (2):
>   vdpa/mlx5: Fix PA offset with unaligned starting iotlb map
>   vdpa/mlx5: Fix suboptimal range on iotlb iteration
> 
>  drivers/vdpa/mlx5/core/mr.c | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
>
Gentle nudge for a review. The bug fixed by the first patch is a very
serious and insidious one.

Thanks,
Dragos
Re: [PATCH vhost 0/2] vdpa/mlx5: Iova mapping related fixes
Posted by Jason Wang 1 week, 6 days ago
On Mon, Nov 11, 2024 at 4:58 PM Dragos Tatulea <dtatulea@nvidia.com> wrote:
>
>
>
> On 21.10.24 15:40, Dragos Tatulea wrote:
> > Here are 2 fixes from Si-Wei:
> > - The first one is an important fix that has to be applied as far
> >   back as possible (hence CC'ing linux-stable).
> > - The second is more of an improvement. That's why it doesn't have the
> >   Fixes tag.
> >
> > I'd like to thank Si-Wei for the effort of finding and fixing these
> > issues. Especially the first issue which was very well hidden and
> > was there since day 1.
> >
> > Si-Wei Liu (2):
> >   vdpa/mlx5: Fix PA offset with unaligned starting iotlb map
> >   vdpa/mlx5: Fix suboptimal range on iotlb iteration
> >
> >  drivers/vdpa/mlx5/core/mr.c | 12 ++++++------
> >  1 file changed, 6 insertions(+), 6 deletions(-)
> >
> Gentle nudge for a review. The bug fixed by the first patch is a very
> serious and insidious one.

I think I've acked to those patches, have you received that?

Thanks

>
> Thanks,
> Dragos
>
>
Re: [PATCH vhost 0/2] vdpa/mlx5: Iova mapping related fixes
Posted by Michael S. Tsirkin 1 week, 5 days ago
On Wed, Nov 13, 2024 at 09:45:22AM +0800, Jason Wang wrote:
> On Mon, Nov 11, 2024 at 4:58 PM Dragos Tatulea <dtatulea@nvidia.com> wrote:
> >
> >
> >
> > On 21.10.24 15:40, Dragos Tatulea wrote:
> > > Here are 2 fixes from Si-Wei:
> > > - The first one is an important fix that has to be applied as far
> > >   back as possible (hence CC'ing linux-stable).
> > > - The second is more of an improvement. That's why it doesn't have the
> > >   Fixes tag.
> > >
> > > I'd like to thank Si-Wei for the effort of finding and fixing these
> > > issues. Especially the first issue which was very well hidden and
> > > was there since day 1.
> > >
> > > Si-Wei Liu (2):
> > >   vdpa/mlx5: Fix PA offset with unaligned starting iotlb map
> > >   vdpa/mlx5: Fix suboptimal range on iotlb iteration
> > >
> > >  drivers/vdpa/mlx5/core/mr.c | 12 ++++++------
> > >  1 file changed, 6 insertions(+), 6 deletions(-)
> > >
> > Gentle nudge for a review. The bug fixed by the first patch is a very
> > serious and insidious one.
> 
> I think I've acked to those patches, have you received that?
> 
> Thanks

I saw your acks Jason, thanks!
Patch 1 is now upstream. Patch 2 is queued but I asked a question about
it.


> >
> > Thanks,
> > Dragos
> >
> >