linux-kernel - Re: [PATCH v8 08/14] crypto: iaa - Add IAA Compression Accelerator Documentation

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: <2f80afbe7c586e7ee9a554787b5daf4f75e85aad.camel@linux.intel.com>
Date:   Mon, 21 Aug 2023 13:44:33 -0500
From:   Tom Zanussi <tom.zanussi@...ux.intel.com>
To:     Pavel Machek <pavel@....cz>
Cc:     herbert@...dor.apana.org.au, davem@...emloft.net,
        fenghua.yu@...el.com, vkoul@...nel.org, dave.jiang@...el.com,
        tony.luck@...el.com, wajdi.k.feghali@...el.com,
        james.guilford@...el.com, kanchana.p.sridhar@...el.com,
        vinodh.gopal@...el.com, giovanni.cabiddu@...el.com,
        linux-kernel@...r.kernel.org, linux-crypto@...r.kernel.org,
        dmaengine@...r.kernel.org
Subject: Re: [PATCH v8 08/14] crypto: iaa - Add IAA Compression Accelerator
 Documentation

Hi Pavel,

On Mon, 2023-08-21 at 19:21 +0200, Pavel Machek wrote:
> Hi!
> 
> > Because the IAA Compression Accelerator requires significant user
> > setup in order to be used properly, this adds documentation on the
> > iaa_crypto driver including setup, usage, and examples.
> > 
> > Signed-off-by: Tom Zanussi <tom.zanussi@...ux.intel.com>
> 
> 
> > +accel-config
> > +------------
> > +
> > +Unlike typical drivers, the iaa_crypto driver does not enable the
> > +device on driver load.  Due to complexity and configurability of
> > the
> > +accelerator devices, it was a design decision to have the user
> > +configure the device and manually enable the desired devices and
> > +workqueues.
> 
> Is the driver really so special? Is it widely used besides of zswap?
> Could some kind of 
> simple default configuration good enough for zswap be provided?

You're right, currently its main use is for zswap (though the original
submission also supported zram).

This idea of a simple default configuration makes sense and has come up
before. I'll try to come up with a good way to accomplish that in the
next revision.

> 
> > +The userspace tool to help doing that is called accel-config. 
> > Using
> > +accel-config to configure device or loading a previously saved
> > config
> > +is highly recommended.  The device can be controlled via sysfs
> > +directly but comes with the warning that do this ONLY if you know
> > +exactly what you are doing.  This document will not cover the
> > sysfs
> > +interface but assumes you will be using accel-config.
> 
> Not covering the interface here is okay, but we really should have
> description somewhere, 
> and there should be pointer to it here.

OK, yeah, that also makes sense - I'll add something for that too.

> 
> > +++ b/Documentation/driver-api/crypto/iaa/index.rst
> > @@ -0,0 +1,20 @@
> > +.. SPDX-License-Identifier: GPL-2.0
> > +
> > +=================================
> > +IAA (Intel Analytics Accelerator)
> > +=================================
> > +
> > +IAA provides hardware compression and decompression via the crypto
> > +API.
> 
> Why is it called "analytics accelerator" when its main function is
> compression?
> 

Well, the compression/decompression functions are only a part of what
the hardware provides, so I guess the designers used 'analytics
accelerator' as an umbrella term covering everything.

>From the hardware documentation, the IAA hardware "provides very high
throughput compression and decompression combined with analytics
primitive functions. The analytic functions are commonly used for
filtering data during analytic query processing. It primarily targets
applications such as big-data and in-memory analytics databases, as
well as application-transparent usages such as memory page
compression."

Hope that helps explain the naming..

Tom


> Best regards,
>                                                                      
>    Pavel