Re: [PATCH V8 05/10] famfs_fuse: GET_DAXDEV message and daxdev_table

[John Groves <John@groves.net>]

On 26/03/21 05:33AM, kernel test robot wrote:
> Hi John,
> 
> kernel test robot noticed the following build errors:
> 
> [auto build test ERROR on mszeredi-fuse/for-next]
> [also build test ERROR on brauner-vfs/vfs.all linus/master v7.0-rc4 next-20260320]
> [If your patch is applied to the wrong git tree, kindly drop us a note.
> And when submitting patch, we suggest to use '--base' as documented in
> https://git-scm.com/docs/git-format-patch#_base_tree_information]
> 
> url:    https://github.com/intel-lab-lkp/linux/commits/John-Groves/famfs_fuse-Basic-fuse-kernel-ABI-enablement-for-famfs/20260320-152719
> base:   https://git.kernel.org/pub/scm/linux/kernel/git/mszeredi/fuse.git for-next
> patch link:    https://lore.kernel.org/r/0100019d06402f14-3a614309-878f-4837-be72-e36fd75d74f6-000000%40email.amazonses.com
> patch subject: [PATCH V8 05/10] famfs_fuse: GET_DAXDEV message and daxdev_table
> config: x86_64-rhel-9.4 (https://download.01.org/0day-ci/archive/20260321/202603210500.wr4l3DMj-lkp@intel.com/config)
> compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260321/202603210500.wr4l3DMj-lkp@intel.com/reproduce)
> 
> If you fix the issue in a separate patch/commit (i.e. not just a new version of
> the same patch/commit), kindly add following tags
> | Reported-by: kernel test robot <lkp@intel.com>
> | Closes: https://lore.kernel.org/oe-kbuild-all/202603210500.wr4l3DMj-lkp@intel.com/
> 
> All errors (new ones prefixed by >>):
> 
>    fs/fuse/famfs.c: In function 'famfs_fuse_get_daxdev':
> >> fs/fuse/famfs.c:163:32: error: implicit declaration of function 'dax_dev_get' [-Wimplicit-function-declaration]
>      163 |                 daxdev->devp = dax_dev_get(daxdev->devno);
>          |                                ^~~~~~~~~~~
> >> fs/fuse/famfs.c:163:30: error: assignment to 'struct dax_device *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
>      163 |                 daxdev->devp = dax_dev_get(daxdev->devno);
>          |                              ^

The fuse_famfs patches are dependent on the famfs dax patches (which include
the missing dax_dev_get() function. The plan that we have discussed from the 
dax side is for Ira to pull in the famfs dax patches, and then for Miklos to 
pull that branch from Ira, and base the fuse_famfs on top of that. 

I'll be pushing some minor revisions to the dax series early this week. At
that point I hope that series is ready.

Looks like the famfs series is no longer in Miklos' for-next; does this
approach work for you, Miklos?

Thanks,
John