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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




----- Original Message -----
> From: "Bart Van Assche" <bart.vanassche@xxxxxxxxxxx>
> To: "Benjamin Drung" <benjamin.drung@xxxxxxxxxxxxxxxx>, linux-rdma@xxxxxxxxxxxxxxx
> 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@xxxxxxxxxxxxxxx
> 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@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux