On 04/22/2018 05:06 PM, Tony Krowiak wrote:
On 04/20/2018 08:26 AM, David Hildenbrand wrote:
On 19.04.2018 23:13, Tony Krowiak wrote:
Introduces a new function to reset the crypto attributes for all
vcpus whether they are running or not. Each vcpu in KVM will
be removed from SIE prior to resetting the crypto attributes in its
SIE state description. After all vcpus have had their crypto attributes
reset the vcpus will be restored to SIE.
This function is incorporated into the kvm_s390_vm_set_crypto(kvm)
function to fix a reported issue whereby the crypto key wrapping
attributes could potentially get out of synch for running vcpus.
Reported-by: Halil Pasic <pasic@xxxxxxxxxxxxxxxxxx>
A reported-by for a code refactoring is strange.
I was asked to include this.
Is this a refactoring or a fix? I the message you state that you are
fixing an issue (aka bug). If you are not, fixing an issue, but indeed
just refactoring, Suggested-by would be more appropriate.
Regards,
Halil
