On Thursday 28 October 2021 16:24:08 Marc Zyngier wrote: > On Thu, 28 Oct 2021 12:37:24 +0100, > Pali Rohár <pali@xxxxxxxxxx> wrote: > > > > On Thursday 28 October 2021 12:30:30 Lorenzo Pieralisi wrote: > > > On Thu, Oct 28, 2021 at 01:13:02PM +0200, Pali Rohár wrote: > > > > > > [...] > > > > > > > > > In commit message I originally tried to explain it that after applying > > > > > > all previous patches which are fixing MSI and Multi-MSI support (part of > > > > > > them is enforcement to use only MSI numbers 0..31), it makes driver > > > > > > compatible with also MSI-X interrupts. > > > > > > > > > > > > If you want to rewrite commit message, let us know, there is no problem. > > > > > > > > > > I think we should. > > > > > > > > > > > > > Signed-off-by: Pali Rohár <pali@xxxxxxxxxx> > > > > > > > > Reviewed-by: Marek Behún <kabel@xxxxxxxxxx> > > > > > > > > > > By the way, this tag should be removed. Marek signed it off, that > > > > > applies to other patches in this series as well. > > > > > > > > Ok! Is this the only issue with this patch series? Or something other > > > > needs to be fixed? > > > > > > The series looks fine to me - only thing for patch[4-10] I'd like > > > to have evidence MarcZ is happy with the approach > > > > Marc, could you look at patches 4-10 if you are happy with them? Link: > > https://lore.kernel.org/linux-pci/20211012164145.14126-5-kabel@xxxxxxxxxx/ > > Started with patch #4, and saw that you are still using > irq_find_mapping + generic_handle_irq which I objected to every time I > looked at this patch ([1], [2]). > > My NAK still stands, and I haven't looked any further, because you > obviously don't really care about review comments. I passed to Marek all patches including handling and fixing these issues. But because Lorenzo wanted smaller patch series, Marek probably has not included them in this batch 2. > M. > > [1] https://lore.kernel.org/r/8735r0qfab.wl-maz@xxxxxxxxxx > [2] https://lore.kernel.org/r/871r6kqf2d.wl-maz@xxxxxxxxxx > > -- > Without deviation from the norm, progress is not possible.
