linux-kernel - Re: [PATCH 23/27] sparc: Use common bits from generic tlb.h

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20140514202242.GA3167@ravnborg.org>
Date:	Wed, 14 May 2014 22:22:42 +0200
From:	Sam Ravnborg <sam@...nborg.org>
To:	Richard Weinberger <richard@....at>
Cc:	linux-arch@...r.kernel.org, arnd@...db.de,
	linux-kernel@...r.kernel.org,
	"David S. Miller" <davem@...emloft.net>, sparclinux@...r.kernel.org
Subject: Re: [PATCH 23/27] sparc: Use common bits from generic tlb.h

On Wed, May 14, 2014 at 10:04:41PM +0200, Richard Weinberger wrote:
> Am 14.05.2014 21:55, schrieb Sam Ravnborg:
> > On Wed, May 14, 2014 at 08:59:55PM +0200, Richard Weinberger wrote:
> >> It is no longer needed to define them on our own.
> >>
> >> Cc: "David S. Miller" <davem@...emloft.net>
> >> Cc: Richard Weinberger <richard@....at>
> >> Cc: sparclinux@...r.kernel.org
> >> Cc: linux-kernel@...r.kernel.org
> >> Signed-off-by: Richard Weinberger <richard@....at>
> >> ---
> >>  arch/sparc/include/asm/tlb_32.h | 8 --------
> >>  arch/sparc/include/asm/tlb_64.h | 3 ---
> >>  2 files changed, 11 deletions(-)
> >>
> >> diff --git a/arch/sparc/include/asm/tlb_32.h b/arch/sparc/include/asm/tlb_32.h
> >> index 6d02d1c..60641a1 100644
> >> --- a/arch/sparc/include/asm/tlb_32.h
> >> +++ b/arch/sparc/include/asm/tlb_32.h
> >> @@ -11,14 +11,6 @@ do {								\
> >>  	flush_tlb_range(vma, vma->vm_start, vma->vm_end);	\
> >>  } while (0)
> >>  
> >> -#define __tlb_remove_tlb_entry(tlb, pte, address) \
> >> -	do { } while (0)
> >> -
> >> -#define tlb_flush(tlb) \
> >> -do {								\
> >> -	flush_tlb_mm((tlb)->mm);				\
> >> -} while (0)
> >> -
> >>  #include <asm-generic/tlb.h>
> >>  
> >>  #endif /* _SPARC_TLB_H */
> > 
> > This part looks OK.
> > 
> >> diff --git a/arch/sparc/include/asm/tlb_64.h b/arch/sparc/include/asm/tlb_64.h
> >> index 190e189..36fdcec 100644
> >> --- a/arch/sparc/include/asm/tlb_64.h
> >> +++ b/arch/sparc/include/asm/tlb_64.h
> >> @@ -22,9 +22,6 @@ extern void smp_flush_tlb_mm(struct mm_struct *mm);
> >>  extern void __flush_tlb_pending(unsigned long, unsigned long, unsigned long *);
> >>  extern void flush_tlb_pending(void);
> >>  
> >> -#define tlb_start_vma(tlb, vma) do { } while (0)
> >> -#define tlb_end_vma(tlb, vma)	do { } while (0)
> >> -#define __tlb_remove_tlb_entry(tlb, ptep, address) do { } while (0)
> >>  #define tlb_flush(tlb)	flush_tlb_pending()
> >>  
> >>  #include <asm-generic/tlb.h>
> > 
> > This does not fly for sparc64.
> > __pte_free_tlb is a static inline in pgalloc_64.h so no
> > define exist to teach tlb.h that an arch override exist.
> 
> Okay, we need a #define __pte_free_tlb __pte_free_tlb.
> 
> > Likewise for __pmd_free_tlb
> 
> __pmd_free_tlb is already a macro so the generic tlb.h should handle it correctly.
> 
> #define __pmd_free_tlb(tlb, pmd, addr)                \
>         pgtable_free_tlb(tlb, pmd, false)
> 
> I'd move both __pmd_free_tlb and __pte_free_tlb into tlb.h such that the include order
> of tlb.h vs pgalloc.h does not matter.
> Are you fine with that?
Yes, as long as sparc64 continue to build it makes good sense
to define these in the same file as done by most other architectures.

	Sam
> 
> Thanks,
> //richard
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/