Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 14 May 2011 01:10:57 +0000 (UTC)
From:      John Baldwin <jhb@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r221867 - head/sys/fs/ext2fs
Message-ID:  <201105140110.p4E1Awgm088066@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: jhb
Date: Sat May 14 01:10:57 2011
New Revision: 221867
URL: http://svn.freebsd.org/changeset/base/221867

Log:
  Merge comments about converting directory entries to be more direct and
  concise.
  
  Inspired by:	Gleb Kurtsou

Modified:
  head/sys/fs/ext2fs/ext2_lookup.c

Modified: head/sys/fs/ext2fs/ext2_lookup.c
==============================================================================
--- head/sys/fs/ext2fs/ext2_lookup.c	Sat May 14 00:45:35 2011	(r221866)
+++ head/sys/fs/ext2fs/ext2_lookup.c	Sat May 14 01:10:57 2011	(r221867)
@@ -119,17 +119,11 @@ static int	ext2_dirbadentry(struct vnode
 /*
  * Vnode op for reading directories.
  *
- * The routine below assumes that the on-disk format of a directory
- * is the same as that defined by <sys/dirent.h>. If the on-disk
- * format changes, then it will be necessary to do a conversion
- * from the on-disk format that read returns to the format defined
- * by <sys/dirent.h>.
- */
-/*
- * this is exactly what we do here - the problem is that the conversion
- * will blow up some entries by four bytes, so it can't be done in place.
- * This is too bad. Right now the conversion is done entry by entry, the
- * converted entry is sent via uiomove.
+ * This function has to convert directory entries from the on-disk
+ * format to the format defined by <sys/dirent.h>.  Unfortunately, the
+ * conversion will blow up some entries by four bytes, so it can't be
+ * done in place.  Instead, the conversion is done entry by entry and
+ * the converted entry is sent via uiomove.
  *
  * XXX allocate a buffer, convert as many entries as possible, then send
  * the whole buffer to uiomove



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201105140110.p4E1Awgm088066>