On Sun, Jun 09, 2013 at 09:09:24PM +0100, Ben Hutchings wrote:
> Commit 52f36be0f4e2 's390/pgtable: Fix check for pgste/storage key
> handling', which was commit b56433cb782d upstream, added a use of
> pgste to ptep_modify_prot_start(), but this variable does not exist.
> In mainline, pgste was added by commit d3383632d4e8 's390/mm: add pte
> invalidation notifier for kvm' and initialised to the return value of
> pgste_get_lock(ptep).  Initialise it similarly here.
> 
> Compile-tested only.
> 
> Signed-off-by: Ben Hutchings <[email protected]>
> ---
> --- a/arch/s390/include/asm/pgtable.h
> +++ b/arch/s390/include/asm/pgtable.h
> @@ -1063,11 +1063,12 @@ static inline pte_t ptep_modify_prot_sta
>                                          unsigned long address,
>                                          pte_t *ptep)
>  {
> +     pgste_t pgste;
>       pte_t pte;
>  
>       mm->context.flush_mm = 1;
>       if (mm_has_pgste(mm))
> -             pgste_get_lock(ptep);
> +             pgste = pgste_get_lock(ptep);
>  
>       pte = *ptep;
>       if (!mm_exclusive(mm))

Looks good. Thanks!

--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to