Re: srp_daemon : ​Disallow ​all targets ​if ​not ​explicitly ​allowed by default

[Laurence Oberman <loberman-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org>]

----- Original Message -----
> From: "Bart Van Assche" <bart.vanassche-XdAiOPVOjttBDgjK7y7TUQ@public.gmane.org>
> To: "Benjamin Drung" <benjamin.drung-EIkl63zCoXaH+58JC4qpiA@public.gmane.org>, linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> Sent: Tuesday, May 9, 2017 1:11:06 PM
> Subject: Re: srp_daemon : ​Disallow ​all targets ​if ​not ​explicitly ​allowed by default
> 
> On 05/09/17 10:07, Benjamin Drung wrote:
> > srptools 1.0.3-2 in Debian disallows all targets if not explicitly
> > allowed by default. Motivation (taken from debian/changelog):
> > 
> > * Don't activate any targets per default. (Closes: #740945)
> >   This is more sensible than the previous default of bringing
> >   up all targets in the IB fabric upon boot. In a larger fabric
> >   with many storage targets available, most of the times only
> >   one or a few targets are wanted on a particular machine.
> > 
> > Do you agree and could you change the default upstream? I prefer not to
> > carry a different behavior in Debian alone.
> 
> Hello Benjamin,
> 
> What I expect is that users will hate this change. They will notice that
> after they have installed and enabled srp_daemon that no targets are
> discovered without having any clue why no automatic login to SRP targets
> occurs.
> 
> Bart.
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

Hello

Indeed, I agree with Bart here, this change will lead to a lot of confusion about why users no longer see device discovery.
For me I would prefer its not changed.

Regards
Laurence
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html