mbox series

[SRU,jammy:gcp,0/1] Incorrect argument to notify_page_enc_status_changed()

Message ID 20240205170052.606221-1-john.cabaj@canonical.com
Headers show
Series [SRU,jammy:gcp,1/1] UBUNTU: SAUCE (no-up): x64/mm: Fix incorrect argument to notify_page_enc_status_changed() | expand

Message

John Cabaj Feb. 5, 2024, 5 p.m. UTC 
BugLink: https://bugs.launchpad.net/bugs/2052451

[Impact]

* Google requested fix to notify_page_enc_status_changed() passing incorrect argument.

[Fix]

* 27ba8c7 modified notify_page_enc_status_changed(), when it should not have

[Test Case]

* Compile tested
* Boot tested
* Tested by Google

[Where things could go wrong]

* Low chance of regression. Isolated logic fix.

John Cabaj (1):
  UBUNTU: SAUCE (no-up): x64/mm: Fix incorrect argument to
    notify_page_enc_status_changed()

 arch/x86/mm/mem_encrypt_amd.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thibault Ferrante Feb. 6, 2024, 12:35 p.m. UTC | #1 
On 05-02-2024 18:00, John Cabaj wrote:
> BugLink: https://bugs.launchpad.net/bugs/2052451
> 
> [Impact]
> 
> * Google requested fix to notify_page_enc_status_changed() passing incorrect argument.
> 
> [Fix]
> 
> * 27ba8c7 modified notify_page_enc_status_changed(), when it should not have
> 
> [Test Case]
> 
> * Compile tested
> * Boot tested
> * Tested by Google
> 
> [Where things could go wrong]
> 
> * Low chance of regression. Isolated logic fix.
> 
> John Cabaj (1):
>    UBUNTU: SAUCE (no-up): x64/mm: Fix incorrect argument to
>      notify_page_enc_status_changed()
> 
>   arch/x86/mm/mem_encrypt_amd.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 

Acked-by: Thibault Ferrante <thibault.ferrante@canonical.com>

--
Thibault
Andrei Gherzan Feb. 6, 2024, 1:42 p.m. UTC | #2 
On 24/02/05 11:00AM, John Cabaj wrote:
> BugLink: https://bugs.launchpad.net/bugs/2052451
> 
> [Impact]
> 
> * Google requested fix to notify_page_enc_status_changed() passing incorrect argument.
> 
> [Fix]
> 
> * 27ba8c7 modified notify_page_enc_status_changed(), when it should not have
> 
> [Test Case]
> 
> * Compile tested
> * Boot tested
> * Tested by Google
> 
> [Where things could go wrong]
> 
> * Low chance of regression. Isolated logic fix.
> 
> John Cabaj (1):
>   UBUNTU: SAUCE (no-up): x64/mm: Fix incorrect argument to
>     notify_page_enc_status_changed()
> 
>  arch/x86/mm/mem_encrypt_amd.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Acked-by: Andrei Gherzan <andrei.gherzan@canonical.com>
John Cabaj Feb. 6, 2024, 2:44 p.m. UTC | #3 
On 2/5/24 11:00 AM, John Cabaj wrote:
> BugLink: https://bugs.launchpad.net/bugs/2052451
> 
> [Impact]
> 
> * Google requested fix to notify_page_enc_status_changed() passing incorrect argument.
> 
> [Fix]
> 
> * 27ba8c7 modified notify_page_enc_status_changed(), when it should not have
> 
> [Test Case]
> 
> * Compile tested
> * Boot tested
> * Tested by Google
> 
> [Where things could go wrong]
> 
> * Low chance of regression. Isolated logic fix.
> 
> John Cabaj (1):
>   UBUNTU: SAUCE (no-up): x64/mm: Fix incorrect argument to
>     notify_page_enc_status_changed()
> 
>  arch/x86/mm/mem_encrypt_amd.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Applied to jammy:linux-gcp master-next branch.


Thanks,
John