[0/4] crypto: IAA stats bugfixes and simplifications

Message

Zanussi, Tom March 4, 2024, 9:20 p.m. UTC

Hi Herbert,

While doing some testing, I noticed a discrepancy in the
decomp_bytes_in stat, which the first patch in this series (crypto:
iaa - fix decomp_bytes_in stats) fixes.

I also realized that there were some other problems unrelated to that
but also that the stats code could be simplified in a number of ways
and that some of it wasn't really useful.  The stats code is debugging
code and has been helpful to quickly verify whether things are
basically working, but since it's there we should make it as accurate
and actually useful as possible.

I realize the second patch (crypto: iaa - Remove comp/decomp delay
statistics) removes the code I just fixed up in a patch you just
merged (crypto: iaa - Fix comp/decomp delay statistics) - let me know
if you want me to combine those if you want to remove the latter from
your branch...

Thanks,

Tom


*** BLURB HERE ***

Tom Zanussi (4):
  crypto: iaa - fix decomp_bytes_in stats
  crypto: iaa - Remove comp/decomp delay statistics
  crypto: iaa - Add global_stats file and remove individual stat files
  crypto: iaa - Change iaa statistics to atomic64_t

 .../driver-api/crypto/iaa/iaa-crypto.rst      |  76 +++++---
 drivers/crypto/intel/iaa/iaa_crypto.h         |  16 +-
 drivers/crypto/intel/iaa/iaa_crypto_main.c    |  13 +-
 drivers/crypto/intel/iaa/iaa_crypto_stats.c   | 183 ++++++++----------
 drivers/crypto/intel/iaa/iaa_crypto_stats.h   |   8 -
 5 files changed, 140 insertions(+), 156 deletions(-)

Comments

Herbert Xu March 28, 2024, 10:44 a.m. UTC | #1

On Mon, Mar 04, 2024 at 03:20:07PM -0600, Tom Zanussi wrote:
> Hi Herbert,
> 
> While doing some testing, I noticed a discrepancy in the
> decomp_bytes_in stat, which the first patch in this series (crypto:
> iaa - fix decomp_bytes_in stats) fixes.
> 
> I also realized that there were some other problems unrelated to that
> but also that the stats code could be simplified in a number of ways
> and that some of it wasn't really useful.  The stats code is debugging
> code and has been helpful to quickly verify whether things are
> basically working, but since it's there we should make it as accurate
> and actually useful as possible.
> 
> I realize the second patch (crypto: iaa - Remove comp/decomp delay
> statistics) removes the code I just fixed up in a patch you just
> merged (crypto: iaa - Fix comp/decomp delay statistics) - let me know
> if you want me to combine those if you want to remove the latter from
> your branch...
> 
> Thanks,
> 
> Tom
> 
> 
> *** BLURB HERE ***
> 
> Tom Zanussi (4):
>   crypto: iaa - fix decomp_bytes_in stats
>   crypto: iaa - Remove comp/decomp delay statistics
>   crypto: iaa - Add global_stats file and remove individual stat files
>   crypto: iaa - Change iaa statistics to atomic64_t
> 
>  .../driver-api/crypto/iaa/iaa-crypto.rst      |  76 +++++---
>  drivers/crypto/intel/iaa/iaa_crypto.h         |  16 +-
>  drivers/crypto/intel/iaa/iaa_crypto_main.c    |  13 +-
>  drivers/crypto/intel/iaa/iaa_crypto_stats.c   | 183 ++++++++----------
>  drivers/crypto/intel/iaa/iaa_crypto_stats.h   |   8 -
>  5 files changed, 140 insertions(+), 156 deletions(-)
> 
> -- 
> 2.34.1

All applied.  Thanks.