linux-kernel - Re: [RFC PATCH 2/2] counter: Introduce the COUNTER_COMP_ARRAY

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] [day] [month] [year] [list]

Message-ID: <Yx0gZXvZX7Os2f0q@ishi>
Date:   Sat, 10 Sep 2022 19:40:21 -0400
From:   William Breathitt Gray <william.gray@...aro.org>
To:     Julien Panis <jpanis@...libre.com>
Cc:     linux-iio@...r.kernel.org, linux-kernel@...r.kernel.org,
        mranostay@...com, fabien.lahoudere@...labora.com,
        gwendal@...omium.org, enric.balletbo@...labora.com,
        bleung@...omium.org, groeck@...omium.org, jic23@...nel.org,
        david@...hnology.com, robertcnelson@...il.com
Subject: Re: [RFC PATCH 2/2] counter: Introduce the COUNTER_COMP_ARRAY_U64
 component type

On Tue, Sep 06, 2022 at 04:41:16PM +0200, Julien Panis wrote:
> 
> 
> On 29/08/2022 17:17, William Breathitt Gray wrote:
> > The COUNTER_COMP_ARRAY_U64 Counter component type is introduced to
> > enable support for Counter array components. With Counter array
> > components, exposure for buffers on counter devices can be defined via
> > new Counter array component macros. This should simplify code for driver
> > authors who would otherwise need to define individual Counter components
> > for each array element.
> > 
> > Three Counter array component macros are introduced::
> > 
> >          COUNTER_COMP_DEVICE_ARRAY_U64(_name, _read, _write, _length)
> >          COUNTER_COMP_COUNT_ARRAY_U64(_name, _read, _write, _length)
> >          COUNTER_COMP_SIGNAL_ARRAY_U64(_name, _read, _write, _length)
> 
> Hi William,
> 
> I have 2 comments :
> 
> 1) What about ENUM ? I guess that it will not be possible to handle ARRAY of
> ENUM ?
>     That would be useful for polarity0/1/2/3 in my ECAP driver
>     (something like COUNTER_COMP_SIGNAL_ARRAY_ENUM for instance)
> 
> 2) I made some tests with COUNTER_COMP_COUNT_ARRAY_U64
>         COUNTER_COMP_COUNT_ARRAY_U64("capture", ecap_cnt_cap_read, NULL,
> ECAP_NB_CEVT)
>     I can get consistent data value when using 'cat captureX' linux
> commands.
>     But I get weird values when using watches from userspace application
> code (I will send my test results to you).
> 
> Julien Panis

I have an idea that might work for supporting enum types, so I'll try to
implement support for that in v2. I was also able to track down the bug
you found so I'll have that fixed as well.

William Breathitt Gray

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)