Re: [PATCH 3/6] xfs: simplify xfs_dax_fault

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jun 20, 2024 at 11:50:26AM -0700, Darrick J. Wong wrote:
> > +	if (!IS_ENABLED(CONFIG_FS_DAX)) {
> > +		ASSERT(0);
> > +		return VM_FAULT_SIGBUS;
> 
> Does this actually work if FS_DAX=n?  AFAICT there's no !DAX stub for
> dax_iomap_fault, so won't that cause a linker error?

IS_ENABLED expands to a compile time constant, so the compiler eliminates
the call and no stub is needed.





[Index of Archives]     [XFS Filesystem Development (older mail)]     [Linux Filesystem Development]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux RAID]     [Linux SCSI]


  Powered by Linux