Fwd: xfs_reno

[Hans-Peter Jansen]

Hi Dave,

I tried to gather Barrys SOB, but failed so far. His trace ends in 2009 google 
wise.

How is this case usually handled?

Here's the current state of things.

Cheers,
Pete


----------  Weitergeleitete Nachricht  ----------

Betreff: xfs_reno
Datum: Mittwoch, 6. März 2013, 12:52:19
Von: Hans-Peter Jansen <hpj at urpla.net>
An: bnaujok at sgi.com

Hi Barry,

attached is a slightly mangled version of your xfs_reno tool, that I badly 
needed recently. While at it, I plan to submit it, as it saved my *ss. Thanks.

Apart from relocation to xfsprogs, I just changed this

+       log_message(LOG_DEBUG, "%s: %llu %lu %s", msg, node->ino,
+                       node->numpaths, node->paths[0]);

from %llu to %lu for the node->numpaths argument. It might still be wrong, as 
numpath is defined as nlink_t which is a __u32 type, but the %s printed 
garbage like this:

Scanning directory tree...
xfs_reno: add_node_path: ino 8611163235, path 
/work/dlbase/hosts/11.2/pico/var/run/screens
xfs_reno: add_node_path: ino 8611163233, path 
/work/dlbase/hosts/11.2/pico/var/run/pcscd/pcscd.events
xfs_reno: add_node_path: ino 8611163234, path 
/work/dlbase/hosts/11.2/pico/var/run/uscreens
xfs_reno: nodehash: 8611163233 692488159933497345 ��]��f�e�
xfs_reno: nodehash: 8611163234 692366801337581569 ��]��f�e�
xfs_reno: nodehash: 8611163235 692223830466232321 ��]��f�e�

I guess, gcc is smart enough to see, that the struct members overlap here, and 
prints the paths[0] argument as a %llu value. What do you think?

Anyway, I will revise this during the course of creating a xlstests test for 
xfs_reno...

Do you allow me to add your Signed-off-by to this patch?

If you want to build this, apply both patches to xfsprogs.

TIA,
Pete

-------------------------------------------------------------
-------------- next part --------------
A non-text attachment was scrubbed...
Name: xfsprogs-xfs_reno.diff
Type: text/x-patch
Size: 50202 bytes
Desc: not available
URL: <http://oss.sgi.com/pipermail/xfs/attachments/20130306/429137a2/attachment-0002.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: xfsprogs-libattr-ac.diff
Type: text/x-patch
Size: 4142 bytes
Desc: not available
URL: <http://oss.sgi.com/pipermail/xfs/attachments/20130306/429137a2/attachment-0003.diff>