Re: linux-next: manual merge of the scsi-mkp tree with the scsi tree

From: Geert Uytterhoeven

Date: Wed Sep 30 2026 - 03:23:28 EST


Hi Mark,

On Tue, 29 Sept 2026 at 13:07, Mark Brown <broonie@xxxxxxxxxx> wrote:
> On Tue, Sep 29, 2026 at 10:48:09AM +0200, Geert Uytterhoeven wrote:
> > On Mon, 28 Sept 2026 at 16:23, Mark Brown <broonie@xxxxxxxxxx> wrote:
>
> > > }
> > > - if (tgtdev)
> > > - mpi3mr_tgtdev_put(tgtdev);
> > > kfree(mr_sas_port);
> > > return NULL;
> > > }
>
> > Are you sure this is correct?
> > Martin's conflict resolution in 0b5b2fb484f3b kept the
>
> > if (tgtdev)
> > mpi3mr_tgtdev_put(tgtdev);
>
> > part.
>
> Dunno. There was another put came in further up the function.

Right. So Martin's resolution looks incorrect, too, as it calls
mpi3mr_tgtdev_put(tgtdev) twice.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds