linux-kernel - Re: [PATCH v9 1/1] gpio: add sloppy logic analyzer using polling

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Fri, 14 Jun 2024 13:58:46 +0200
From: "Arnd Bergmann" <arnd@...db.de>
To: "Wolfram Sang" <wsa+renesas@...g-engineering.com>
Cc: "Bartosz Golaszewski" <brgl@...ev.pl>,
 "Linus Walleij" <linus.walleij@...aro.org>,
 Linux-Renesas <linux-renesas-soc@...r.kernel.org>,
 "Jonathan Corbet" <corbet@....net>, "Kent Gibson" <warthog618@...il.com>,
 linux-doc@...r.kernel.org, linux-kernel@...r.kernel.org,
 "open list:GPIO SUBSYSTEM" <linux-gpio@...r.kernel.org>
Subject: Re: [PATCH v9 1/1] gpio: add sloppy logic analyzer using polling

On Fri, Jun 14, 2024, at 12:03, Wolfram Sang wrote:
> Hi Arnd, everyone,
>
>> I could imagine treating both gpio-virtuser and this code as
>> a gpiolib extension rather than a consumer (which is usually
>> part of some other subsystem's driver).
>
> I have difficulties seeing this. For the analyzer, at least. It does not
> extend gpiolib in a way another consumer could make use of it?

That's the same as the chardev support in gpiolib: it doesn't
provide functionality to in-kernel consumers but does give
an interface to userspace.

Arguably, integrating the logic analyzer into the chardev
device itself would make sense from an interface, as you
could define it as ioctls and mmap instead of the debugfs
interface.

Of course we don't want to make it a first-class interface
because of the reasons you explain in the cover letter,
but it would totally make sense to me to call it a
debugging feature of libgpio instead of calling it a
standalone driver.

     Arnd