On Wed, 2017-05-10 at 16:02 +0200, Roland Fehrenbacher wrote: > it's hard to judge what is the most wanted default setup. The reason why > this was changed was the cited Debian bug report. I'm also a friend of > well-defined configs, so I agree with that opinion. I believe, > adding a '#' in front of a line of a config file shouldn't overburden an > admin who deals with IB storage under Linux, if she/he prefers the most > simple config. The change in Debian is also clearly communicated (NEWS > file), so I don't really see a problem. Finally: There are many packages > in Debian that have a different default config as compared to upstream. Hello Roland, Thanks for chiming in. Are you perhaps referring to https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=740945 ? If so, are you aware that changing /etc/srp_daemon.conf was not necessary because of the PORTS=NONE setting in /etc/default/srptools? BTW, I just reinstalled srptools on an Ubuntu 17.04 system (apt-get remove -y srptools && dpkg --purge srptools && rm -f /etc/default/srptools /etc/srp_daemon.conf && rm -rf /var/cache/apt/archives && apt-get install -y srptools). From what I see in the installed configuration files it looks like the change you mentioned has either not been applied or it has been reverted? # head -n 99 /etc/default/srptools /etc/srp_daemon.conf ==> /etc/default/srptools <== #How often should srpdeamon rescan the fabric (seconds) RETRIES=60 #Where should srp-deamon log to LOG=/var/log/srp_daemon.log # What ports should srp-deamon be started on. # Format is CA:port # ALL or NONE will run on all ports on none # respectively PORTS=NONE #PORTS=ALL #PORTS="mthca0:1 mlx4_0:2" ==> /etc/srp_daemon.conf <== ## This is an example rules configuration file for srp_daemon. ## #This is a comment ## disallow the following dgid #d dgid=fe800000000000000002c90200402bd5 ## allow target with the following ioc_guid #a ioc_guid=00a0b80200402bd7 ## allow target with the following pkey #a pkey=ffff ## allow target with the following id_ext and ioc_guid #a id_ext=200500A0B81146A1,ioc_guid=00a0b80200402bef ## disallow all the rest #d ## ## Here is another example: ## ## Allow all targets and set queue size to 128. # a queue_size=128,max_cmd_per_lun=128 Thanks, 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
