xfs
[Top] [All Lists]

TAKE 983337 - fix extent corruption in xfs_iext_irec_compact_full()

To: sgi.bugs.xfs@xxxxxxxxxxxx, xfs@xxxxxxxxxxx
Subject: TAKE 983337 - fix extent corruption in xfs_iext_irec_compact_full()
From: lachlan@xxxxxxx (Lachlan McIlroy)
Date: Thu, 19 Jun 2008 14:39:03 +1000 (EST)
Sender: xfs-bounce@xxxxxxxxxxx
fix extent corruption in xfs_iext_irec_compact_full()

This function is used to compact the indirect extent list by moving
extents from one page to the previous to fill them up.  After we
move some extents to an earlier page we need to shuffle the remaining
extents to the start of the page.  The actual bug here is the second
argument to memmove() needs to index past the extents, that were copied
to the previous page, and move the remaining extents.  For pages that
are already full (ie ext_avail == 0) the compaction code has no net
effect so don't do it.


Date:  Thu Jun 19 14:38:15 AEST 2008
Workarea:  redback.melbourne.sgi.com:/home/lachlan/isms/2.6.x-agno2
Inspected by:  hch
Author:  lachlan

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


Modid:  xfs-linux-melb:xfs-kern:31332a
fs/xfs/xfs_inode.c - 1.506 - changed
http://oss.sgi.com/cgi-bin/cvsweb.cgi/xfs-linux/xfs_inode.c.diff?r1=text&tr1=1.506&r2=text&tr2=1.505&f=h
        - fix extent corruption in xfs_iext_irec_compact_full()




<Prev in Thread] Current Thread [Next in Thread>
  • TAKE 983337 - fix extent corruption in xfs_iext_irec_compact_full(), Lachlan McIlroy <=