Whamcloud - gitweb
class_obd.c: plugged a few minor memory leaks
[fs/lustre-release.git] / lustre / obdclass / obdcontrol
index 8b1ddaa..0e11f78 100755 (executable)
@@ -61,6 +61,8 @@ eval 'sub OBD_IOC_COPY () { &_IOC(3, ord(\'f\'), 22, 4);}' unless
   defined(&OBD_IOC_COPY);
 eval 'sub OBD_IOC_MIGR () { &_IOC(3, ord(\'f\'), 23, 4);}' unless
   defined(&OBD_IOC_MIGR);
+eval 'sub OBD_IOC_PUNCH () { &_IOC(3, ord(\'f\'), 24, 4);}' unless
+  defined(&OBD_IOC_PUNCH);
 eval 'sub OBD_SNAP_SETTABLE () { &_IOC(3, ord(\'f\'), 40, 4);}' unless
   defined(&OBD_SNAP_SETTABLE);
 eval 'sub OBD_SNAP_PRINTTABLE () { &_IOC(3, ord(\'f\'), 41, 4);}' unless
@@ -73,13 +75,14 @@ eval 'sub OBD_SNAP_RESTORE() { &_IOC(3, ord(\'f\'), 43, 4);}' unless
 eval 'sub OBD_EXT2_RUNIT () { &_IOC(3, ord(\'f\'), 61, 4);}' unless
   defined(&OBD_EXT2_RUNIT);
 
-eval 'sub ATTR_MODE () {1;}' unless defined(&ATTR_MODE);
-eval 'sub ATTR_UID () {2;}' unless defined(&ATTR_UID);
-eval 'sub ATTR_GID () {4;}' unless defined(&ATTR_GID);
-eval 'sub ATTR_SIZE () {8;}' unless defined(&ATTR_SIZE);
-eval 'sub ATTR_ATIME () {16;}' unless defined(&ATTR_ATIME);
-eval 'sub ATTR_MTIME () {32;}' unless defined(&ATTR_MTIME);
-eval 'sub ATTR_CTIME () {64;}' unless defined(&ATTR_CTIME);
+eval 'sub OBD_MD_FLALL   () {~0;}'   unless defined(&OBD_MD_FLALL);
+eval 'sub OBD_MD_FLATIME () {1<<1;}' unless defined(&OBD_MD_FLATIME);
+eval 'sub OBD_MD_FLMTIME () {1<<2;}' unless defined(&OBD_MD_FLMTIME);
+eval 'sub OBD_MD_FLCTIME () {1<<3;}' unless defined(&OBD_MD_FLCTIME);
+eval 'sub OBD_MD_FLSIZE  () {1<<4;}' unless defined(&OBD_MD_FLSIZE);
+eval 'sub OBD_MD_FLMODE  () {1<<7;}' unless defined(&OBD_MD_FLMODE);
+eval 'sub OBD_MD_FLUID   () {1<<8;}' unless defined(&OBD_MD_FLUID);
+eval 'sub OBD_MD_FLGID   () {1<<9;}' unless defined(&OBD_MD_FLGID);
 
 use Getopt::Long;
 use File::stat;
@@ -89,6 +92,112 @@ use Term::ReadLine;
 use IO::Handle;
 
 
+# NOTE long long are layed out in ia32 memory as follows:
+# u = 0xaaaabbbbccccdddd has ccccdddd at &u and aaaabbbb 4 bytes on
+# this may be different on other architectures
+
+# we use 32-bit integers for all 64-bit quantities in this program
+# #define OBD_INLINESZ 60
+# #define OBD_OBDMDSZ  60
+# /* Note: 64-bit types are 64-bit aligned in structure */
+# struct obdo {
+#      obd_id                  o_id;
+#      obd_gr                  o_gr;
+#      obd_time                o_atime;
+#      obd_time                o_mtime;
+#      obd_time                o_ctime;
+#      obd_size                o_size;
+#      obd_blocks              o_blocks;
+#      obd_blksize             o_blksize;
+#      obd_mode                o_mode;
+#      obd_uid                 o_uid;
+#      obd_gid                 o_gid;
+#      obd_flag                o_flags;
+#      obd_flag                o_obdflags;
+#      obd_count               o_nlink;
+#      obd_count               o_generation;
+#      obd_flag                o_valid;        /* hot fields in this obdo */
+#      char                    o_inline[60];
+#      char                    o_obdmd[60];
+#      struct list_head        o_list;
+#      struct obd_ops          *o_op;
+# };
+
+sub obdo_pack {
+    my $obdo = shift;
+    pack "LL LL LL LL LL LL LL L L L L L L L L L a60 a60 L L L", 
+    $obdo->{id}, 0, 
+    $obdo->{gr}, 0, 
+    $obdo->{atime}, 0, 
+    $obdo->{mtime}, 0 ,
+    $obdo->{ctime}, 0, 
+    $obdo->{size}, 0, 
+    $obdo->{blocks}, 0, 
+    $obdo->{blksize},
+    $obdo->{mode},
+    $obdo->{uid},
+    $obdo->{gid},
+    $obdo->{flags},
+    $obdo->{obdflags},
+    $obdo->{nlink},    
+    $obdo->{generation},       
+    $obdo->{valid},    
+    $obdo->{inline},
+    $obdo->{obdmd},
+    0, 0, # struct list_head 
+    0;  #  struct obd_ops 
+}
+
+sub obdo_unpack {
+    my $buf = shift;
+    my $offset = shift;
+    my $obdo;
+    ($obdo->{id},
+    $obdo->{gr},
+    $obdo->{atime},
+    $obdo->{mtime},
+    $obdo->{ctime},
+    $obdo->{size},
+    $obdo->{blocks},
+    $obdo->{blksize},
+    $obdo->{mode},
+    $obdo->{uid},
+    $obdo->{gid},
+    $obdo->{flags},
+    $obdo->{obdflags},
+    $obdo->{nlink},
+    $obdo->{generation},
+    $obdo->{valid},
+    $obdo->{inline},
+    $obdo->{obdmd}) = unpack "x${offset}Lx4 Lx4 Lx4 Lx4 Lx4 Lx4 Lx4 L L L L L L L L L a60 a60", $buf;
+    $obdo;
+}
+
+sub obdo_print {
+
+    my $obdo = shift;
+
+    printf "id: %d\ngrp: %d\natime: %s\nmtime: %s\nctime: %s\nsize: %d\nblocks: %d\nblksize: %d\nmode: %o\nuid: %d\ngid: %d\nflags: %x\nobdflags: %x\nnlink: %d\nvalid: %x\ninline: %s\nobdmd: %s\n",
+    $obdo->{id},
+    $obdo->{gr},
+    $obdo->{atime},
+    $obdo->{mtime},
+    $obdo->{ctime},
+    $obdo->{size},
+    $obdo->{blocks},
+    $obdo->{blksize},
+    $obdo->{mode},
+    $obdo->{uid},
+    $obdo->{gid},
+    $obdo->{flags},
+    $obdo->{obdflags},
+    $obdo->{nlink},
+    $obdo->{valid},
+    $obdo->{inline},
+    $obdo->{obdmd};
+}
+
+
 my ($file);
 
 GetOptions("f!" => \$file, "device=s" => \$::device, ) || die "Getoptions";
@@ -102,9 +211,9 @@ my $arg;
 
 my %commands =
     ('device' => {func => "Device", doc => "device <dev>: open another OBD device"},
-     'create' => {func => "Create", doc => "create: creates a new inode"},
-     'attach' => {func => "Attach", doc => "attach {ext2_obd | snap_obd snapdev snapidx tableno | scsi_obd adapter bus tid lun}"}, #
-     'detach' => {func => "Detach", doc => "detach this device"},
+     'create' => {func => "Create", doc => "create [<num> [<mode> [quiet]]]: create new object(s) (files, unless mode is given)"},
+     'attach' => {func => "Attach", doc => "attach { obdext2 | obdsnap snapdev snapidx tableno | obdscsi adapter bus tid lun }: attach this minor device to the specified driver" },
+     'detach' => {func => "Detach", doc => "detach this minor device"},
      'testext2iterator' => {func => "TestExt2Iterator", doc => "test ext2 iterator function"},
      'snapset' => {func => "SnapSetTable", doc => "snapset <tableno> <file>: set the table (created with snaptable) as table #tableno" },
      'snapprint' => {func => "SnapPrint", doc => "snapprint <tableno>: output the contents of table #tableno to the syslog"},
@@ -113,21 +222,22 @@ my %commands =
      'snaptable' => {func => "SnapShotTable", doc => "snaptable: build a snapshot table (interactive)"},
      'copy' => {func => "Copy", doc => "copy <srcid> <tgtid>: copy objects"},
      'migrate' => {func => "Migrate", doc => "migrate <srcid> <tgtid>: migrate data from one object to another"},
-     'format' => {func => "Format", doc => "format type adapter bus tid lun size"},
-     'partition' => {func => "Partition", doc => "partition type adapter bus tid lun partition size"},
-     'setup' => {func => "Setup", doc => "setup [type]: link this OBD device to the underlying device (default type ext2_obd)"},
+     'partition' => {func => "Partition", doc => "partition <type> <adapter> <bus> <tid> <lun> <partition> <size>: create a partition"},
+     'format' => {func => "Format", doc => "format <type> <adapter> <bus> <tid> <lun> <size>: format a partition"},
+     'setup' => {func => "Setup", doc => "setup [type]: link this OBD device to the underlying device (default type obdext2)"},
      'connect' => {func => "Connect", doc => "connect: allocates client ID for this session"},
      'disconnect' => {func => "Disconnect", doc => "disconnect [id]: frees client resources"},
      'sync' => {func => "Sync", doc => "sync: flushes buffers to disk"},
-     'destroy' => {func => "Destroy", doc => "destroy <inode>: destroys an inode"},
+     'destroy' => {func => "Destroy", doc => "destroy <id>: destroys an object"},
      'cleanup' => {func => "Cleanup", doc => "cleanup the minor obd device"},
      'dec_use_count' => {func => "Decusecount", doc => "decreases the module use count so that the module can be removed following an oops"},
-     'read' => {func => "Read", doc => "read <inode> <count> [offset]"},
-     'fsread' => {func => "Read2", doc => "read <inode> <count> [offset]"},
-     'write' => {func => "Write", doc => "write <inode> <offset> <text>"},
-     'setattr' => {func => "Setattr", doc => "setattr <inode> [mode [uid [gid [size [atime [mtime [ctime]]]]]]]"},
-     'getattr' => {func => "Getattr", doc => "getattr <inode>: displays inode object attributes"},
-     'preallocate' => {func => "Preallocate", doc => "preallocate [num]: requests preallocation of num inodes."},
+     'read' => {func => "Read", doc => "read <id> <count> [offset]: read data from object"},
+     'fsread' => {func => "Read2", doc => "read <id> <count> [offset]: read data from object"},
+     'write' => {func => "Write", doc => "write <id> <offset> <text>: write data to object"},
+     'punch' => {func => "Punch", doc => "punch <id> <start> <count>: punch a hole in object"},
+     'setattr' => {func => "Setattr", doc => "setattr <id> [mode [uid [gid [size [atime [mtime [ctime]]]]]]]: sets object attributes"},
+     'getattr' => {func => "Getattr", doc => "getattr <id>: displays object attributes"},
+     'preallocate' => {func => "Preallocate", doc => "preallocate [num]: requests preallocation of num objects."},
      'statfs' => {func => "Statfs", doc => "statfs: filesystem status information"},
      'help' => {func => \&Help,  doc => "help: this message"},
      'quit' => {func => \&Quit,  doc => "see \"exit\""},
@@ -210,8 +320,8 @@ sub process_line {
 sub execute_line {
     my $line = shift;
 
-    my @arg = split(' ', $line);
-    my $word = shift @arg;
+    my @cmdline = split(' ', $line);
+    my $word = shift @cmdline;
 
     my $cmd;
     if ( $file ) {
@@ -225,11 +335,11 @@ sub execute_line {
     }
 
     if ($cmd eq "help" || $cmd eq "exit" || $cmd eq "quit") {
-       return (&{$commands{$cmd}->{func}}(@arg));
+       return (&{$commands{$cmd}->{func}}(@cmdline));
     }
 
     # Call the function.
-    return (&{$commands{$cmd}->{func}}(@arg));
+    return (&{$commands{$cmd}->{func}}(@cmdline));
 }
 
 
@@ -251,7 +361,6 @@ sub Device {
 }
 
 
-
 sub Attach {
     my $err = 0;
     my $type = shift;
@@ -261,11 +370,11 @@ sub Attach {
     if ( ! $type ) {
        print "error: missing type\n";
 usage:
-       print "usage: attach {ext2_obd | snap_obd}\n"; # XXX add scsi_obd later
+       print "usage: attach {obdext2 | obdsnap | obdscsi}\n";
        return;
     }
 
-    if ($type eq "scsi_obd" ) {
+    if ($type eq "obdscsi" ) {
        my $adapter = shift;
        my $bus = shift;
        my $tid = shift;
@@ -273,20 +382,16 @@ usage:
 
        $data = pack("iiii", $adapter, $bus, $tid, $lun);
        $datalen = 4 * 4;
-    } elsif ($type eq "snap_obd" ) {
+    } elsif ($type eq "obdsnap" ) {
        my $snapdev = shift;
        my $snapidx = shift;
        my $tableno = shift;
 
        $data = pack("iii", $snapdev, $snapidx, $tableno);
        $datalen = 3 * 4;
-    } elsif ($type eq "ext2_obd") {
-       my $basedev = shift;
-       # $basedev = "/dev/loop0" unless $basedev;
-
-       if (!defined($::st = stat($basedev))) {
-           die "Unable to stat $basedev.\n";
-       }
+    } elsif ($type eq "obdext2") {
+       $data = pack("i", 4711);   # bogus data
+       $datalen = 0;
     } else {
        print "error: unknown attach type $type\n";
        goto usage;
@@ -309,6 +414,7 @@ usage:
     }
 }
 
+
 sub Detach {
     my $err = 0;
     my $data = "";
@@ -331,14 +437,13 @@ sub TestExt2Iterator {
     }
 
     my $err = 0;
-    my $type = "ext2_obd";
+    my $type = "obdext2";
  
     $data = pack("i", 4711); # bogus data
     $datalen = 4;
 
     my $len = length($type);
     my $cl = length($data);
-    my $add = pack("p", $data);
     print "type $type (len $len), datalen $datalen ($cl)\n";
     my $packed = pack("Lipip", $::client_id, length($type), $type, $datalen, $data);
 
@@ -361,14 +466,13 @@ sub SnapDelete {
     }
 
     my $err = 0;
-    my $type = "snap_obd";
+    my $type = "obdsnap";
  
     $data = pack("i", 4711); # bogus data
     $datalen = 4;
 
     my $len = length($type);
     my $cl = length($data);
-    my $add = pack("p", $data);
     print "type $type (len $len), datalen $datalen ($cl)\n";
     my $packed = pack("Lipip", $::client_id, length($type), $type, $datalen, $data);
 
@@ -386,27 +490,63 @@ sub SnapDelete {
 }
 
 
-# XXX the parameter to snaprestore SHOULD be the snapshot number of the
-#     snapshot to be restored, and this routine should handle all of the
-#     removal of intervening snapshots and changing the snaptable to have
-#     the "restored" snapshot as current and move the previous current.
+#      this routine does the whole job
 sub SnapRestore { 
+    my $restoreto = shift;
+    my $snaptable = shift;
+    my $tableno = shift;
+    my $restoretime;
+
+    # don't do anything until connected
     if (!defined($::client_id)) {
        print "You must first ``connect''.\n";
        return;
     }
 
+    if ( ! $snaptable || ! defined $restoreto ) {
+       print "Usage: snaprestore \"restore to slot\" \"snaptable\" \"tableno\"\n";
+       return;
+    }
+
+    if ( ! -f $snaptable ) {
+       print "Table $snaptable doesn't exist\n";
+       return;
+    }
+   
+    my $table = ReadSnapShotTable($snaptable);
+    $restoretime = FindSnapInTable($table, $restoreto);
+    if ( ! defined $table->{0} || ! defined $restoretime ) {
+       PrintSnapShotTable($table);
+       print "No current or $restoreto slot in this table\n";
+       return;
+    }
+
+    my $currentindex = $table->{0};
+    if (  $table->{$restoretime} == $currentindex ) {
+       print "You should not restore to the current snapshot\n";
+       return;
+    }
+    
+    # swap the entries for 0 and $restoreto
+    my $tmp = $table->{$restoretime};
+    $table->{$restoretime} = $table->{0};
+    $table->{0} = $tmp;
+    # PrintSnapShotTable($table);
+
+    # write it back
+    WriteSnapShotTable($snaptable, $table);
+
+    # set it in the kernel
+    SnapSetTable($tableno, $snaptable);
+
+    # ready for the ioctl
     my $err = 0;
-    my $type = "snap_obd";
-    my $prevcurrent = shift;
-    $data = pack("i", $prevcurrent); # where previous current snapshot now lives
+    my $type = "obdsnap";
+    $data = pack("i", $currentindex); # slot of previous current snapshot 
     $datalen = 4;
 
     my $len = length($type);
     my $cl = length($data);
-    my $add = pack("p", $data);
-    print "type $type (len $len), datalen $datalen ($cl)\n";
     my $packed = pack("Lipip", $::client_id, length($type), $type, $datalen, $data);
 
     my $rc = ioctl(DEV_OBD, &OBD_SNAP_RESTORE, $packed);
@@ -414,17 +554,37 @@ sub SnapRestore {
     if (!defined $rc) {
        print STDERR "ioctl failed: $!\n";
     } elsif ($rc eq "0 but true") {
-       print "Finished (success)\n";
+       print "Snaprestore finished (success)\n";
+       delete $table->{$restoretime} if defined $restoretime;
+       # write it back
+       WriteSnapShotTable($snaptable, $table);
+       
+       # set it in the kernel
+       SnapSetTable($tableno, $snaptable);
+       # PrintSnapShotTable($table);
+
     } else {
        print "ioctl returned error code $rc.\n";
     }
 }
 
+sub FindSnapInTable { 
+    my $table = shift;
+    my $snapno =shift;
 
+    foreach my $restoretime ( keys %{$table} ) {
+       if ( $table->{$restoretime} == $snapno) { 
+           print "Found key $restoretime for snapno $snapno\n";
+           return $restoretime;
+       }
+    }
+    undef;
+}
+           
 
 sub SnapPrint { 
     my $err = 0;
-    my $type = "snap_obd";
+    my $type = "obdsnap";
     my $snaptableno = shift;
 
     $data = pack("i", $snaptableno);
@@ -432,7 +592,6 @@ sub SnapPrint {
 
     my $len = length($type);
     my $cl = length($data);
-    my $add = pack("p", $data);
     print "type $type (len $len), datalen $datalen ($cl)\n";
     my $packed = pack("Lipip", $::client_id, length($type), $type, $datalen, $data);
 
@@ -449,7 +608,7 @@ sub SnapPrint {
 
 sub SnapSetTable {
     my $err = 0;
-    my $type = "snap_obd";
+    my $type = "obdsnap";
     my $snaptableno = shift;
     my $file = shift;
     my $snapcount;
@@ -473,13 +632,13 @@ sub SnapSetTable {
     $data = pack("ii", $snaptableno, $snapcount);
     $datalen = 2 * 4;
     foreach my $time (sort keys %{$table}) {
+       # XXX we should change to pack LL instead of I for times
        $data .= pack("Ii", $time, $table->{$time});
        $datalen += 8;
     }
 
     my $len = length($type);
     my $cl = length($data);
-    my $add = pack("p", $data);
     print "type $type (len $len), datalen $datalen ($cl)\n";
     my $packed = pack("Lipip", $::client_id, length($type), $type, $datalen, $data);
 
@@ -596,12 +755,18 @@ sub WriteSnapShotTable {
 
 sub Copy {
     my $err = 0;
-    my $srcid = shift;
-    my $tgtid = shift;
-    my $data = pack("III", $::client_id, $srcid, $tgtid);
-    my $datalen = 12;
+    my $src_obdo;
+    my $dst_obdo;
+
+    # Note: _copy IOCTL takes parameters as dst, src.
+    #       Copy function takes parameters as src, dst.
+    $src_obdo->{id} = shift;
+    $dst_obdo->{id} = shift;
+    $src_obdo->{valid} = &OBD_MD_FLALL;
+
+    # XXX need to fix copy so we can have 2 client IDs here
+    my $packed = pack("L", $::client_id) . obdo_pack($dst_obdo) . pack("L", $::client_id) . obdo_pack($src_obdo);
 
-    my $packed = pack("ip", $datalen, $data);
     my $rc = ioctl(DEV_OBD, &OBD_IOC_COPY, $packed);
 
     if (!defined $rc) {
@@ -615,12 +780,16 @@ sub Copy {
 
 sub Migrate {
     my $err = 0;
-    my $srcid = shift;
-    my $tgtid = shift;
-    my $data = pack("III", $::client_id, $srcid, $tgtid);
-    my $datalen = 12;
 
-    my $packed = pack("ip", $datalen, $data);
+    # Note: _migr IOCTL takes parameters as dst, src.
+    #       Migrate function takes parameters as src, dst.
+    $src_obdo->{id} = shift;
+    $dst_obdo->{id} = shift;
+    $src_obdo->{valid} = &OBD_MD_FLALL;
+
+    # We pack a dummy connection ID here
+    my $packed = pack("L", $::client_id) . obdo_pack($dst_obdo) . pack("L", $::client_id) . obdo_pack($src_obdo);
+
     my $rc = ioctl(DEV_OBD, &OBD_IOC_MIGR, $packed);
 
     if (!defined $rc) {
@@ -672,14 +841,19 @@ sub Partition {
 
 sub Setup {
     my $err = 0;
-    my $type = shift;
+    my $arg = shift;
     my $data;
     my $datalen = 0;
-    
-    $type = "ext2_obd" unless $type;
 
-    if ( $type eq "ext2_obd" ) {
-       my $dev = shift;
+    # XXX we need a getinfo ioctl to validate parameters 
+    # by type here
+
+    if ($arg && !defined($::st = stat($arg))) {
+           print "$dev is not a valid device\n";
+           return;
+    }
+    
+    if ( $arg ) {
        $dev = $::st->rdev() unless $dev;
        $data = pack("i", $dev);
        $datalen = 4;
@@ -758,38 +932,52 @@ sub Disconnect {
 }
 
 sub Create {
-    my $arg = shift;
+    if (!defined($::client_id)) {
+       print "You must first ``connect''.\n";
+       return;
+    }
+
+    my $num = shift;
+    my $mode = shift;
     my $quiet = shift;
     my $rc;
     my $prealloc = 0;
 
-    if (defined($quiet) && $quiet ne "quiet") {
-       print "syntax: create [number of objects [quiet]]\n";
+    if (!defined($num)) {
+        $num = 1;
+    }
+
+    if (!defined($mode)) {
+        $mode = 0100644;        # create a file (rw-r--r--) if not specified
+    }
+
+    if (scalar($num) < 1 || defined($quiet) && $quiet ne "quiet") {
+       print "usage: create [<number of objects> [<mode> [quiet]]]\n";
        return;
     }
 
-    my $packed = pack("IL", $::client_id, $prealloc);
-    if (!defined($arg) || scalar($arg) < 2) {
-       print "Creating 1 object...\n";
-       $rc = ioctl(DEV_OBD, &OBD_IOC_CREATE, $packed);
-       if (!defined($quiet)) {
-           my $ino = unpack("L", $packed);
-           print "Created object #$ino.\n";
-       }
-    } else {
-       my $i;
+    my $i;
+    my $id = 0;                        # can't currently request IDs
+
+    print "Creating " . scalar($num) . " object";
+    if (scalar($num) > 1) {
+       print "s";
+    }
+    print "\n";
+
+    for ($i = 0; $i < scalar($num); $i++) {
+       my $obdo;
+       $obdo->{id} = $id;
+       $obdo->{mode} = scalar($mode);
+       $obdo->{valid} = &OBD_MD_FLMODE;
 
-       print "Creating " . scalar($arg) . " objects...\n";
-       for ($i = 0; $i < scalar($arg); $i++) {
-           $rc = ioctl(DEV_OBD, &OBD_IOC_CREATE, $packed);
-           my $ino = unpack("L", $packed);
-           if ($rc ne "0 but true") {
-               last;
-               $packed = pack("IL", $::client_id, $prealloc);
-           } elsif (!defined($quiet)) {
-               $packed = pack("IL", $::client_id, $prealloc);
-               print "Created object #$ino.\n";
-           }
+       my $packed = pack("I", $::client_id) . obdo_pack($obdo);
+       $rc = ioctl(DEV_OBD, &OBD_IOC_CREATE, $packed);
+       if ($rc ne "0 but true") {
+           last;
+       } elsif (!defined($quiet)) {
+           $obdo = obdo_unpack($packed, 4);
+           print "Created object #$obdo->{id}.\n";
        }
     }
 
@@ -821,15 +1009,15 @@ sub Destroy {
        return;
     }
 
-    my $arg = shift;
+    my $id = shift;
 
-    if (!defined($arg) || scalar($arg) < 1) {
-       print "destroy requires the object number to destroy.\n";
+    if (!defined($id) || scalar($id) < 1) {
+       print "usage: destroy <object number>\n";
        return;
     }
 
-    print "Destroying object $arg...\n";
-    my $packed = pack("IL", $::client_id, $arg);
+    print "Destroying object $id...\n";
+    my $packed = pack("IL", $::client_id, $id);
     my $rc = ioctl(DEV_OBD, &OBD_IOC_DESTROY, $packed);
 
     if (!defined $rc) {
@@ -847,32 +1035,25 @@ sub Getattr {
        return;
     }
 
-    my $inode = shift;
+    my $id = shift;
 
-    if (!defined($inode) || scalar($inode) < 1) {
+    if (!defined($id) || scalar($id) < 1) {
        print "invalid arguments; type \"help getattr\" for a synopsis\n";
        return;
     }
 
     # see Setattr
-    my $packed = pack("ILsx2lLLLI", $::client_id, $inode, 0, 0, 0, 0, 0, 0, 0,
-                     0);
+    my $obdo;
+    $obdo->{id} = $id;
+    $obdo->{valid} = &OBD_MD_FLALL;
+    my $packed = pack("L", $::client_id) . obdo_pack($obdo);
     my $rc = ioctl(DEV_OBD, &OBD_IOC_GETATTR, $packed);
-
+    
     if (!defined $rc) {
        print STDERR "ioctl failed: $!\n";
     } elsif ($rc eq "0 but true") {
-       my ($valid, $mode, $uid, $gid, $size, $atime, $mtime, $ctime, $flags);
-       ($valid, $mode, $uid, $gid, $size, $atime, $mtime, $ctime, $flags) =
-         unpack("ISssx2lLLLI", $packed);
-
-       printf("Inode: %d  Mode:  %o\n", $inode, $mode);
-       printf("User: %6d   Group: %6d   Size: %d\n", $uid, $gid, $size);
-       printf("ctime: %08lx -- %s\n", $ctime, scalar(gmtime($ctime)));
-       printf("atime: %08lx -- %s\n", $atime, scalar(gmtime($atime)));
-       printf("mtime: %08lx -- %s\n", $mtime, scalar(gmtime($mtime)));
-       printf("flags: %08x\n", $flags);
-       print "Finished (success)\n";
+       $obdo = obdo_unpack($packed,  4); 
+       obdo_print($obdo);
     } else {
        print "ioctl returned error code $rc.\n";
     }
@@ -884,59 +1065,49 @@ sub Setattr {
        return;
     }
 
-    my $inode = shift;
-    my $valid = 0;
-    my $mode = oct(shift);
-    my $uid = shift;
-    my $gid = shift;
-    my $size = shift;
-    my $atime = shift;
-    my $mtime = shift;
-    my $ctime = shift;
+    my $id = shift;
 
-    if (defined($uid)) {
-       $valid |= &ATTR_UID;
+    if (!defined($id) || scalar($id) < 1) {
+       print "invalid arguments; type \"help setattr\" for a synopsis\n";
+       return;
     }
-    if (defined($gid)) {
-       $valid |= &ATTR_GID;
+
+    # XXX we do not currently set all of the fields in the obdo
+    my $obdo;
+    $obdo->{id} = $id;
+    $obdo->{mode} = oct(shift);
+    $obdo->{uid} = shift;
+    $obdo->{gid} = shift;
+    $obdo->{size} = shift;
+    $obdo->{atime} = shift;
+    $obdo->{mtime} = shift;
+    $obdo->{ctime} = shift;
+    $obdo->{valid} = 0;
+
+    if (defined($obdo->{atime})) {
+       $obdo->{valid} |= &OBD_MD_FLATIME;
     }
-    if (defined($size)) {
-       $valid |= &ATTR_SIZE;
+    if (defined($obdo->{mtime})) {
+       $obdo->{valid} |= &OBD_MD_FLMTIME;
     }
-    if (defined($atime)) {
-       $valid |= &ATTR_ATIME;
+    if (defined($obdo->{ctime})) {
+       $obdo->{valid} |= &OBD_MD_FLCTIME;
     }
-    if (defined($mtime)) {
-       $valid |= &ATTR_MTIME;
+    if (defined($obdo->{size})) {
+       $obdo->{valid} |= &OBD_MD_FLSIZE;
     }
-    if (defined($ctime)) {
-       $valid |= &ATTR_CTIME;
+    if (defined($obdo->{mode})) {
+       $obdo->{valid} |= &OBD_MD_FLMODE;
     }
-    if (defined($mode)) {
-       $valid |= &ATTR_MODE;
+    if (defined($obdo->{uid})) {
+       $obdo->{valid} |= &OBD_MD_FLUID;
     }
-
-    if (!defined($inode) || scalar($inode) < 1) {
-       print "invalid arguments; type \"help setattr\" for a synopsis\n";
-       return;
+    if (defined($obdo->{gid})) {
+       $obdo->{valid} |= &OBD_MD_FLGID;
     }
 
-    #struct iattr {
-    #        unsigned int    ia_valid; (32)
-    #        umode_t         ia_mode; (16)
-    #        uid_t           ia_uid; (16)
-    #        gid_t           ia_gid; (16)
-    # -- 16 bit alignment here! --
-    #        off_t           ia_size; (32)
-    #        time_t          ia_atime; (32)
-    #        time_t          ia_mtime; (32)
-    #        time_t          ia_ctime; (32)
-    #        unsigned int    ia_attr_flags; (32)
-    #};
-
-    printf "valid is %x, mode is %o\n", $valid, $mode;
-    my $packed = pack("ILLSssx2ILLLL", $::client_id, $inode, $valid, $mode,
-                     $uid, $gid, $size, $atime, $mtime, $ctime, 0);
+    printf "valid is %x, mode is %o\n", $obdo->{valid}, $obdo->{mode};
+    my $packed = pack("L", $::client_id) . obdo_pack($obdo);
     my $rc = ioctl(DEV_OBD, &OBD_IOC_SETATTR, $packed);
 
     if (!defined $rc) {
@@ -954,11 +1125,11 @@ sub Read {
        return;
     }
 
-    my $inode = shift;
+    my $id = shift;
     my $count = shift;
     my $offset = shift;
   
-    if (!defined($inode) || scalar($inode) < 1 || !defined($count) ||
+    if (!defined($id) || scalar($id) < 1 || !defined($count) ||
        $count < 1 || (defined($offset) && $offset < 0)) {
        print "invalid arguments; type \"help read\" for a synopsis\n";
        return;
@@ -969,14 +1140,18 @@ sub Read {
     }
 
     print("Reading $count bytes starting at byte $offset from object " .
-         "$inode...\n");
+         "$id...\n");
 
     # "allocate" a large enough buffer
     my $buf = sprintf("%${count}s", " ");
     die "suck" if (length($buf) != $count);
 
+    my $obdo;
+    $obdo->{id} = $id;
+
     # the perl we're using doesn't support pack type Q, and offset is 64 bits
-    my $packed = pack("ILpLLL", $::client_id, $inode, $buf, $count, $offset, 0);
+    my $packed = pack("L", $::client_id) . obdo_pack($obdo) .
+                pack("p LL LL", $buf, $count, $offset);
 
     my $rc = ioctl(DEV_OBD, &OBD_IOC_READ, $packed);
 
@@ -1003,11 +1178,11 @@ sub Read2 {
        return;
     }
 
-    my $inode = shift;
+    my $id = shift;
     my $count = shift;
     my $offset = shift;
   
-    if (!defined($inode) || scalar($inode) < 1 || !defined($count) ||
+    if (!defined($id) || scalar($id) < 1 || !defined($count) ||
        $count < 1 || (defined($offset) && $offset < 0)) {
        print "invalid arguments; type \"help read\" for a synopsis\n";
        return;
@@ -1018,14 +1193,18 @@ sub Read2 {
     }
 
     print("Reading $count bytes starting at byte $offset from object " .
-         "$inode...\n");
+         "$id...\n");
 
     # "allocate" a large enough buffer
     my $buf = sprintf("%${count}s", " ");
     die "suck" if (length($buf) != $count);
 
+    my $obdo;
+    $obdo->{id} = $id;
+
     # the perl we're using doesn't support pack type Q, and offset is 64 bits
-    my $packed = pack("ILpLLL", $::client_id, $inode, $buf, $count, $offset, 0);
+    my $packed = pack("L", $::client_id) . obdo_pack($obdo) .
+                pack("p LL LL", $buf, $count, $offset);
 
     my $rc = ioctl(DEV_OBD, &OBD_IOC_READ2, $packed);
 
@@ -1052,12 +1231,12 @@ sub Write {
        return;
     }
 
-    my $inode = shift;
+    my $id = shift;
     my $offset = shift;
     my $text = join(' ', @_);
     my $count = length($text);
 
-    if (!defined($inode) || scalar($inode) < 1 || !defined($offset) ||
+    if (!defined($id) || scalar($id) < 1 || !defined($offset) ||
        scalar($offset) < 0) {
        print "invalid arguments; type \"help write\" for a synopsis\n";
        return;
@@ -1068,11 +1247,15 @@ sub Write {
        $count = 0;
     }
 
-    print("Writing $count bytes starting at byte $offset to object " .
-         "$inode...\n");
+    print("Writing $count bytes starting at byte $offset to object $id...\n");
+
+    my $obdo;
+    $obdo->{id} = $id;
 
     # the perl we're using doesn't support pack type Q
-    my $packed = pack("ILpLLL", $::client_id, $inode, $text, $count, $offset, 0);
+    my $packed = pack("L", $::client_id) . obdo_pack($obdo) .
+                pack("p LL LL", $buf, $count, $offset);
+
     my $rc = ioctl(DEV_OBD, &OBD_IOC_WRITE, $packed);
 
     $retval = unpack("l", $packed);
@@ -1091,21 +1274,64 @@ sub Write {
     }
 }
 
+sub Punch {
+    if (!defined($::client_id)) {
+       print "You must first ``connect''.\n";
+       return;
+    }
+
+    my $id = shift;
+    my $start = shift;
+    my $count = shift;
+
+    if (!defined($id) || scalar($id) < 1 || !defined($start) ||
+       scalar($start) < 0 || !defined($count) || scalar($count) < 0) {
+       print "invalid arguments; type \"help punch\" for a synopsis\n";
+       return;
+    }
+
+    print("Punching $count bytes starting at byte $start in object $id...\n");
+
+    my $obdo;
+    $obdo->{id} = $id;
+
+    # the perl we're using doesn't support pack type Q
+    my $packed = pack("L", $::client_id) . obdo_pack($obdo) .
+                pack("p LL LL", $buf, $start, $count);
+
+    my $rc = ioctl(DEV_OBD, &OBD_IOC_PUNCH, $packed);
+
+    $retval = unpack("l", $packed);
+
+    if (!defined $rc) {
+       print STDERR "ioctl failed: $!\n";
+    } elsif ($rc eq "0 but true") {
+       if ($retval >= 0) {
+               print "\nPunched $retval of an attempted $count bytes.\n";
+               print "Finished (success)\n";
+       } else {
+               print "Finished (error $retval)\n";
+       }
+    } else {
+       print "ioctl returned error code $rc.\n";
+    }
+}
+
 sub Preallocate {
-    my $arg = shift;
+    my $num = shift;
 
     if (!defined($::client_id)) {
        print "You must first ``connect''.\n";
        return;
     }
 
-    if (!defined($arg) || scalar($arg) < 1 || scalar($arg) > 32) {
-       $arg = 32;
+    if (!defined($num) || scalar($num) < 1 || scalar($num) > 32) {
+       $num = 32;
     }
 
-    print "Preallocating $arg inodes...\n";
-    my $packed = pack("LLx128", $::client_id, $arg);
-    # client id, alloc, inodes[32]
+    print "Preallocating $num objects...\n";
+    # client id, alloc, id[32]
+    my $packed = pack("LLx128", $::client_id, $num);
 
     my $rc = ioctl(DEV_OBD, &OBD_IOC_PREALLOCATE, $packed);
 
@@ -1113,11 +1339,11 @@ sub Preallocate {
        print STDERR "ioctl failed: $!\n";
     } elsif ($rc eq "0 but true") {
        my $alloc = unpack("x4L", $packed);
-       my @inodes = unpack("x8L32", $packed);
+       my @ids = unpack("x8L32", $packed);
        my $i;
 
-       print "Got $alloc inodes: ";
-       foreach $i (@inodes) {
+       print "Got $alloc objects: ";
+       foreach $i (@ids) {
            print $i . " ";
        }
        print "\nFinished (success)\n";
@@ -1178,12 +1404,12 @@ sub Statfs {
 }
 
 sub Help {
-    my $arg = shift;
+    my $cmd = shift;
 
-    if ( !$arg || !$commands{$arg} ) {
+    if ( !$cmd || !$commands{$cmd} ) {
        print "Comands: ", join( ' ', @jcm_cmd_list), "\n";
     } else {
-       print "Usage: " .  $commands{$arg}->{doc} . "\n";
+       print "Usage: " .  $commands{$cmd}->{doc} . "\n";
     }
 }