Re: [PATCH] PCI: hv: Only reuse existing IRTE allocation for Multi-MSI

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Aug 10, 2022 at 09:51:23PM +0000, Dexuan Cui wrote:
> > From: Jeffrey Hugo <quic_jhugo@xxxxxxxxxxx>
> > Sent: Thursday, August 4, 2022 7:22 AM
> >  ...
> > > Fixes: b4b77778ecc5 ("PCI: hv: Reuse existing IRTE allocation in
> > compose_msi_msg()")
> > > Signed-off-by: Dexuan Cui <decui@xxxxxxxxxxxxx>
> > > Cc: Jeffrey Hugo <quic_jhugo@xxxxxxxxxxx>
> > > Cc: Carl Vanderlip <quic_carlv@xxxxxxxxxxx>
> > > ---
> > 
> > I'm sorry a regression has been discovered.  Right now, the issue
> > doesn't make sense to me.  I'd love to know what you find out.
> > 
> > This stopgap solution appears reasonable to me.
> > 
> > Reviewed-by: Jeffrey Hugo <quic_jhugo@xxxxxxxxxxx>
> 
> Hi Lorenzo, Bjorn and all,
> Would you please take a look at the patch?

I am not very happy with this patch, it is a temporary solution to
a potential problem (or reworded: we don't know what the problem is
but we are fixing it anyway - in a way that is potentially not
related to the bug at all).

If the commit you are fixing is the problem I'd rather revert it,
waiting to understand the problem and to rework the code accordingly.

I don't think b4b77778ecc5 is essential to Hyper-V code - it is a
rework, let's fix it and repost it when it is updated through the
debugging you are carrying out. In the meantime we can revert it
to fix the issue.

Thanks,
Lorenzo



[Index of Archives]     [Linux Samsung SoC]     [Linux Rockchip SoC]     [Linux Actions SoC]     [Linux for Synopsys ARC Processors]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]


  Powered by Linux