[dm-devel] [PATCH v2 2/3] multipathd: update dm_info on multipath change events

2021-12-15 Thread Benjamin Marzinski
When multipathd gets a change event for a multipath device, the dm info may have changed, so update it. Signed-off-by: Benjamin Marzinski Reviewed-by: Martin Wilck --- multipathd/main.c | 1 + 1 file changed, 1 insertion(+) diff --git a/multipathd/main.c b/multipathd/main.c index fe40f6e3..2e9

[dm-devel] [PATCH v2 3/3] multipathd: avoid unnecessary path read-only reloads

2021-12-15 Thread Benjamin Marzinski
A mulitpath device can only be reloaded read/write when all paths are read/write. Also, whenever a read-only scsi device is rescanned, the scsi subsystem will first unconditionally issue a uevent with DISK_RO=0 before checking the read-only status, and if it the device is still read-only, issuing a

[dm-devel] [PATCH v2 0/3] multipathd: avoid unnecessary read-only reloads

2021-12-15 Thread Benjamin Marzinski
This patchset is an expanded version of my single patch "multipathd: avoid unnecessary path read-only reloads" That also makes some changes to how the multipath dm_info is handled. changes from v1, (suggested by Martin Wilck): 0001: This has been completely rewritten to embed the dmi structure

[dm-devel] [PATCH v2 1/3] libmultipath: embed dm_info in multipath structure

2021-12-15 Thread Benjamin Marzinski
Instead of having multipath allocate its dm_info structure, it should just embed it to save on the extra allocations. This also lets us have only one function that all callers use to fill a dm_info structure. Signed-off-by: Benjamin Marzinski --- libmultipath/configure.c | 7 +++-- lib

Re: [PATCH 4/5] dax: remove the copy_from_iter and copy_to_iter methods

2021-12-15 Thread Stefan Hajnoczi
On Wed, Dec 15, 2021 at 10:43:33AM -0500, Vivek Goyal wrote: > On Wed, Dec 15, 2021 at 10:30:50AM +, Stefan Hajnoczi wrote: > > On Tue, Dec 14, 2021 at 03:32:43PM -0500, Vivek Goyal wrote: > > > On Tue, Dec 14, 2021 at 08:41:30AM -0800, Dan Williams wrote: > > > > On Tue, Dec 14, 2021 at 6:23 A

Re: [PATCH 4/5] dax: remove the copy_from_iter and copy_to_iter methods

2021-12-15 Thread Dan Williams
On Wed, Dec 15, 2021 at 7:53 AM Vivek Goyal wrote: > > On Tue, Dec 14, 2021 at 03:43:38PM -0800, Dan Williams wrote: > > On Tue, Dec 14, 2021 at 12:33 PM Vivek Goyal wrote: > > > > > > On Tue, Dec 14, 2021 at 08:41:30AM -0800, Dan Williams wrote: > > > > On Tue, Dec 14, 2021 at 6:23 AM Vivek Goya

Re: [dm-devel] [PATCH 4/5] dax: remove the copy_from_iter and copy_to_iter methods

2021-12-15 Thread Vivek Goyal
On Tue, Dec 14, 2021 at 03:43:38PM -0800, Dan Williams wrote: > On Tue, Dec 14, 2021 at 12:33 PM Vivek Goyal wrote: > > > > On Tue, Dec 14, 2021 at 08:41:30AM -0800, Dan Williams wrote: > > > On Tue, Dec 14, 2021 at 6:23 AM Vivek Goyal wrote: > > > > > > > > On Mon, Dec 13, 2021 at 09:23:18AM +01

Re: [dm-devel] [PATCH 4/5] dax: remove the copy_from_iter and copy_to_iter methods

2021-12-15 Thread Vivek Goyal
On Wed, Dec 15, 2021 at 10:30:50AM +, Stefan Hajnoczi wrote: > On Tue, Dec 14, 2021 at 03:32:43PM -0500, Vivek Goyal wrote: > > On Tue, Dec 14, 2021 at 08:41:30AM -0800, Dan Williams wrote: > > > On Tue, Dec 14, 2021 at 6:23 AM Vivek Goyal wrote: > > > > > > > > On Mon, Dec 13, 2021 at 09:23:1

Re: [PATCH 4/5] dax: remove the copy_from_iter and copy_to_iter methods

2021-12-15 Thread Stefan Hajnoczi
On Tue, Dec 14, 2021 at 03:32:43PM -0500, Vivek Goyal wrote: > On Tue, Dec 14, 2021 at 08:41:30AM -0800, Dan Williams wrote: > > On Tue, Dec 14, 2021 at 6:23 AM Vivek Goyal wrote: > > > > > > On Mon, Dec 13, 2021 at 09:23:18AM +0100, Christoph Hellwig wrote: > > > > On Sun, Dec 12, 2021 at 06:44:2

[PATCH 4/4] dax: remove the copy_from_iter and copy_to_iter methods

2021-12-15 Thread Christoph Hellwig
These methods indirect the actual DAX read/write path. In the end pmem uses magic flush and mc safe variants and fuse and dcssblk use plain ones while device mapper picks redirects to the underlying device. Add set_dax_nocache() and set_dax_nomc() APIs to control which copy routines are used to r

[PATCH 2/4] dax: simplify dax_synchronous and set_dax_synchronous

2021-12-15 Thread Christoph Hellwig
Remove the pointless wrappers. Signed-off-by: Christoph Hellwig Reviewed-by: Pankaj Gupta Reviewed-by: Dan Williams --- drivers/dax/super.c | 8 include/linux/dax.h | 12 ++-- 2 files changed, 6 insertions(+), 14 deletions(-) diff --git a/drivers/dax/super.c b/drivers/dax/su

[PATCH 3/4] dax: remove the DAXDEV_F_SYNC flag

2021-12-15 Thread Christoph Hellwig
Remove the DAXDEV_F_SYNC flag and thus the flags argument to alloc_dax and just let the drivers call set_dax_synchronous directly. Signed-off-by: Christoph Hellwig Reviewed-by: Pankaj Gupta Reviewed-by: Dan Williams --- drivers/dax/bus.c| 3 ++- drivers/dax/super.c | 6 +--

[PATCH 1/4] uio: remove copy_from_iter_flushcache() and copy_mc_to_iter()

2021-12-15 Thread Christoph Hellwig
These two wrappers are never used. Signed-off-by: Christoph Hellwig --- drivers/nvdimm/pmem.c | 4 +--- include/linux/uio.h | 20 +--- 2 files changed, 2 insertions(+), 22 deletions(-) diff --git a/drivers/nvdimm/pmem.c b/drivers/nvdimm/pmem.c index 4190c8c46ca88..d225bcfa67c

devirtualize kernel access to DAX v2

2021-12-15 Thread Christoph Hellwig
Hi Dan, this series cleans up a few loose end ends and then removes the copy_from_iter and copy_to_iter dax_operations methods in favor of straight calls. Changes since v1: - reword a 'no check' comment - clean up the flags for the copy routine variants - drop the last patch Diffstat: driver