Re: [PATCH] virtio pmem: fix async flush ordering

2019-11-20 Thread Dan Williams
On Wed, Nov 20, 2019 at 11:23 PM Dan Williams wrote: > > On Wed, Nov 20, 2019 at 9:26 AM Jeff Moyer wrote: > > > > Pankaj Gupta writes: > > > > > Remove logic to create child bio in the async flush function which > > > causes child bio to get executed after parent bio 'pmem_make_request' > > >

Re: [PATCH] virtio pmem: fix async flush ordering

2019-11-20 Thread Dan Williams
On Wed, Nov 20, 2019 at 9:26 AM Jeff Moyer wrote: > > Pankaj Gupta writes: > > > Remove logic to create child bio in the async flush function which > > causes child bio to get executed after parent bio 'pmem_make_request' > > completes. This resulted in wrong ordering of REQ_PREFLUSH with the

Re: [PATCH] virtio pmem: fix async flush ordering

2019-11-20 Thread Pankaj Gupta
> > > Remove logic to create child bio in the async flush function which > > causes child bio to get executed after parent bio 'pmem_make_request' > > completes. This resulted in wrong ordering of REQ_PREFLUSH with the > > data write request. > > > > Instead we are performing flush from the

Re: [PATCH] virtio pmem: fix async flush ordering

2019-11-20 Thread Jeff Moyer
Pankaj Gupta writes: > Remove logic to create child bio in the async flush function which > causes child bio to get executed after parent bio 'pmem_make_request' > completes. This resulted in wrong ordering of REQ_PREFLUSH with the > data write request. > > Instead we are performing flush fr

Re: [PATCH v2 16/18] powerpc/papr_scm: Switch to numa_map_to_online_node()

2019-11-20 Thread Michael Ellerman
Dan Williams writes: > Now that the core exports numa_map_to_online_node() switch to that > instead of the locally coded duplicate. > > Cc: Benjamin Herrenschmidt > Cc: Paul Mackerras > Cc: Michael Ellerman Acked-by: Michael Ellerman cheers > Cc: "Oliver O'Halloran" > Reported-by: "Aneesh

[PATCH] virtio pmem: fix async flush ordering

2019-11-20 Thread Pankaj Gupta
Remove logic to create child bio in the async flush function which causes child bio to get executed after parent bio 'pmem_make_request' completes. This resulted in wrong ordering of REQ_PREFLUSH with the data write request. Instead we are performing flush from the parent bio to maintain the