public inbox for linux-xfs@vger.kernel.org
 help / color / mirror / Atom feed
* TAKE 982930 - fix dir2 shortform structures on ARM old ABI
@ 2008-06-06  5:08 Tim Shimmin
  2008-06-06  5:17 ` Dave Chinner
  0 siblings, 1 reply; 2+ messages in thread
From: Tim Shimmin @ 2008-06-06  5:08 UTC (permalink / raw)
  To: xfs, sgi.bugs.xfs

Pack some shortform dir2 structures for the ARM old ABI architecture.

Finally put this in for Eric and the users of this arch.

>From Eric initially...

This should fix the longstanding issues with xfs and old ABI
arm boxes, which lead to various asserts and xfs shutdowns,
and for which an (incorrect) patch has been floating around
for years.

I've verified this patch by comparing the on-disk structure
layouts using pahole from the dwarves package, as well as
running through a bit of xfsqa under qemu-arm, modified so
that the check/repair phase after each test actually executes
check/repair from the x86 host, on the filesystem populated
by the arm emulator.  Thus far it all looks good.

There are 2 other structures with extra padding at the end,
but they don't seem to cause trouble.  I suppose they could
be packed as well: xfs_dir2_data_unused_t and xfs_dir2_sf_t.

Note that userspace needs a similar treatment, and any
filesystems which were running with the previous rogue
"fix" will now see corruption (either in the kernel, or
during xfs_repair) with this fix properly in place; it
may be worth teaching xfs_repair to identify and fix that
specific issue.

Signed-off-by: Eric Sandeen <sandeen@sandeen.net>

Date:  Fri Jun  6 15:07:06 AEST 2008
Workarea:  chook.melbourne.sgi.com:/build/tes/2.6.x-xfs-quilt
Inspected by:  sandeen@sandeen.net

The following file(s) were checked into:
  longdrop.melbourne.sgi.com:/isms/linux/2.6.x-xfs-melb


Modid:  xfs-linux-melb:xfs-kern:31280a
fs/xfs/xfs_dir2_sf.h - 1.25 - changed
http://oss.sgi.com/cgi-bin/cvsweb.cgi/xfs-linux/xfs_dir2_sf.h.diff?r1=text&tr1=1.25&r2=text&tr2=1.24&f=h
	- Fix up some dir2 shortform structures for ARM using packing

fs/xfs/linux-2.6/xfs_linux.h - 1.167 - changed
http://oss.sgi.com/cgi-bin/cvsweb.cgi/xfs-linux/linux-2.6/xfs_linux.h.diff?r1=text&tr1=1.167&r2=text&tr2=1.166&f=h
	- setup __arch_pack for packing for ARM old ABI

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-06-06  5:16 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-06-06  5:08 TAKE 982930 - fix dir2 shortform structures on ARM old ABI Tim Shimmin
2008-06-06  5:17 ` Dave Chinner

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox