sergiotarxz/nuttx
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Fix prolblems with redefinitions of fdb macro. With the fix to the syslog prototype, a LOG priority must now be the first paramter. Fixed by replacing all occurrences of fdbg with syslog(LOG_DEBUG, and eliminating the macro redefinitions. Noted by Sebastien Lorquet.

Browse Source
This commit is contained in:
Gregory Nutt 2014-11-12 07:15:13 -06:00
parent f06289e4d5
commit 6a9296c397
1 changed files with 28 additions and 27 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

45
fs/nxffs/nxffs_dump.c
View File

@ -55,12 +55,6 @@
/**************************************************************************** /****************************************************************************
* Pre-processor Definitions * Pre-processor Definitions
****************************************************************************/ ****************************************************************************/
/* Re-define fdbg so that the output does not have so much diagnostic info.
* This should still, however, always agree with the defintion in debug.h.
*/
#undef fdbg
#define fdbg syslog
/**************************************************************************** /****************************************************************************
* Private Types * Private Types
@ -173,7 +167,8 @@ static inline ssize_t nxffs_analyzeinode(FAR struct nxffs_blkinfo_s *blkinfo,
if (blkinfo->verbose) if (blkinfo->verbose)
{ {
fdbg(g_format, blkinfo->block, offset, "INODE", "UNVERFD", datlen); syslog(LOG_DEBUG, g_format,
blkinfo->block, offset, "INODE", "UNVERFD", datlen);
} }
return ERROR; return ERROR;
@ -199,7 +194,8 @@ static inline ssize_t nxffs_analyzeinode(FAR struct nxffs_blkinfo_s *blkinfo,
if (crc != ecrc) if (crc != ecrc)
{ {
fdbg(g_format, blkinfo->block, offset, "INODE", "CRC BAD", datlen); syslog(LOG_DEBUG, g_format,
blkinfo->block, offset, "INODE", "CRC BAD", datlen);
return ERROR; return ERROR;
} }
@ -209,19 +205,22 @@ static inline ssize_t nxffs_analyzeinode(FAR struct nxffs_blkinfo_s *blkinfo,
{ {
if (blkinfo->verbose) if (blkinfo->verbose)
{ {
fdbg(g_format, blkinfo->block, offset, "INODE", "OK ", datlen); syslog(LOG_DEBUG, g_format,
blkinfo->block, offset, "INODE", "OK ", datlen);
} }
} }
else if (state == INODE_STATE_DELETED) else if (state == INODE_STATE_DELETED)
{ {
if (blkinfo->verbose) if (blkinfo->verbose)
{ {
fdbg(g_format, blkinfo->block, offset, "INODE", "DELETED", datlen); syslog(LOG_DEBUG, g_format,
blkinfo->block, offset, "INODE", "DELETED", datlen);
} }
} }
else else
{ {
fdbg(g_format, blkinfo->block, offset, "INODE", "CORRUPT", datlen); syslog(LOG_DEBUG, g_format,
blkinfo->block, offset, "INODE", "CORRUPT", datlen);
} }
/* Return the block-relative offset to the next byte after the inode name */ /* Return the block-relative offset to the next byte after the inode name */
@ -271,7 +270,8 @@ static inline ssize_t nxffs_analyzedata(FAR struct nxffs_blkinfo_s *blkinfo,
if (crc != ecrc) if (crc != ecrc)
{ {
fdbg(g_format, blkinfo->block, offset, "DATA ", "CRC BAD", datlen); syslog(LOG_DEBUG, g_format,
blkinfo->block, offset, "DATA ", "CRC BAD", datlen);
return ERROR; return ERROR;
} }
@ -279,7 +279,8 @@ static inline ssize_t nxffs_analyzedata(FAR struct nxffs_blkinfo_s *blkinfo,
if (blkinfo->verbose) if (blkinfo->verbose)
{ {
fdbg(g_format, blkinfo->block, offset, "DATA ", "OK ", datlen); syslog(LOG_DEBUG, g_format,
blkinfo->block, offset, "DATA ", "OK ", datlen);
} }
return SIZEOF_NXFFS_DATA_HDR + datlen; return SIZEOF_NXFFS_DATA_HDR + datlen;
@ -309,7 +310,7 @@ static inline void nxffs_analyze(FAR struct nxffs_blkinfo_s *blkinfo)
blkhdr = (FAR struct nxffs_block_s *)blkinfo->buffer; blkhdr = (FAR struct nxffs_block_s *)blkinfo->buffer;
if (memcmp(blkhdr->magic, g_blockmagic, NXFFS_MAGICSIZE) != 0) if (memcmp(blkhdr->magic, g_blockmagic, NXFFS_MAGICSIZE) != 0)
{ {
fdbg(g_format, blkinfo->block, 0, "BLOCK", "NO FRMT", syslog(LOG_DEBUG, g_format, blkinfo->block, 0, "BLOCK", "NO FRMT",
blkinfo->geo.blocksize); blkinfo->geo.blocksize);
} }
else if (blkhdr->state == BLOCK_STATE_GOOD) else if (blkhdr->state == BLOCK_STATE_GOOD)
@ -320,7 +321,7 @@ static inline void nxffs_analyze(FAR struct nxffs_blkinfo_s *blkinfo)
{ {
if (blkinfo->verbose) if (blkinfo->verbose)
{ {
fdbg(g_format, blkinfo->block, 0, "BLOCK", "ERASED ", syslog(LOG_DEBUG, g_format, blkinfo->block, 0, "BLOCK", "ERASED ",
blkinfo->geo.blocksize); blkinfo->geo.blocksize);
} }
@ -329,19 +330,19 @@ static inline void nxffs_analyze(FAR struct nxffs_blkinfo_s *blkinfo)
#if 0 /* Too much output, to little information */ #if 0 /* Too much output, to little information */
else else
{ {
fdbg(g_format, blkinfo->block, 0, "BLOCK", "IN USE ", syslog(LOG_DEBUG, g_format, blkinfo->block, 0, "BLOCK", "IN USE ",
blkinfo->geo.blocksize); blkinfo->geo.blocksize);
} }
#endif #endif
} }
else if (blkhdr->state == BLOCK_STATE_BAD) else if (blkhdr->state == BLOCK_STATE_BAD)
{ {
fdbg(g_format, blkinfo->block, 0, "BLOCK", "BAD ", syslog(LOG_DEBUG, g_format, blkinfo->block, 0, "BLOCK", "BAD ",
blkinfo->geo.blocksize); blkinfo->geo.blocksize);
} }
else else
{ {
fdbg(g_format, blkinfo->block, 0, "BLOCK", "CORRUPT", syslog(LOG_DEBUG, g_format, blkinfo->block, 0, "BLOCK", "CORRUPT",
blkinfo->geo.blocksize); blkinfo->geo.blocksize);
} }
@ -448,8 +449,8 @@ int nxffs_dump(FAR struct mtd_dev_s *mtd, bool verbose)
/* Now read every block on the device */ /* Now read every block on the device */
fdbg("NXFFS Dump:\n"); syslog(LOG_DEBUG, "NXFFS Dump:\n");
fdbg(g_hdrformat); syslog(LOG_DEBUG, g_hdrformat);
blkinfo.nblocks = blkinfo.geo.erasesize * blkinfo.geo.neraseblocks / blkinfo.geo.blocksize; blkinfo.nblocks = blkinfo.geo.erasesize * blkinfo.geo.neraseblocks / blkinfo.geo.blocksize;
for (blkinfo.block = 0, blkinfo.offset = 0; for (blkinfo.block = 0, blkinfo.offset = 0;
@ -474,7 +475,7 @@ int nxffs_dump(FAR struct mtd_dev_s *mtd, bool verbose)
* just report the read error and continue. * just report the read error and continue.
*/ */
fdbg(g_format, blkinfo.block, 0, "BLOCK", "RD FAIL", syslog(LOG_DEBUG, g_format, blkinfo.block, 0, "BLOCK", "RD FAIL",
blkinfo.geo.blocksize); blkinfo.geo.blocksize);
#endif #endif
} }
@ -486,7 +487,7 @@ int nxffs_dump(FAR struct mtd_dev_s *mtd, bool verbose)
} }
} }
fdbg("%d blocks analyzed\n", blkinfo.nblocks); syslog(LOG_DEBUG, "%d blocks analyzed\n", blkinfo.nblocks);
kmm_free(blkinfo.buffer); kmm_free(blkinfo.buffer);
return OK; return OK;