linux.git/drivers/crypto/qat, branch v6.0

crypto: qat - expose device config through sysfs for 4xxx

2022-07-08T07:15:59+00:00

qat_4xxx devices can be configured to allow either crypto or compression
operations. At the moment, devices are configured statically according to
the following rule:
- odd numbered devices assigned to compression services
- even numbered devices assigned to crypto services

Expose the sysfs attribute /sys/bus/pci/devices//qat/cfg_services
to allow to detect the configuration of a device and to change it.

The `cfg_service` attribute is only exposed for qat_4xxx devices and it
is limited to two configurations: (1) "sym;asym" for crypto services and
"dc" for compression services.

Signed-off-by: Giovanni Cabiddu 
Co-developed-by: Tomasz Kowallik 
Signed-off-by: Tomasz Kowallik 
Reviewed-by: Adam Guerin 
Reviewed-by: Fiona Trahe 
Reviewed-by: Wojciech Ziemba 
Reviewed-by: Vladis Dronov 
Signed-off-by: Herbert Xu

crypto: qat - relocate and rename adf_sriov_prepare_restart()

2022-07-08T07:15:59+00:00

The function adf_sriov_prepare_restart() is used in adf_sriov.c to stop
and shutdown a device preserving its configuration.

Since this function will be re-used by the logic that allows to
reconfigure the device through sysfs, move it to adf_init.c and rename
it as adf_dev_shutdown_cache_cfg();

Signed-off-by: Giovanni Cabiddu 
Reviewed-by: Adam Guerin 
Reviewed-by: Fiona Trahe 
Reviewed-by: Wojciech Ziemba 
Reviewed-by: Vladis Dronov 
Signed-off-by: Herbert Xu

crypto: qat - change behaviour of adf_cfg_add_key_value_param()

2022-07-08T07:15:59+00:00

The function adf_cfg_add_key_value_param() allows to insert duplicates
entries in the key value store of the driver.

Change the behaviour of that function to the following policy:
- if the key doesn't exist, add it;
- if the key already exists with a different value, then delete it and
  replace it with a new one containing the new value;
- if the key exists with the same value, then return without doing
  anything.

The behaviour of this function has been changed in order to easily
update key-values in the driver database. In particular this is required
to update the value of the ServiceEnables key used to change the service
loaded on a device.

Signed-off-by: Giovanni Cabiddu 
Reviewed-by: Adam Guerin 
Reviewed-by: Fiona Trahe 
Reviewed-by: Wojciech Ziemba 
Reviewed-by: Vladis Dronov 
Signed-off-by: Herbert Xu

crypto: qat - expose device state through sysfs for 4xxx

2022-07-08T07:15:59+00:00

Expose the device state through an attribute in sysfs and allow to
change it. This is to stop and shutdown a QAT device in order to change
its configuration.

The state attribute has been added to a newly created `qat` attribute
group which will contain all _QAT specific_ attributes.

The logic that implements the sysfs entries is part of a new file,
adf_sysfs.c. This exposes an entry point to allow the driver to create
attributes.

The function that creates the sysfs attributes is called from the probe
function of the driver and not in the state machine init function to
allow the change of states even if the device is in the down state.

In order to restore the device configuration between a transition from
down to up, the function that configures the devices has been abstracted
into the HW data structure.

The `state` attribute is only exposed for qat_4xxx devices.

Signed-off-by: Giovanni Cabiddu 
Co-developed-by: Tomasz Kowallik 
Signed-off-by: Tomasz Kowallik 
Reviewed-by: Adam Guerin 
Reviewed-by: Fiona Trahe 
Reviewed-by: Wojciech Ziemba 
Reviewed-by: Vladis Dronov 
Signed-off-by: Herbert Xu

crypto: qat - Removes the x86 dependency on the QAT drivers

2022-06-24T09:12:29+00:00

This dependency looks outdated. After the previous patch, we have been able
to use this driver to encrypt some data and to create working VF on arm64.
We have not tested it yet on any big endian machine, hence the new dependency

Signed-off-by: Yoan Picchi 
Signed-off-by: Herbert Xu

crypto: qat - replace get_current_node() with numa_node_id()

2022-06-24T09:12:29+00:00

Currently the QAT driver code uses a self-defined wrapper function
called get_current_node() when it wants to learn the current NUMA node.
This implementation references the topology_physical_package_id[] array,
which more or less coincidentally contains the NUMA node id, at least
on x86.

Because this is not universal, and Linux offers a direct function to
learn the NUMA node ID, replace that function with a call to
numa_node_id(), which would work everywhere.

This fixes the QAT driver operation on arm64 machines.

Reported-by: Yoan Picchi 
Signed-off-by: Andre Przywara 
Signed-off-by: Yoan Picchi 
Signed-off-by: Herbert Xu

crypto: qat - add support for 401xx devices

2022-05-20T05:49:18+00:00

QAT_401xx is a derivative of 4xxx. Add support for that device in the
qat_4xxx driver by including the DIDs (both PF and VF), extending the
probe and the firmware loader.

Signed-off-by: Giovanni Cabiddu 
Reviewed-by: Srinivas Kerekare 
Signed-off-by: Herbert Xu

crypto: qat - re-enable registration of algorithms

2022-05-20T05:49:18+00:00

Re-enable the registration of algorithms after fixes to (1) use
pre-allocated buffers in the datapath and (2) support the
CRYPTO_TFM_REQ_MAY_BACKLOG flag.

This reverts commit 8893d27ffcaf6ec6267038a177cb87bcde4dd3de.

Cc: stable@vger.kernel.org
Signed-off-by: Giovanni Cabiddu 
Reviewed-by: Marco Chiappero 
Reviewed-by: Adam Guerin 
Reviewed-by: Wojciech Ziemba 
Signed-off-by: Herbert Xu

crypto: qat - honor CRYPTO_TFM_REQ_MAY_SLEEP flag

2022-05-20T05:49:18+00:00

If a request has the flag CRYPTO_TFM_REQ_MAY_SLEEP set, allocate memory
using the flag GFP_KERNEL otherwise use GFP_ATOMIC.

Signed-off-by: Giovanni Cabiddu 
Reviewed-by: Adam Guerin 
Reviewed-by: Wojciech Ziemba 
Signed-off-by: Herbert Xu

crypto: qat - add param check for DH

2022-05-20T05:49:18+00:00

Reject requests with a source buffer that is bigger than the size of the
key. This is to prevent a possible integer underflow that might happen
when copying the source scatterlist into a linear buffer.

Cc: stable@vger.kernel.org
Signed-off-by: Giovanni Cabiddu 
Reviewed-by: Adam Guerin 
Reviewed-by: Wojciech Ziemba 
Signed-off-by: Herbert Xu