spelling/wording in sys/kern/vfs_bio.c

classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

spelling/wording in sys/kern/vfs_bio.c

Bret S. Lambert-2
Good morning tech@

        A few typos/plural agreement/apostrophe spelling changes, including
a personal favorite (insuring is a financial action).

- Bret

--- vfs_bio.c.orig Sun Aug 27 19:50:03 2006
+++ vfs_bio.c Sun Aug 27 20:31:59 2006
@@ -216,7 +216,7 @@
  * but not more than 25% and if possible
  * not less than 2 * MAXBSIZE. locleanpages
  * value must be not too small, but probably
- * there are no reason to set it more than 1-2 MB.
+ * there is no reason to set it more than 1-2 MB.
  */
  locleanpages = bufpages / 20;
  if (locleanpages < btoc(2 * MAXBSIZE))
@@ -239,7 +239,7 @@
  bp = getblk(vp, blkno, size, 0, 0);
 
  /*
- * If buffer does not have data valid, start a read.
+ * If buffer does not have valid data, start a read.
  * Note that if buffer is B_INVAL, getblk() won't return it.
  * Therefore, it's valid if its I/O has completed or been delayed.
  */
@@ -590,8 +590,8 @@
  * a given vnode and block offset. If it is found in the
  * block cache, mark it as having been found, make it busy
  * and return it. Otherwise, return an empty block of the
- * correct size. It is up to the caller to insure that the
- * cached blocks be of the correct size.
+ * correct size. It is up to the caller to ensure that the
+ * cached blocks are of the correct size.
  */
 struct buf *
 getblk(struct vnode *vp, daddr_t blkno, int size, int slpflag, int slptimeo)
@@ -677,7 +677,7 @@
  *
  * If the buffer shrinks, data is lost, so it's up to the
  * caller to have written it out *first*; this routine will not
- * start a write.  If the buffer grows, it's the callers
+ * start a write.  If the buffer grows, it's the caller's
  * responsibility to fill out the buffer's additional contents.
  */
 void

Reply | Threaded
Open this post in threaded view
|

Re: spelling/wording in sys/kern/vfs_bio.c

Jason McIntyre-2
On Mon, Aug 28, 2006 at 08:24:51AM -0400, tbert wrote:
> Good morning tech@
>
> A few typos/plural agreement/apostrophe spelling changes, including
> a personal favorite (insuring is a financial action).
>

thanks, fixed, except one (noted below)

> @@ -590,8 +590,8 @@
>   * a given vnode and block offset. If it is found in the
>   * block cache, mark it as having been found, make it busy
>   * and return it. Otherwise, return an empty block of the
> - * correct size. It is up to the caller to insure that the
> - * cached blocks be of the correct size.
> + * correct size. It is up to the caller to ensure that the
> + * cached blocks are of the correct size.
>   */
>  struct buf *
>  getblk(struct vnode *vp, daddr_t blkno, int size, int slpflag, int slptimeo)

"be of the correct size" is not incorrect (though it sounds a little
formal).

jmc