* Incorporates Zach's feedback on bug #783 to change the CDEBUG

  to represent actual bytes written (for partial pages)
* Updated llparser.pm to display html links only when all RPC lines
  for a transaction are present

diff --git a/lustre/llite/iod.c b/lustre/llite/iod.c
index 8b30a20..33ef519 100644 (file)
--- a/lustre/llite/iod.c
+++ b/lustre/llite/iod.c
@@ -150,9 +150,13 @@ static void ll_writeback( struct inode *inode,
 {
         int rc, i;
         struct obd_brw_set *set;
+        unsigned int nbytes;
         ENTRY;
-        CDEBUG(D_VFSTRACE, "VFS Op:inode=%lu,bytes=%u\n", inode->i_ino, llwp->npgs << PAGE_SHIFT);
 
+        nbytes = ((llwp->npgs-1)<< PAGE_SHIFT) + llwp->pga[llwp->npgs-1].count;
+
+        CDEBUG(D_VFSTRACE, "VFS Op:inode=%lu,bytes=%u\n", 
+               inode->i_ino, nbytes);
         set = obd_brw_set_new();
         if (set == NULL) {
                 EXIT;

diff --git a/lustre/utils/llparser.pm b/lustre/utils/llparser.pm
index 0d4d01e..317c5df 100644 (file)
--- a/lustre/utils/llparser.pm
+++ b/lustre/utils/llparser.pm
@@ -1116,13 +1116,14 @@ sub HTML_rightpane
        print $HTMHANDLE $summary_indent_string x $summary_indent;
        print $HTMHANDLE "$text\n</A>";
     } elsif ($text =~ /rpcxid #(\d+)(.*)/) {
-       if ($text =~ /link=/) {
-           $tmprpc = shift;
-           $pidhashref = shift;
            my $allexist = ($tmprpc->[$e_srvRPCent] &&
                            $tmprpc->[$e_srvRPCexit] &&
                            $tmprpc->[$e_cliRPCent] &&
                            $tmprpc->[$e_cliRPCexit]);
+        if ($text =~ /link=/) {
+            $tmprpc = shift;
+            $pidhashref = shift;
+            
            if ($tmprpc->[$e_rpcopc] < 104) {
                $anchortag = sprintf "%s_%s_%s_%s", 
                $tmprpc->[$e_rpcopc], $tmprpc->[$e_rpcxid],