Re: [PATCH v2] arch/cacheflush: Introduce flush_all_caches()

2022-08-23 Thread Dan Williams
Davidlohr Bueso wrote: > On Mon, 22 Aug 2022, Dan Williams wrote: > > >Davidlohr Bueso wrote: > >> On Sun, 21 Aug 2022, Christoph Hellwig wrote: > >> > >> >On Fri, Aug 19, 2022 at 10:10:24AM -0700, Davidlohr Bueso wrote: > >> >> index b192d917a6d0..ac4d4fd4e508 100644 > >> >> --- a/arch/x86/includ

Re: [PATCH v2] arch/cacheflush: Introduce flush_all_caches()

2022-08-23 Thread Davidlohr Bueso
On Mon, 22 Aug 2022, Dan Williams wrote: Davidlohr Bueso wrote: On Sun, 21 Aug 2022, Christoph Hellwig wrote: >On Fri, Aug 19, 2022 at 10:10:24AM -0700, Davidlohr Bueso wrote: >> index b192d917a6d0..ac4d4fd4e508 100644 >> --- a/arch/x86/include/asm/cacheflush.h >> +++ b/arch/x86/include/asm/ca

Re: [PATCH v2] arch/cacheflush: Introduce flush_all_caches()

2022-08-22 Thread Dan Williams
Davidlohr Bueso wrote: > On Sun, 21 Aug 2022, Christoph Hellwig wrote: > > >On Fri, Aug 19, 2022 at 10:10:24AM -0700, Davidlohr Bueso wrote: > >> index b192d917a6d0..ac4d4fd4e508 100644 > >> --- a/arch/x86/include/asm/cacheflush.h > >> +++ b/arch/x86/include/asm/cacheflush.h > >> @@ -10,4 +10,8 @@

Re: [PATCH v2] arch/cacheflush: Introduce flush_all_caches()

2022-08-22 Thread Davidlohr Bueso
On Sun, 21 Aug 2022, Christoph Hellwig wrote: On Fri, Aug 19, 2022 at 10:10:24AM -0700, Davidlohr Bueso wrote: index b192d917a6d0..ac4d4fd4e508 100644 --- a/arch/x86/include/asm/cacheflush.h +++ b/arch/x86/include/asm/cacheflush.h @@ -10,4 +10,8 @@ void clflush_cache_range(void *addr, unsigne

Re: [PATCH v2] arch/cacheflush: Introduce flush_all_caches()

2022-08-21 Thread Christoph Hellwig
On Fri, Aug 19, 2022 at 10:10:24AM -0700, Davidlohr Bueso wrote: > index b192d917a6d0..ac4d4fd4e508 100644 > --- a/arch/x86/include/asm/cacheflush.h > +++ b/arch/x86/include/asm/cacheflush.h > @@ -10,4 +10,8 @@ > > void clflush_cache_range(void *addr, unsigned int size); > > +/* see comments i

Re: [PATCH v2] arch/cacheflush: Introduce flush_all_caches()

2022-08-20 Thread Davidlohr Bueso
On Fri, 19 Aug 2022, Ira Weiny wrote: Did you mean "must"? Yep. + * such as those which caches are in a consistent state. The + * caller can verify the situation early on. + */ +#ifndef flush_all_caches +# define flush_all_caches_capable() false +static inline void flush_all_caches(void) +{

Re: [PATCH v2] arch/cacheflush: Introduce flush_all_caches()

2022-08-19 Thread Ira Weiny
On Fri, Aug 19, 2022 at 10:10:24AM -0700, Davidlohr Bueso wrote: > With CXL security features, global CPU cache flushing nvdimm requirements > are no longer specific to that subsystem, even beyond the scope of > security_ops. CXL will need such semantics for features not necessarily > limited to pe

[PATCH v2] arch/cacheflush: Introduce flush_all_caches()

2022-08-19 Thread Davidlohr Bueso
With CXL security features, global CPU cache flushing nvdimm requirements are no longer specific to that subsystem, even beyond the scope of security_ops. CXL will need such semantics for features not necessarily limited to persistent memory. The functionality this is enabling is to be able to ins