On Thu, 2026-01-22 at 22:29 +0100, Xose Vazquez Perez wrote: > Info from https://marc.info/?l=linux-kernel&m=122123037102604 : > > "Kiyoshi Ueda <[email protected]> wrote: > [...] > Some basic function/performance testings are done with NEC iStorage > (active-active multipath)..."
Thanks... so, you inferred the settings just from the "active-active"? That might be correct, but it's hard to say with confidence after 15+ years. Nobody has complained about this device not being in our HW table. I admire your detective skills, but I don't think that adding it now, guessing the parameters, makes much sense. Regards, Martin > > Added to BLIST in: > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=12427d99489966185dc12a0b6f725d682a3277ca > > Cc: Mike Christie <[email protected]> > Cc: Martin Wilck <[email protected]> > Cc: Benjamin Marzinski <[email protected]> > Cc: Christophe Varoqui <[email protected]> > Cc: DM_DEVEL-ML <[email protected]> > Signed-off-by: Xose Vazquez Perez <[email protected]> > --- > libmultipath/hwtable.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/libmultipath/hwtable.c b/libmultipath/hwtable.c > index 23c54933..4fc245a4 100644 > --- a/libmultipath/hwtable.c > +++ b/libmultipath/hwtable.c > @@ -872,6 +872,13 @@ static struct hwentry default_hw[] = { > .pgfailback = -FAILBACK_IMMEDIATE, > .prio_name = PRIO_ALUA, > }, > + { > + // iStorage series > + .vendor = "NEC", > + .product = "iStorage", > + .pgpolicy = MULTIBUS, > + .no_path_retry = 30, > + }, > /* > * Oracle > */
