Jack Morgenstein wrote:
> The sysfs entries you refer to are introduced in commit 
> 7ff93f8b7ecbc36e7ffc5c11a61643821c1bfee5
> which patches in ofed but not upstream are you referring to?

Hi Jack,

I took another look, indeed the mlx4_port{1,2} sysfs entries are introduced in 
the commit
you pointed on and their permissions looks okay (S_IRUGO | S_IWUSR), they are 
not world writable. 

As for the port_trigger sysfs entry, it is introduced by a patch shipped with 
ofed which isn't upstream (mlx4_1190_sense_port_trigger.patch) and indeed this 
entry is world writable.

So the question here, if there's any reason for multi-protocol related patches 
such as this
guy and its such not to be pushed upstream? I failed to get any constructive 
response (== pathces to Roland or Dave Miller) from Yevgeny and I was hoping 
you could be helpful here.

Or.

> Sumeet Lahorani wrote:
>> # find /sys -type f -perm -222
>> /sys/devices/pci0000:00/0000:00:04.0/0000:13:00.0/port_trigger
>> /sys/devices/pci0000:00/0000:00:04.0/0000:13:00.0/mlx4_port2
>> /sys/devices/pci0000:00/0000:00:04.0/0000:13:00.0/mlx4_port1
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to