Re: [External] [PATCH] hostmem: Add clear option to file backend

2023-03-02 Thread Stefan Hajnoczi
On Thu, Mar 02, 2023 at 02:56:43PM +0100, David Hildenbrand wrote: > On 02.03.23 12:57, Feiran Zheng wrote: > > > > > > > On 2 Mar 2023, at 11:44, Daniel P. Berrangé wrote: > > > > > > On Thu, Mar 02, 2023 at 12:31:46PM +0100, David Hildenbrand wrote: > > > > On 02.03.23 12:09, Fam Zheng wrote:

Re: [External] [PATCH] hostmem: Add clear option to file backend

2023-03-02 Thread David Hildenbrand
On 02.03.23 14:56, David Hildenbrand wrote: On 02.03.23 12:57, Feiran Zheng wrote: On 2 Mar 2023, at 11:44, Daniel P. Berrangé wrote: On Thu, Mar 02, 2023 at 12:31:46PM +0100, David Hildenbrand wrote: On 02.03.23 12:09, Fam Zheng wrote: This adds a memset to clear the backing memory. This

Re: [External] [PATCH] hostmem: Add clear option to file backend

2023-03-02 Thread David Hildenbrand
On 02.03.23 12:57, Feiran Zheng wrote: On 2 Mar 2023, at 11:44, Daniel P. Berrangé wrote: On Thu, Mar 02, 2023 at 12:31:46PM +0100, David Hildenbrand wrote: On 02.03.23 12:09, Fam Zheng wrote: This adds a memset to clear the backing memory. This is useful in the case of PMEM DAX to drop di

Re: [External] [PATCH] hostmem: Add clear option to file backend

2023-03-02 Thread Feiran Zheng
> On 2 Mar 2023, at 11:54, David Hildenbrand wrote: > > On 02.03.23 12:48, Feiran Zheng wrote: >>> On 2 Mar 2023, at 11:39, David Hildenbrand wrote: >>> >>> On 02.03.23 12:37, Feiran Zheng wrote: > On 2 Mar 2023, at 11:31, David Hildenbrand wrote: > > On 02.03.23 12:09, Fam Zhe

Re: [External] [PATCH] hostmem: Add clear option to file backend

2023-03-02 Thread Feiran Zheng
> On 2 Mar 2023, at 11:44, Daniel P. Berrangé wrote: > > On Thu, Mar 02, 2023 at 12:31:46PM +0100, David Hildenbrand wrote: >> On 02.03.23 12:09, Fam Zheng wrote: >>> This adds a memset to clear the backing memory. This is useful in the >>> case of PMEM DAX to drop dirty data, if the backing m

Re: [External] [PATCH] hostmem: Add clear option to file backend

2023-03-02 Thread David Hildenbrand
On 02.03.23 12:48, Feiran Zheng wrote: On 2 Mar 2023, at 11:39, David Hildenbrand wrote: On 02.03.23 12:37, Feiran Zheng wrote: On 2 Mar 2023, at 11:31, David Hildenbrand wrote: On 02.03.23 12:09, Fam Zheng wrote: This adds a memset to clear the backing memory. This is useful in the case

Re: [External] [PATCH] hostmem: Add clear option to file backend

2023-03-02 Thread Feiran Zheng
> On 2 Mar 2023, at 11:39, David Hildenbrand wrote: > > On 02.03.23 12:37, Feiran Zheng wrote: >>> On 2 Mar 2023, at 11:31, David Hildenbrand wrote: >>> >>> On 02.03.23 12:09, Fam Zheng wrote: This adds a memset to clear the backing memory. This is useful in the case of PMEM DAX to

Re: [PATCH] hostmem: Add clear option to file backend

2023-03-02 Thread Daniel P . Berrangé
On Thu, Mar 02, 2023 at 12:31:46PM +0100, David Hildenbrand wrote: > On 02.03.23 12:09, Fam Zheng wrote: > > This adds a memset to clear the backing memory. This is useful in the > > case of PMEM DAX to drop dirty data, if the backing memory is handed > > over from a previous application or firmwar

Re: [External] [PATCH] hostmem: Add clear option to file backend

2023-03-02 Thread David Hildenbrand
On 02.03.23 12:37, Feiran Zheng wrote: On 2 Mar 2023, at 11:31, David Hildenbrand wrote: On 02.03.23 12:09, Fam Zheng wrote: This adds a memset to clear the backing memory. This is useful in the case of PMEM DAX to drop dirty data, if the backing memory is handed over from a previous applic

Re: [External] [PATCH] hostmem: Add clear option to file backend

2023-03-02 Thread Feiran Zheng
> On 2 Mar 2023, at 11:31, David Hildenbrand wrote: > > On 02.03.23 12:09, Fam Zheng wrote: >> This adds a memset to clear the backing memory. This is useful in the >> case of PMEM DAX to drop dirty data, if the backing memory is handed >> over from a previous application or firmware which did

Re: [PATCH] hostmem: Add clear option to file backend

2023-03-02 Thread David Hildenbrand
On 02.03.23 12:09, Fam Zheng wrote: This adds a memset to clear the backing memory. This is useful in the case of PMEM DAX to drop dirty data, if the backing memory is handed over from a previous application or firmware which didn't clean up before exiting. Why can't the VM manager do that ins

[PATCH] hostmem: Add clear option to file backend

2023-03-02 Thread Fam Zheng
This adds a memset to clear the backing memory. This is useful in the case of PMEM DAX to drop dirty data, if the backing memory is handed over from a previous application or firmware which didn't clean up before exiting. Signed-off-by: Fam Zheng --- backends/hostmem-file.c | 20