On Wed, Jan 24, 2024 at 12:20:40PM +1100, Stephen Rothwell wrote:
> Hi all,
>
> On Tue, 23 Jan 2024 12:52:27 +1100 Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote:
> >
> > After merging the vfs-brauner tree, today's linux-next build (powerpc
> > allnoconfig) failed like this:
> >
> > In file included from <command-line>:
> > In function 'i_size_read',
> > inlined from '__iomap_dio_rw' at fs/iomap/direct-io.c:570:16:
> > include/linux/compiler_types.h:435:45: error: call to '__compiletime_assert_229' declared with attribute error: Need native word sized stores/loads for atomicity.
> > 435 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
> > | ^
> > include/linux/compiler_types.h:416:25: note: in definition of macro '__compiletime_assert'
> > 416 | prefix ## suffix(); \
> > | ^~~~~~
> > include/linux/compiler_types.h:435:9: note: in expansion of macro '_compiletime_assert'
> > 435 | _compiletime_assert(condition, msg, __compiletime_assert_, __COUNTER__)
> > | ^~~~~~~~~~~~~~~~~~~
> > include/linux/compiler_types.h:438:9: note: in expansion of macro 'compiletime_assert'
> > 438 | compiletime_assert(__native_word(t), \
> > | ^~~~~~~~~~~~~~~~~~
> > include/asm-generic/barrier.h:206:9: note: in expansion of macro 'compiletime_assert_atomic_type'
> > 206 | compiletime_assert_atomic_type(*p); \
> > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> > include/linux/fs.h:911:16: note: in expansion of macro 'smp_load_acquire'
> > 911 | return smp_load_acquire(&inode->i_size);
> > | ^~~~~~~~~~~~~~~~
> >
> > Caused by commit
> >
> > 4bbd51d0f0ad ("fs: make the i_size_read/write helpers be smp_load_acquire/store_release()")
> >
> > I have used the vfs-brauner tree from next-20240122 for today.
>
> Pending a better resolution, today I have reverted that commit and the
> following one.
I had dropped both from vfs.misc yesterday night. Maybe it didn't make
it in time.
