Re: [PATCH v3 1/2] PCI/aer_inject: Convert inject_lock to raw_spinlock_t

From: Guangbo Cui
Date: Mon Oct 27 2025 - 11:19:02 EST


On Mon, Oct 27, 2025 at 09:11:25AM +0100, Sebastian Andrzej Siewior wrote:
> > @@ -445,7 +445,7 @@ static int aer_inject(struct aer_error_inj *einj)
> > rperr->source_id &= 0x0000ffff;
> > rperr->source_id |= PCI_DEVID(einj->bus, devfn) << 16;
> > }
> > - spin_unlock_irqrestore(&inject_lock, flags);
> > + raw_spin_unlock_irqrestore(&inject_lock, flags);
> >
> > if (aer_mask_override) {
> > pci_write_config_dword(dev, pos_cap_err + PCI_ERR_COR_MASK,
>
> This is the last hunk. You miss the module exit part. This won't
> compile on its own, as such it can't be applied. It might be chosen for
> a backport. A change must always be self-contained.

I haven’t yet developed the good habit of keeping each patch fully
self-contained. I will fix this in the next version.

Best regards,
Guangbo