Module Name: src
Committed By: sborrill
Date: Fri Apr 3 16:23:41 UTC 2009
Modified Files:
src/sys/dev/raidframe: rf_disks.c rf_netbsdkintf.c
Log Message:
Switch various printfs from %ld and %d to PRIu64, etc. to be more consistent
about types (for instance uint32_t was being printed with %d).
To generate a diff of this commit:
cvs rdiff -u -r1.70 -r1.71 src/sys/dev/raidframe/rf_disks.c
cvs rdiff -u -r1.259 -r1.260 src/sys/dev/raidframe/rf_netbsdkintf.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/sys/dev/raidframe/rf_disks.c
diff -u src/sys/dev/raidframe/rf_disks.c:1.70 src/sys/dev/raidframe/rf_disks.c:1.71
--- src/sys/dev/raidframe/rf_disks.c:1.70 Mon Apr 28 20:23:56 2008
+++ src/sys/dev/raidframe/rf_disks.c Fri Apr 3 16:23:41 2009
@@ -1,4 +1,4 @@
-/* $NetBSD: rf_disks.c,v 1.70 2008/04/28 20:23:56 martin Exp $ */
+/* $NetBSD: rf_disks.c,v 1.71 2009/04/03 16:23:41 sborrill Exp $ */
/*-
* Copyright (c) 1999 The NetBSD Foundation, Inc.
* All rights reserved.
@@ -60,7 +60,7 @@
***************************************************************/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rf_disks.c,v 1.70 2008/04/28 20:23:56 martin Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rf_disks.c,v 1.71 2009/04/03 16:23:41 sborrill Exp $");
#include <dev/raidframe/raidframevar.h>
@@ -143,9 +143,9 @@
} else {
if (disks[c].numBlocks < min_numblks)
min_numblks = disks[c].numBlocks;
- DPRINTF6("Disk at col %d: dev %s numBlocks %ld blockSize %d (%ld MB)\n",
+ DPRINTF6("Disk at col %d: dev %s numBlocks %" PRIu64 " blockSize %d (%ld MB)\n",
c, disks[c].devname,
- (long int) disks[c].numBlocks,
+ disks[c].numBlocks,
disks[c].blockSize,
(long int) disks[c].numBlocks *
disks[c].blockSize / 1024 / 1024);
@@ -253,9 +253,9 @@
} else {
disks[i].status = rf_ds_spare; /* change status to
* spare */
- DPRINTF6("Spare Disk %d: dev %s numBlocks %ld blockSize %d (%ld MB)\n", i,
+ DPRINTF6("Spare Disk %d: dev %s numBlocks %" PRIu64 " blockSize %d (%ld MB)\n", i,
disks[i].devname,
- (long int) disks[i].numBlocks, disks[i].blockSize,
+ disks[i].numBlocks, disks[i].blockSize,
(long int) disks[i].numBlocks *
disks[i].blockSize / 1024 / 1024);
}
@@ -271,17 +271,17 @@
goto fail;
}
if (disks[i].numBlocks < raidPtr->sectorsPerDisk) {
- RF_ERRORMSG3("Spare disk %s (%d blocks) is too small to serve as a spare (need %ld blocks)\n",
+ RF_ERRORMSG3("Spare disk %s (%d blocks) is too small to serve as a spare (need %" PRIu64 " blocks)\n",
disks[i].devname, disks[i].blockSize,
- (long int) raidPtr->sectorsPerDisk);
+ raidPtr->sectorsPerDisk);
ret = EINVAL;
goto fail;
} else
if (disks[i].numBlocks > raidPtr->sectorsPerDisk) {
- RF_ERRORMSG3("Warning: truncating spare disk %s to %ld blocks (from %ld)\n",
+ RF_ERRORMSG3("Warning: truncating spare disk %s to %" PRIu64 " blocks (from %" PRIu64 ")\n",
disks[i].devname,
- (long int) raidPtr->sectorsPerDisk,
- (long int) disks[i].numBlocks);
+ raidPtr->sectorsPerDisk,
+ disks[i].numBlocks);
disks[i].numBlocks = raidPtr->sectorsPerDisk;
}
@@ -980,9 +980,10 @@
goto fail;
} else {
disks[spare_number].status = rf_ds_spare;
- DPRINTF6("Spare Disk %d: dev %s numBlocks %ld blockSize %d (%ld MB)\n", spare_number,
+ DPRINTF6("Spare Disk %d: dev %s numBlocks %" PRIu64 " blockSize %d (%ld MB)\n",
+ spare_number,
disks[spare_number].devname,
- (long int) disks[spare_number].numBlocks,
+ disks[spare_number].numBlocks,
disks[spare_number].blockSize,
(long int) disks[spare_number].numBlocks *
disks[spare_number].blockSize / 1024 / 1024);
@@ -998,20 +999,20 @@
goto fail;
}
if (disks[spare_number].numBlocks < raidPtr->sectorsPerDisk) {
- RF_ERRORMSG3("Spare disk %s (%d blocks) is too small to serve as a spare (need %ld blocks)\n",
+ RF_ERRORMSG3("Spare disk %s (%d blocks) is too small to serve as a spare (need %" PRIu64 " blocks)\n",
disks[spare_number].devname,
disks[spare_number].blockSize,
- (long int) raidPtr->sectorsPerDisk);
+ raidPtr->sectorsPerDisk);
rf_close_component(raidPtr, raidPtr->raid_cinfo[raidPtr->numCol+spare_number].ci_vp, 0);
ret = EINVAL;
goto fail;
} else {
if (disks[spare_number].numBlocks >
raidPtr->sectorsPerDisk) {
- RF_ERRORMSG3("Warning: truncating spare disk %s to %ld blocks (from %ld)\n",
+ RF_ERRORMSG3("Warning: truncating spare disk %s to %" PRIu64 " blocks (from %" PRIu64 ")\n",
disks[spare_number].devname,
- (long int) raidPtr->sectorsPerDisk,
- (long int) disks[spare_number].numBlocks);
+ raidPtr->sectorsPerDisk,
+ disks[spare_number].numBlocks);
disks[spare_number].numBlocks = raidPtr->sectorsPerDisk;
}
Index: src/sys/dev/raidframe/rf_netbsdkintf.c
diff -u src/sys/dev/raidframe/rf_netbsdkintf.c:1.259 src/sys/dev/raidframe/rf_netbsdkintf.c:1.260
--- src/sys/dev/raidframe/rf_netbsdkintf.c:1.259 Sun Mar 15 17:17:23 2009
+++ src/sys/dev/raidframe/rf_netbsdkintf.c Fri Apr 3 16:23:41 2009
@@ -1,4 +1,4 @@
-/* $NetBSD: rf_netbsdkintf.c,v 1.259 2009/03/15 17:17:23 cegger Exp $ */
+/* $NetBSD: rf_netbsdkintf.c,v 1.260 2009/04/03 16:23:41 sborrill Exp $ */
/*-
* Copyright (c) 1996, 1997, 1998, 2008 The NetBSD Foundation, Inc.
* All rights reserved.
@@ -139,7 +139,7 @@
***********************************************************/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: rf_netbsdkintf.c,v 1.259 2009/03/15 17:17:23 cegger Exp $");
+__KERNEL_RCSID(0, "$NetBSD: rf_netbsdkintf.c,v 1.260 2009/04/03 16:23:41 sborrill Exp $");
#ifdef _KERNEL_OPT
#include "opt_compat_netbsd.h"
@@ -2308,15 +2308,15 @@
*/
if (lp->d_secperunit != rs->sc_size)
printf("raid%d: WARNING: %s: "
- "total sector size in disklabel (%d) != "
- "the size of raid (%ld)\n", unit, rs->sc_xname,
- lp->d_secperunit, (long) rs->sc_size);
+ "total sector size in disklabel (%" PRIu32 ") != "
+ "the size of raid (%" PRIu64 ")\n", unit, rs->sc_xname,
+ lp->d_secperunit, rs->sc_size);
for (i = 0; i < lp->d_npartitions; i++) {
pp = &lp->d_partitions[i];
if (pp->p_offset + pp->p_size > rs->sc_size)
printf("raid%d: WARNING: %s: end of partition `%c' "
- "exceeds the size of raid (%ld)\n",
- unit, rs->sc_xname, 'a' + i, (long) rs->sc_size);
+ "exceeds the size of raid (%" PRIu64 ")\n",
+ unit, rs->sc_xname, 'a' + i, rs->sc_size);
}
}