* RFE: export watchdog features and version by /sys
@ 2019-06-06 10:01 Karel Zak
2019-06-06 17:13 ` Guenter Roeck
0 siblings, 1 reply; 3+ messages in thread
From: Karel Zak @ 2019-06-06 10:01 UTC (permalink / raw)
To: Wim Van Sebroeck, linux-watchdog
Hi,
I'd like to improve wdctl(8) to read watchdog info from /sys, but I
found that sysfs does not provide all information like ioctl API.
Unfortunately, there is no way how from /sys get info about supported
watchdog features and firmware version. This info is accessible only
by WDIOC_GETSUPPORT ioclt (struct watchdog_info->options and
watchdog_info->firmware_version).
It would be nice to have "options" and "firmware_version" also in in
sysfs. Please :-)
Karel
--
Karel Zak <kzak@redhat.com>
http://karelzak.blogspot.com
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: RFE: export watchdog features and version by /sys
2019-06-06 10:01 RFE: export watchdog features and version by /sys Karel Zak
@ 2019-06-06 17:13 ` Guenter Roeck
2019-06-17 13:57 ` Karel Zak
0 siblings, 1 reply; 3+ messages in thread
From: Guenter Roeck @ 2019-06-06 17:13 UTC (permalink / raw)
To: Karel Zak; +Cc: Wim Van Sebroeck, linux-watchdog
On Thu, Jun 06, 2019 at 12:01:31PM +0200, Karel Zak wrote:
>
> Hi,
>
> I'd like to improve wdctl(8) to read watchdog info from /sys, but I
> found that sysfs does not provide all information like ioctl API.
>
> Unfortunately, there is no way how from /sys get info about supported
> watchdog features and firmware version. This info is accessible only
> by WDIOC_GETSUPPORT ioclt (struct watchdog_info->options and
> watchdog_info->firmware_version).
>
> It would be nice to have "options" and "firmware_version" also in in
> sysfs. Please :-)
>
Makes sense. Care to submit patches implementing it ?
We would have to resolve the conflict against the existing
firmware_version attribute provided by ziirave_wdt.
Thanks,
Guenter
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: RFE: export watchdog features and version by /sys
2019-06-06 17:13 ` Guenter Roeck
@ 2019-06-17 13:57 ` Karel Zak
0 siblings, 0 replies; 3+ messages in thread
From: Karel Zak @ 2019-06-17 13:57 UTC (permalink / raw)
To: Guenter Roeck; +Cc: Wim Van Sebroeck, linux-watchdog
On Thu, Jun 06, 2019 at 10:13:04AM -0700, Guenter Roeck wrote:
> On Thu, Jun 06, 2019 at 12:01:31PM +0200, Karel Zak wrote:
> >
> > Hi,
> >
> > I'd like to improve wdctl(8) to read watchdog info from /sys, but I
> > found that sysfs does not provide all information like ioctl API.
> >
> > Unfortunately, there is no way how from /sys get info about supported
> > watchdog features and firmware version. This info is accessible only
> > by WDIOC_GETSUPPORT ioclt (struct watchdog_info->options and
> > watchdog_info->firmware_version).
> >
> > It would be nice to have "options" and "firmware_version" also in in
> > sysfs. Please :-)
> >
>
> Makes sense. Care to submit patches implementing it ?
Frankly, although it seems like relatively simple task it would be
better if some kernel developer will do it :-)
> We would have to resolve the conflict against the existing
> firmware_version attribute provided by ziirave_wdt.
Hmm, maybe use "version" as a filename for the attribute.
Karel
--
Karel Zak <kzak@redhat.com>
http://karelzak.blogspot.com
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2019-06-17 13:57 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-06-06 10:01 RFE: export watchdog features and version by /sys Karel Zak
2019-06-06 17:13 ` Guenter Roeck
2019-06-17 13:57 ` Karel Zak
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox