Re: [PATCH] [SCSI] advansys: fix host resource leak in EISA probe error path

[Hannes Reinecke]

On 4/17/26 08:29, Guangshuo Li wrote:
> Hi Hannes,
>
> Thanks for the feedback.
>
> On Fri, 17 Apr 2026 at 13:56, Hannes Reinecke <hare@xxxxxxxx> wrote:
> > You must be kidding ... EISA is died over a decade ago.
> >
> > If you _really_ are concerned about this please remove EISA support
> > completely from the driver.
>
> I agree that EISA is obsolete, and I understand that this path is
> unlikely to matter on modern systems. My intent was simply to clean up
> an inconsistency I noticed while reviewing the existing error handling
> code.
>
> If maintaining the EISA path is not worthwhile, I’m fine with dropping
> this patch. I can also take a look at what removing the EISA support
> would involve.
Please, drop the patch, and rather invest time to check how to drop
EISA support. Fixing issues for code paths which are never exercised
is a bit pointless.

Cheers,

Hannes
--
Dr. Hannes Reinecke                  Kernel Storage Architect
hare@xxxxxxxx                               +49 911 74053 688
SUSE Software Solutions GmbH, Frankenstr. 146, 90461 Nürnberg
HRB 36809 (AG Nürnberg), GF: I. Totev, A. McDonald, W. Knoblich