Thread (2 messages) 2 messages, 2 authors, 2016-10-29

Re: [PATCH 03/17] batman-adv: Add network_coding and mcast sysfs files to README

From: Sven Eckelmann <hidden>
Date: 2016-10-29 11:46:59
Also in: batman

On Samstag, 29. Oktober 2016 12:56:28 CEST Jiri Pirko wrote:
[...]
quoted
quoted
I strongly believe it is a huge mistake to use sysfs for things like
this. This should be done via generic netlink api.
This doesn't change the problem that it is already that way. This patch
only adds the list of available files to the README.
Sure. Just found out you did it like that. Therefore I commented. I
suggest to rework the api to use genl entirely.
Fair enough, I have added it to the issue tracker [1].

It seems there is no easy way to drop support for modifying batman-adv
attributes of the interface or its ports via sysfs in the near
future. But disallowing sysfs for new attributes might be a viable
policy.

Kind regards,
	Sven

[1] https://www.open-mesh.org/issues/300

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help