Re: xfs_efi_item slab corruption. (v3.9-10936-g51a26ae)

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

 



On 05/07/13 17:22, Dave Chinner wrote:
On Tue, May 07, 2013 at 03:24:28PM -0500, Mark Tinguely wrote:
On 05/07/13 15:22, Dave Jones wrote:
On Tue, May 07, 2013 at 03:04:33PM -0500, Mark Tinguely wrote:
  >   On 05/07/13 14:59, Dave Jones wrote:
  >   >   On Tue, May 07, 2013 at 02:58:15PM -0500, Mark Tinguely wrote:
  >   >
  >   >     >    >    I can hit this almost instantly with fsx. I'll do a bisect, though
  >   >     >    >    it sounds like you already have a suspect.
  >   >     >    >
  >   >     >
  >   >     >    If you want to try kmem debug of Linux 3.8 that would help.
  >   >
  >   >   I'm not sure what that is.
  >
  >   Sorry, if you would test Linux 3.8 with "CONFIG_DEBUG_SLAB=y".

Ah, done that. (I pretty much always run with it).

This is something new. Even 3.9 was fine. It's only since
the recent xfs merge.

	Dave


git revert 666d644cd72a9ec58b353209ff191d7430f3b357

That won't prevent the use after free. That commit fixed a problem
that could lead to a use after free, but what we are seeing here is
that it has ultimately exposed a previously unknown issue that
causes the use after free.

Basically what is happening is that there are two commits for the
EFD being processed, when there should only be one. I'm not sure how
this is happening yet, but these three traces came out from my debug
sequentially when running generic/006:


Sorry for the misleading statement. Yes, I agree that patch is a good thing. I meant that Dave and only Dave revert it and only to test if that patch was the change that caused the new symptom - which we know now that it is.

I added some asserts and did not learn anything new except where the efi item was already freed.

--Mark.

_______________________________________________
xfs mailing list
xfs@xxxxxxxxxxx
http://oss.sgi.com/mailman/listinfo/xfs




[Index of Archives]     [Linux XFS Devel]     [Linux Filesystem Development]     [Filesystem Testing]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux