Re: [PATCH 4/4] dax: Cleanup extra dax_region references

2023-06-15 Thread Dave Jiang
On 6/2/23 23:14, Dan Williams wrote: Now that free_dev_dax_id() internally manages the references it needs the extra references taken by the dax_region drivers are not needed. Reported-by: Ira Weiny Signed-off-by: Dan Williams Reviewed-by: Dave Jiang --- drivers/dax/bus.c |

Re: [PATCH 4/4] dax: Cleanup extra dax_region references

2023-06-06 Thread Ira Weiny
Fan Ni wrote: > The 06/02/2023 23:14, Dan Williams wrote: > > Now that free_dev_dax_id() internally manages the references it needs > > the extra references taken by the dax_region drivers are not needed. > > > > Reported-by: Ira Weiny > > Signed-off-by: Dan Williams > > Reviewed-by: Fan Ni >

Re: [PATCH 4/4] dax: Cleanup extra dax_region references

2023-06-06 Thread Fan Ni
The 06/02/2023 23:14, Dan Williams wrote: > Now that free_dev_dax_id() internally manages the references it needs > the extra references taken by the dax_region drivers are not needed. > > Reported-by: Ira Weiny > Signed-off-by: Dan Williams Reviewed-by: Fan Ni One minor comment as below. >

Re: [PATCH 4/4] dax: Cleanup extra dax_region references

2023-06-03 Thread Ira Weiny
Dan Williams wrote: > Now that free_dev_dax_id() internally manages the references it needs > the extra references taken by the dax_region drivers are not needed. > > Reported-by: Ira Weiny Reviewed-by: Ira Weiny > Signed-off-by: Dan Williams > --- > drivers/dax/bus.c |4 +--- >

[PATCH 4/4] dax: Cleanup extra dax_region references

2023-06-03 Thread Dan Williams
Now that free_dev_dax_id() internally manages the references it needs the extra references taken by the dax_region drivers are not needed. Reported-by: Ira Weiny Signed-off-by: Dan Williams --- drivers/dax/bus.c |4 +--- drivers/dax/bus.h |1 - drivers/dax/cxl.c |