<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> feel like this approach already isn't optimal as in reality most<br>systems aren't going to have half of those sensors, and you're then<br>creating useless matches for things that don't matter to your platform.<br></blockquote><div>I'm assuming that you're referring to the general approach of PSUSensors and not referring to extending the list using a config file?<br>If so then I agree.<br>I'm also a bit confused about scanning every hwmon attribute on the system and seeing if has a name that is in the permit list..then seeing if it has an appropriate configuration. Why not just grab the configuration and using that information navigate to the appropriate hwmon path and take what is there? (Why even bother with validating the driver name?) . Any context would be helpful; I'm trying to grok the design.<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Maybe something like allowing a device to be exported with a different<br>keyword other then 'type' in Entity Manager would allow us to just use<br>one PSUSensor config type, then your export could be hidden in your EM<br>config?</blockquote><div>Sure, or maybe a catchall type for something PSUSensor can consume (and bypass the name check?)</div><div> </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Aug 7, 2020 at 10:06 AM James Feist <<a href="mailto:james.feist@linux.intel.com">james.feist@linux.intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 8/7/2020 9:54 AM, Jason Ling wrote:<br>
>> What about making the device/type lists in PSUSensors extendable using JSON?<br>
> <br>
>     Say more about what you're wanting to do here..... <br>
> <br>
>   Take <br>
> <a href="https://github.com/openbmc/dbus-sensors/blob/master/src/PSUSensorMain.cpp#L43" rel="noreferrer" target="_blank">https://github.com/openbmc/dbus-sensors/blob/master/src/PSUSensorMain.cpp#L43</a> and <br>
> <a href="https://github.com/openbmc/dbus-sensors/blob/master/src/PSUSensorMain.cpp#L59" rel="noreferrer" target="_blank">https://github.com/openbmc/dbus-sensors/blob/master/src/PSUSensorMain.cpp#L59</a> and <br>
> make it so those can be extended using (for example) without an upstream <br>
> code change. I picked JSON as the easiest example.<br>
> IIRC PSUSensors does validity checks to make sure that the device emits <br>
> a name in its 'permit list' (hwmontempsensor is less picky) so tricking <br>
> PSUSensors into gathering telemetry for a non-public device is tricky.<br>
<br>
I feel like this approach already isn't optimal as in reality most <br>
systems aren't going to have half of those sensors, and you're then <br>
creating useless matches for things that don't matter to your platform. <br>
Maybe something like allowing a device to be exported with a different <br>
keyword other then 'type' in Entity Manager would allow us to just use <br>
one PSUSensor config type, then your export could be hidden in your EM <br>
config?<br>
<br>
<br>
> <br>
>     Can you give an<br>
>     example of what you would use it for?<br>
> <br>
> Sure, the primary use case would be<br>
> It's a non public device. Would rather not broadcast information about <br>
> it or have to obfuscate the device name. Really would rather not <br>
> maintain patches until the device is made public.<br>
> <br>
> <br>
> <br>
> On Fri, Aug 7, 2020 at 9:39 AM Ed Tanous <<a href="mailto:ed@tanous.net" target="_blank">ed@tanous.net</a> <br>
> <mailto:<a href="mailto:ed@tanous.net" target="_blank">ed@tanous.net</a>>> wrote:<br>
> <br>
>     On Fri, Aug 7, 2020 at 9:36 AM Jason Ling <<a href="mailto:jasonling@google.com" target="_blank">jasonling@google.com</a><br>
>     <mailto:<a href="mailto:jasonling@google.com" target="_blank">jasonling@google.com</a>>> wrote:<br>
>      ><br>
>      > Slightly different topic:<br>
>      > What about making the device/type lists in PSUSensors extendable<br>
>     using JSON?<br>
>      ><br>
> <br>
>     Say more about what you're wanting to do here.....  Can you give an<br>
>     example of what you would use it for?<br>
> <br>
</blockquote></div>