Re: [PATCH v9 02/10] dax: Introduce holder for dax_device

From: Christoph Hellwig
Date: Fri Jan 21 2022 - 02:17:45 EST


On Thu, Jan 20, 2022 at 06:22:00PM -0800, Darrick J. Wong wrote:
> Hm, so that means XFS can only support dax+pmem when there aren't
> partitions in use? Ew.

Yes. Or any sensible DAX usage going forward for that matter.

>
> > > (2) extent the holder mechanism to cover a rangeo
>
> I don't think I was around for the part where "hch balked at a notifier
> call chain" -- what were the objections there, specifically? I would
> hope that pmem problems would be infrequent enough that the locking
> contention (or rcu expiration) wouldn't be an issue...?

notifiers are a nightmare untype API leading to tons of boilerplate
code. Open coding the notification is almost always a better idea.