On Wed, May 22, 2013 at 12:15:21AM -0400, Dave Jones wrote:
> On Wed, May 22, 2013 at 02:03:18PM +1000, Dave Chinner wrote:
> > That doesn't make a whole lot of sense to me. What am I missing?
> > Are you seeing this fire at all from notify_change()?
> > WARN_ON_ONCE(!mutex_is_locked(&inode->i_mutex));
> > <Light Bulb>
> > What's wrong with this code in do_truncate()?
> > /* Remove suid/sgid on truncate too */
> > ret = should_remove_suid(dentry);
> > if (ret)
> > newattrs.ia_valid |= ret | ATTR_FORCE;
> > mutex_lock(&dentry->d_inode->i_mutex);
> > ret = notify_change(dentry, &newattrs);
> > mutex_unlock(&dentry->d_inode->i_mutex);
> > Patch below to fix this.
> > However, it probably doesn't fix the fact that truncate can change
> > the size and kill suid/sgid bits at the same time and XFS doesn't
> > appear to handle that sanely right now. Can you run the patch below
> > just so when it fails we can see that the mask is actually sane?
> [ 36.339105] XFS (sda2): xfs_setattr_size: mask 0xa068 mismatch on file
So, still the same strange mask. That just doesn't seem right.
> [ 36.350823] XFS: Assertion failed: 0, file: fs/xfs/xfs_iops.c, line: 730
> [ 36.359459] ------------[ cut here ]------------
> [ 36.365247] kernel BUG at fs/xfs/xfs_message.c:108!
> [ 36.371360] invalid opcode: 0000 [#1] PREEMPT SMP DEBUG_PAGEALLOC
> [ 36.379091] Modules linked in: xfs libcrc32c snd_hda_codec_realtek
> snd_hda_codec_hdmi microcode(+) pcspkr snd_hda_intel snd_hda_codec snd_hwdep
> snd_seq snd_seq_device snd_pcm e1000e snd_page_alloc snd_timer ptp snd
> soundcore pps_core
> [ 36.405431] CPU: 1 PID: 2887 Comm: cc1 Not tainted 3.10.0-rc2+ #4
Your compiler is triggering this? That doesn't seem likely...
> [ 36.578436] Call Trace:
> [ 36.581514] [<ffffffffa01bc3ef>] xfs_setattr_size+0x48f/0x630 [xfs]
> [ 36.738533] [<ffffffffa01bc5c6>] xfs_vn_setattr+0x36/0x40 [xfs]
> [ 36.746047] [<ffffffff811c8e2c>] notify_change+0x1dc/0x360
> [ 36.753024] [<ffffffff811a9d9d>] do_truncate+0x6d/0xa0
> [ 36.759574] [<ffffffffa01ae0a0>] ? xfs_extent_busy_ag_cmp+0x20/0x20 [xfs]
> [ 36.768182] [<ffffffff811bb4af>] do_last+0x54f/0xe40
> [ 36.775319] [<ffffffff811bbe53>] path_openat+0xb3/0x530
> [ 36.782780] [<ffffffff810b3951>] ?
> [ 36.792408] [<ffffffff811bc958>] do_filp_open+0x38/0x80
> [ 36.799870] [<ffffffff816ea961>] ? _raw_spin_unlock+0x31/0x60
> [ 36.807981] [<ffffffff811cb49f>] ? __alloc_fd+0xaf/0x200
> [ 36.815544] [<ffffffff811aae19>] do_sys_open+0xe9/0x1c0
> [ 36.822989] [<ffffffff811aaf0e>] SyS_open+0x1e/0x20
This has come through the open path via handle_truncate(), which
means that ATTR_MTIME|ATTR_CTIME|ATTR_OPEN|ATTR_FILE should also be
set in the mask. They aren't, and that says to me that something
else has been blottoed before XFS trips over this. Memory
Can you print out the entire struct iattr? perhaps even hexdump it?