PARTIAL TAKE 987159 - Fix extent list corruption in xfs_iext_irec_compact_full().

PARTIAL TAKE 987159 - Fix extent list corruption in xfs_iext_irec_compact_full().

[Lachlan McIlroy]

Fix extent list corruption in xfs_iext_irec_compact_full().

If we don't move all the records from the next buffer into the current buffer
then we need to update the er_extoff field of the next buffer as we shift the
remaining records to the start of the buffer.

Date:  Tue Sep 23 13:20:30 AEST 2008
Workarea:  redback.melbourne.sgi.com:/home/lachlan/isms/2.6.x-xfs
Inspected by:  
sandeen@sandeen.net
cattelan@thebarn.com
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:32165a
fs/xfs/xfs_inode.c - 1.521 - changed
http://oss.sgi.com/cgi-bin/cvsweb.cgi/xfs-linux/xfs_inode.c.diff?r1=text&tr1=1.521&r2=text&tr2=1.520&f=h
	- Fix extent list corruption in xfs_iext_irec_compact_full().

Re: PARTIAL TAKE 987159 - Fix extent list corruption in xfs_iext_irec_compact_full().

[Eric Sandeen]

Lachlan McIlroy wrote:
> Fix extent list corruption in xfs_iext_irec_compact_full().
> 
> If we don't move all the records from the next buffer into the current buffer
> then we need to update the er_extoff field of the next buffer as we shift the
> remaining records to the start of the buffer.
> 
> Date:  Tue Sep 23 13:20:30 AEST 2008
> Workarea:  redback.melbourne.sgi.com:/home/lachlan/isms/2.6.x-xfs
> Inspected by:  
> sandeen@sandeen.net
> cattelan@thebarn.com
> 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:32165a
> fs/xfs/xfs_inode.c - 1.521 - changed
http://oss.sgi.com/cgi-bin/cvsweb.cgi/xfs-linux/> xfs_inode.c.diff?r1=text&tr1=1.521&r2=text&tr2=1.520&f=h
> http://oss.sgi.com/cgi-bin/cvsweb.cgi/xfs-linux/xfs_inode.c.diff?r1=text&tr1=1.521&r2=text&tr2=1.520&f=h
> 	- Fix extent list corruption in xfs_iext_irec_compact_full().

Are there plans to try to get this, and possibly the compact_full
removal pushed into 2.6.27 still?  (and maybe even 2.6.26.x?)

-Eric

Re: PARTIAL TAKE 987159 - Fix extent list corruption in xfs_iext_irec_compact_full().

[Eric Sandeen]

Eric Sandeen wrote:
> Lachlan McIlroy wrote:


>> 	- Fix extent list corruption in xfs_iext_irec_compact_full().
> 
> Are there plans to try to get this, and possibly the compact_full
> removal pushed into 2.6.27 still?  (and maybe even 2.6.26.x?)
> 
> -Eric
> 
> 

ugh already in, sorry.  ignore me!

-Eric

Re: PARTIAL TAKE 987159 - Fix extent list corruption in xfs_iext_irec_compact_full().

[Mark Goodwin]

Eric Sandeen wrote:
> Eric Sandeen wrote:
>> Lachlan McIlroy wrote:
> 
> 
>>> 	- Fix extent list corruption in xfs_iext_irec_compact_full().
>> Are there plans to try to get this, and possibly the compact_full
>> removal pushed into 2.6.27 still?  (and maybe even 2.6.26.x?)
>>
>> -Eric
>>
>>
> 
> ugh already in, sorry.  ignore me!

yes, already in .27, but not yet .26 as far as I know.

Cheers

-- 

  Mark Goodwin                                  markgw@sgi.com
  Engineering Manager for XFS and PCP    Phone: +61-3-99631937
  SGI Australian Software Group           Cell: +61-4-18969583
-------------------------------------------------------------

Re: PARTIAL TAKE 987159 - Fix extent list corruption in xfs_iext_irec_compact_full().

[Russell Cattelan]

Mark Goodwin wrote:
>
>
> Eric Sandeen wrote:
>> Eric Sandeen wrote:
>>> Lachlan McIlroy wrote:
>>
>>
>>>>     - Fix extent list corruption in xfs_iext_irec_compact_full().
>>> Are there plans to try to get this, and possibly the compact_full
>>> removal pushed into 2.6.27 still?  (and maybe even 2.6.26.x?)
>>>
>>> -Eric
>>>
>>>
>>
>> ugh already in, sorry.  ignore me!
>
> yes, already in .27, but not yet .26 as far as I know.
>
> Cheers
>
BTW can we get xfsidbg fixed up so it compiles.
Also it would be nice to add the extra debug patch into the extent list 
print function.