ACK/cmnt: [SRU][Bionic][PATCH 0/1] crypto: vmx - Fix sleep-in-atomic bugs

[Kleber Souza]

On 10/12/18 20:22, Joseph Salisbury wrote:
> BugLink: https://bugs.launchpad.net/bugs/1790832
> 
> == SRU Justification ==
> IBM is requesting this patch in all releases order to fix the sleep-in-atomic 
> bugs in AES-CBC and AES-XTS VMX implementations.
> 
> This patch is a clean cherry pick and has already been applied to
> Cosmic.  It is also needed in Xenial, but there was a minor context
> diff, so the Xenial SRU will be sent separatly.  
> 
> == Fix ==
> 0522236d4f9c ("crypto: vmx - Fix sleep-in-atomic bugs")
> 
> == Regression Potential ==
> Low.  This patch has also been cc'd to upstream stable, so it has had
> additional upstream review.
> 
> == Test Case ==
> A test kernel was built with this patch and tested by the original bug reporter.
> The bug reporter states the test kernel resolved the bug.
> 
> 
> Ondrej Mosnacek (1):
>   crypto: vmx - Fix sleep-in-atomic bugs
> 
>  drivers/crypto/vmx/aes_cbc.c | 30 ++++++++++++++----------------
>  drivers/crypto/vmx/aes_xts.c | 21 ++++++++++++++-------
>  2 files changed, 28 insertions(+), 23 deletions(-)
> 

Acked-by: Kleber Sacilotto de Souza <kleber.souza at canonical.com>


Hi Joe,

If I understood correctly, only the Xenial version of the patch needed
some context adjustment, is that correct?

If that's the case this patch can be applied as is, otherwise the
provenance needs to be adjusted as well.

Thanks,
Kleber