From adde79883ffe726fc3237edb4756a99fba608797 Mon Sep 17 00:00:00 2001 From: Rob Landley Date: Tue, 16 May 2006 15:32:30 +0000 Subject: [PATCH] Largeish cleanup patch from Tito, mostly if statement therapy. --- miscutils/hdparm.c | 884 ++++++++++++++++++--------------------------- 1 file changed, 350 insertions(+), 534 deletions(-) diff --git a/miscutils/hdparm.c b/miscutils/hdparm.c index 5b51f71d0..bf24a2d3a 100644 --- a/miscutils/hdparm.c +++ b/miscutils/hdparm.c @@ -181,11 +181,11 @@ static const char * const pkt_str[] = { "", /* word 0, bits 12-8 = 1c */ "", /* word 0, bits 12-8 = 1d */ "", /* word 0, bits 12-8 = 1e */ - "Unknown", /* word 0, bits 12-8 = 1f */ + "Unknown", /* word 0, bits 12-8 = 1f */ }; static const char * const ata1_cfg_str[] = { /* word 0 in ATA-1 mode */ - "reserved", /* bit 0 */ + "Reserved", /* bit 0 */ "hard sectored", /* bit 1 */ "soft sectored", /* bit 2 */ "not MFM encoded ", /* bit 3 */ @@ -263,39 +263,39 @@ static const char * const ata1_cfg_str[] = { /* word 0 in ATA-1 mode */ #ifdef CONFIG_FEATURE_HDPARM_GET_IDENTITY static const char *minor_str[MINOR_MAX+2] = { /* word 81 value: */ "Unspecified", /* 0x0000 */ - "ATA-1 X3T9.2 781D prior to revision 4", /* 0x0001 */ + "ATA-1 X3T9.2 781D prior to rev.4", /* 0x0001 */ "ATA-1 published, ANSI X3.221-1994", /* 0x0002 */ - "ATA-1 X3T9.2 781D revision 4", /* 0x0003 */ + "ATA-1 X3T9.2 781D rev.4", /* 0x0003 */ "ATA-2 published, ANSI X3.279-1996", /* 0x0004 */ - "ATA-2 X3T10 948D prior to revision 2k", /* 0x0005 */ - "ATA-3 X3T10 2008D revision 1", /* 0x0006 */ - "ATA-2 X3T10 948D revision 2k", /* 0x0007 */ - "ATA-3 X3T10 2008D revision 0", /* 0x0008 */ - "ATA-2 X3T10 948D revision 3", /* 0x0009 */ + "ATA-2 X3T10 948D prior to rev.2k", /* 0x0005 */ + "ATA-3 X3T10 2008D rev.1", /* 0x0006 */ + "ATA-2 X3T10 948D rev.2k", /* 0x0007 */ + "ATA-3 X3T10 2008D rev.0", /* 0x0008 */ + "ATA-2 X3T10 948D rev.3", /* 0x0009 */ "ATA-3 published, ANSI X3.298-199x", /* 0x000a */ - "ATA-3 X3T10 2008D revision 6", /* 0x000b */ - "ATA-3 X3T13 2008D revision 7 and 7a", /* 0x000c */ - "ATA/ATAPI-4 X3T13 1153D revision 6", /* 0x000d */ - "ATA/ATAPI-4 T13 1153D revision 13", /* 0x000e */ - "ATA/ATAPI-4 X3T13 1153D revision 7", /* 0x000f */ - "ATA/ATAPI-4 T13 1153D revision 18", /* 0x0010 */ - "ATA/ATAPI-4 T13 1153D revision 15", /* 0x0011 */ + "ATA-3 X3T10 2008D rev.6", /* 0x000b */ + "ATA-3 X3T13 2008D rev.7 and 7a", /* 0x000c */ + "ATA/ATAPI-4 X3T13 1153D rev.6", /* 0x000d */ + "ATA/ATAPI-4 T13 1153D rev.13", /* 0x000e */ + "ATA/ATAPI-4 X3T13 1153D rev.7", /* 0x000f */ + "ATA/ATAPI-4 T13 1153D rev.18", /* 0x0010 */ + "ATA/ATAPI-4 T13 1153D rev.15", /* 0x0011 */ "ATA/ATAPI-4 published, ANSI INCITS 317-1998", /* 0x0012 */ - "ATA/ATAPI-5 T13 1321D revision 3", - "ATA/ATAPI-4 T13 1153D revision 14", /* 0x0014 */ - "ATA/ATAPI-5 T13 1321D revision 1", /* 0x0015 */ + "ATA/ATAPI-5 T13 1321D rev.3", + "ATA/ATAPI-4 T13 1153D rev.14", /* 0x0014 */ + "ATA/ATAPI-5 T13 1321D rev.1", /* 0x0015 */ "ATA/ATAPI-5 published, ANSI INCITS 340-2000", /* 0x0016 */ - "ATA/ATAPI-4 T13 1153D revision 17", /* 0x0017 */ - "ATA/ATAPI-6 T13 1410D revision 0", /* 0x0018 */ - "ATA/ATAPI-6 T13 1410D revision 3a", /* 0x0019 */ - "ATA/ATAPI-7 T13 1532D revision 1", /* 0x001a */ - "ATA/ATAPI-6 T13 1410D revision 2", /* 0x001b */ - "ATA/ATAPI-6 T13 1410D revision 1", /* 0x001c */ + "ATA/ATAPI-4 T13 1153D rev.17", /* 0x0017 */ + "ATA/ATAPI-6 T13 1410D rev.0", /* 0x0018 */ + "ATA/ATAPI-6 T13 1410D rev.3a", /* 0x0019 */ + "ATA/ATAPI-7 T13 1532D rev.1", /* 0x001a */ + "ATA/ATAPI-6 T13 1410D rev.2", /* 0x001b */ + "ATA/ATAPI-6 T13 1410D rev.1", /* 0x001c */ "ATA/ATAPI-7 published, ANSI INCITS 397-2005", /* 0x001d */ - "ATA/ATAPI-7 T13 1532D revision 0", /* 0x001e */ + "ATA/ATAPI-7 T13 1532D rev.0", /* 0x001e */ "Reserved" /* 0x001f */ "Reserved" /* 0x0020 */ - "ATA/ATAPI-7 T13 1532D revision 4a", /* 0x0021 */ + "ATA/ATAPI-7 T13 1532D rev.4a", /* 0x0021 */ "ATA/ATAPI-6 published, ANSI INCITS 361-2002", /* 0x0022 */ "Reserved" /* 0x0023-0xfffe*/ }; @@ -368,8 +368,8 @@ static const char * const cmd_feat_str[] = { /* --------------*/ "", /* word 83 bit 15: !valid bit */ "", /* word 83 bit 14: valid bit */ - "FLUSH CACHE EXT command", /* word 83 bit 13 */ - "Mandatory FLUSH CACHE command ", /* word 83 bit 12 */ + "FLUSH CACHE EXT cmd", /* word 83 bit 13 */ + "Mandatory FLUSH CACHE cmd ", /* word 83 bit 12 */ "Device Configuration Overlay feature set ", "48-bit Address feature set ", /* word 83 bit 10 */ "", @@ -465,10 +465,6 @@ static const char * const secu_str[] = { #undef DO_FLUSHCACHE /* under construction: force cache flush on -W0 */ /* Busybox messages and functions */ - -static const char bb_msg_shared_mem[] = "could not %s sharedmem buf"; -static const char bb_msg_op_not_supp[] = " operation not supported on %s disks"; - static int bb_ioctl(int fd, int request, void *argp, const char *string) { int e = ioctl(fd, request, argp); @@ -485,51 +481,31 @@ static int bb_ioctl_alt(int fd, int cmd, unsigned char *args, int alt, const cha return bb_ioctl(fd, cmd, args, string); } - -static void if_printf(unsigned long i, char *fmt, ... ) -{ - va_list ap; - va_start(ap, fmt); - if (i) - vprintf(fmt, ap); - va_end(ap); -} - static void on_off(unsigned int value); -static void if_printf_on_off(unsigned long get_arg,const char *fmt, unsigned long arg) +static void print_flag_on_off(unsigned long get_arg, const char *s, unsigned long arg) { + if (get_arg) { - printf(fmt, arg); + printf(" setting %s to %ld", s, arg); on_off(arg); } } static void bb_ioctl_on_off(int fd, int request, void *argp, const char *string, - const char * fmt) + const char * str) { if (ioctl(fd, request, &argp) != 0) bb_perror_msg(" %s", string); else { - printf(fmt, (unsigned long) argp); + printf(" %s\t= %2ld", str, (unsigned long) argp); on_off((unsigned long) argp); } } #ifdef CONFIG_FEATURE_HDPARM_GET_IDENTITY -static void if_else_printf(unsigned long i, char *fmt1, char *fmt2, ... ) -{ - va_list ap; - va_start(ap, fmt2); - if (i) - vprintf(fmt1, ap); - else - vprintf(fmt2, ap); - va_end(ap); -} - static void print_ascii(uint16_t *p, uint8_t length); static void xprint_ascii(uint16_t *val ,int i, char * string, int n) @@ -547,27 +523,6 @@ static void if_strcat(unsigned long test, char *modes, char *string) strcat(modes,string); } #endif - -static void sync_and_sleep(int i) -{ - sync(); - sleep(i); -} - -static uint16_t check_if_min_and_set_val(uint16_t a, uint16_t b) -{ - if ( a < b) - a = b; - return a; -} - -static uint16_t check_if_maj_and_set_val(uint16_t a, uint16_t b) -{ - if ( a > b) - a = b; - return a; -} - /* end of busybox specific stuff */ #ifdef CONFIG_FEATURE_HDPARM_GET_IDENTITY @@ -605,7 +560,7 @@ static void print_ascii(uint16_t *p, uint8_t length) { break; if ((cl = (char) 0x00ff&(*p)) != ' ') { - if_printf((cl != '\0'),"%c",cl); + if (cl != '\0') printf("%c",cl); p++; ii++; break; @@ -637,6 +592,7 @@ static void identify(uint16_t *id_supplied) uint8_t chksum = 0; uint32_t ll, mm, nn, oo; uint64_t bbbig; /* (:) */ + const char *strng; if (BB_BIG_ENDIAN) { swab(id_supplied, buf, sizeof(buf)); @@ -669,9 +625,7 @@ static void identify(uint16_t *id_supplied) /*"Unknown device type:\n\tbits 15&14 of general configuration word 0 both set to 1.\n"*/ bb_error_msg_and_die("Unknown device type"); - if_printf(!(val[GEN_CONFIG] & MEDIA_REMOVABLE),"non-"); - printf("removable media\n"); - + printf("%sremovable media\n", !(val[GEN_CONFIG] & MEDIA_REMOVABLE) ? "non-" : ""); /* Info from the specific configuration word says whether or not the * ID command completed correctly. It is only defined, however in * ATA/ATAPI-5 & 6; it is reserved (value theoretically 0) in prior @@ -683,10 +637,10 @@ static void identify(uint16_t *id_supplied) (val[CONFIG]==PWRD_NID_VAL) || (val[CONFIG]==PWRD_ID_VAL) ) { like_std = 5; - if_printf(((val[CONFIG]==STBY_NID_VAL) || (val[CONFIG]==STBY_ID_VAL)), - "powers-up in standby; SET FEATURES subcmd spins-up.\n"); - if_printf((((val[CONFIG]==STBY_NID_VAL) || (val[CONFIG]==PWRD_NID_VAL)) && (val[GEN_CONFIG] & INCOMPLETE)), - "\n\tWARNING: ID response incomplete.\n\tFollowing data may be incorrect.\n\n"); + if((val[CONFIG]==STBY_NID_VAL) || (val[CONFIG]==STBY_ID_VAL)) + printf("powers-up in standby; SET FEATURES subcmd spins-up.\n"); + if(((val[CONFIG]==STBY_NID_VAL) || (val[CONFIG]==PWRD_NID_VAL)) && (val[GEN_CONFIG] & INCOMPLETE)) + printf("\n\tWARNING: ID response incomplete.\n\tFollowing data may be incorrect.\n\n"); } /* output the model and serial numbers and the fw revision */ @@ -703,9 +657,9 @@ static void identify(uint16_t *id_supplied) { if (val[MINOR] && (val[MINOR] <= MINOR_MAX)) { - like_std=check_if_min_and_set_val(like_std, 3); + if(like_std < 3) like_std = 3; std = actual_ver[val[MINOR]]; - if_printf(std,"\n\tUsed: %s ",minor_str[val[MINOR]]); + if(std) printf("\n\tUsed: %s ",minor_str[val[MINOR]]); } /* looks like when they up-issue the std, they obsolete one; @@ -726,17 +680,17 @@ static void identify(uint16_t *id_supplied) like_std = ii; kk = like_std >4 ? like_std-4: 0; } - min_std=check_if_maj_and_set_val(min_std, ii); + if(min_std > ii) min_std = ii; } jj <<= 1; } - like_std=check_if_min_and_set_val(like_std, 3); + if(like_std < 3) like_std = 3; } /* Figure out what standard the device is using if it hasn't told * us. If we know the std, check if the device is using any of * the words from the next level up. It happens. */ - like_std=check_if_min_and_set_val(like_std, std); + if(like_std < std) like_std = std; if (((std == 5) || (!std && (like_std < 6))) && ((((val[CMDS_SUPP_1] & VALID) == VALID_VAL) && @@ -798,11 +752,11 @@ static void identify(uint16_t *id_supplied) jj = val[CDR_MAJOR] >> 1; for (ii = 1; ii <15; ii++) { - if_printf((jj & 0x0001),"-%u ", ii); + if(jj & 0x0001) printf("-%u ", ii); jj >>= 1; } } - if_else_printf((!kk),"\n\tLikely used CD-ROM ATAPI-1\n","\n"); + printf("%s\n", (!kk) ? "\n\tLikely used CD-ROM ATAPI-1" : "" ); /* the cdrom stuff is more like ATA-2 than anything else, so: */ like_std = 2; } @@ -817,41 +771,29 @@ static void identify(uint16_t *id_supplied) jj = val[GEN_CONFIG] >> 1; for (ii = 1; ii < 15; ii++) { - if_printf((jj & 0x0001),"\t%s\n",ata1_cfg_str[ii]); + if(jj & 0x0001) printf("\t%s\n",ata1_cfg_str[ii]); jj >>=1; } } if (dev == ATAPI_DEV) { - printf("\tDRQ response: "); /* Data Request (DRQ) */ - switch(val[GEN_CONFIG] & DRQ_RESPONSE_TIME) - { - case DRQ_3MS_VAL : - printf("3ms.\n"); - break; - case DRQ_INTR_VAL : - printf("<=10ms with INTRQ\n"); - break; - case DRQ_50US_VAL : - printf("50us.\n"); - break; - default : - printf("unknown.\n"); - break; - } - printf("\tPacket size: "); - switch(val[GEN_CONFIG] & PKT_SIZE_SUPPORTED) - { - case PKT_SIZE_12_VAL : - printf("12 bytes\n"); - break; - case PKT_SIZE_16_VAL : - printf("16 bytes\n"); - break; - default : - printf("Unknown\n"); - break; - } + if ((val[GEN_CONFIG] & DRQ_RESPONSE_TIME) == DRQ_3MS_VAL) + strng = "3ms"; + else if ((val[GEN_CONFIG] & DRQ_RESPONSE_TIME) == DRQ_INTR_VAL) + strng = "<=10ms with INTRQ"; + else if ((val[GEN_CONFIG] & DRQ_RESPONSE_TIME) == DRQ_50US_VAL) + strng ="50us"; + else + strng = "Unknown"; + printf("\tDRQ response: %s\n\tPacket size: ", strng); /* Data Request (DRQ) */ + + if ((val[GEN_CONFIG] & PKT_SIZE_SUPPORTED) == PKT_SIZE_12_VAL) + strng = "12 bytes"; + else if ((val[GEN_CONFIG] & PKT_SIZE_SUPPORTED) == PKT_SIZE_16_VAL) + strng = "16 bytes"; + else + strng = "Unknown"; + puts(strng); } else { @@ -866,8 +808,8 @@ static void identify(uint16_t *id_supplied) printf("\tLogical\t\tmax\tcurrent\n\tcylinders\t%u\t%u\n\theads\t\t%u\t%u\n\tsectors/track\t%u\t%u\n\t--\n", val[LCYLS],jj?val[LCYLS_CUR]:0, val[LHEADS],jj?val[LHEADS_CUR]:0, val[LSECTS],jj?val[LSECTS_CUR]:0); - if_printf(((min_std == 1) && (val[TRACK_BYTES] || val[SECT_BYTES])), - "\tbytes/track: %u\tbytes/sector: %u\n",val[TRACK_BYTES], val[SECT_BYTES]); + if ((min_std == 1) && (val[TRACK_BYTES] || val[SECT_BYTES])) + printf("\tbytes/track: %u\tbytes/sector: %u\n",val[TRACK_BYTES], val[SECT_BYTES]); if (jj) { @@ -901,8 +843,10 @@ static void identify(uint16_t *id_supplied) bbbig = (bbbig<<9)/1000000; printf("\tdevice size with M = 1000*1000: %11llu MBytes ",(long long unsigned int)bbbig); - if_else_printf((bbbig > 1000),"(%llu GB)\n","\n",bbbig/1000); - + if (bbbig > 1000) + printf("(%llu GB)\n", bbbig/1000); + else + printf("\n"); } /* hw support of commands (capabilities) */ @@ -910,48 +854,39 @@ static void identify(uint16_t *id_supplied) if (dev == ATAPI_DEV) { - if (eqpt != CDROM) - if_printf((val[CAPAB_0] & CMD_Q_SUP),"Cmd queuing, "); - - if_printf((val[CAPAB_0] & OVLP_SUP),"Cmd overlap, "); + if (eqpt != CDROM && (val[CAPAB_0] & CMD_Q_SUP)) printf("Cmd queuing, "); + if (val[CAPAB_0] & OVLP_SUP) printf("Cmd overlap, "); } - if_printf((val[CAPAB_0] & LBA_SUP),"LBA, "); + if (val[CAPAB_0] & LBA_SUP) printf("LBA, "); if (like_std != 1) { - printf("IORDY"); - if_printf((!(val[CAPAB_0] & IORDY_SUP)),"(may be)"); - if_else_printf((val[CAPAB_0] & IORDY_OFF),"(can","(cannot"); - printf(" be disabled)\n"); + printf("IORDY%s(can%s be disabled)\n", + !(val[CAPAB_0] & IORDY_SUP) ? "(may be)" : "", + (val[CAPAB_0] & IORDY_OFF) ? "" :"not"); } else printf("no IORDY\n"); if ((like_std == 1) && val[BUF_TYPE]) { - kk = val[BUF_TYPE]; - printf("\tBuffer type: %04x: ",kk); - if_else_printf((kk < 2),"single port, single-sector","dual port, multi-sector"); - if_printf((kk > 2)," with read caching ability"); - printf("\n"); + printf("\tBuffer type: %04x: %s%s\n", val[BUF_TYPE], + (val[BUF_TYPE] < 2) ? "single port, single-sector" : "dual port, multi-sector", + (val[BUF_TYPE] > 2) ? " with read caching ability" : ""); } - jj = 0; + if ((min_std == 1) && (val[BUFFER__SIZE] && (val[BUFFER__SIZE] != NOVAL_1))) { - printf("\tBuffer size: %.1fkB",(float)val[BUFFER__SIZE]/2); - jj = 1; + printf("\tBuffer size: %.1fkB\n",(float)val[BUFFER__SIZE]/2); } if ((min_std < 4) && (val[RW_LONG])) { - printf("\tbytes avail on r/w long: %u",val[RW_LONG]); - jj = 1; + printf("\tbytes avail on r/w long: %u\n",val[RW_LONG]); } if ((eqpt != CDROM) && (like_std > 3)) { - printf("\tQueue depth: %u",(val[QUEUE_DEPTH] & DEPTH_BITS)+1); - jj = 1; + printf("\tQueue depth: %u\n",(val[QUEUE_DEPTH] & DEPTH_BITS)+1); } - if_printf(jj,"\n"); if (dev == ATA_DEV) { @@ -959,8 +894,7 @@ static void identify(uint16_t *id_supplied) printf("\tCan%s perform double-word IO\n",(!val[DWORD_IO]) ?"not":""); else { - printf("\tStandby timer values: spec'd by "); - if_else_printf((val[CAPAB_0] & STD_STBY),"Standard","Vendor"); + printf("\tStandby timer values: spec'd by %s", (val[CAPAB_0] & STD_STBY) ? "Standard" : "Vendor"); if ((like_std > 3) && ((val[CAPAB_1] & VALID) == VALID_VAL)) printf(", %s device specific minimum\n",(val[CAPAB_1] & MIN_STANDBY_TIMER)?"with":"no"); else @@ -972,8 +906,10 @@ static void identify(uint16_t *id_supplied) else { printf("Max = %u\tCurrent = ",val[SECTOR_XFER_MAX] & SECTOR_XFER); - if_else_printf((val[SECTOR_XFER_CUR] & MULTIPLE_SETTING_VALID), - "%u\n","?\n",val[SECTOR_XFER_CUR] & SECTOR_XFER); + if (val[SECTOR_XFER_CUR] & MULTIPLE_SETTING_VALID) + printf("%u\n", val[SECTOR_XFER_CUR] & SECTOR_XFER); + else + printf("?\n"); } if ((like_std > 3) && (val[CMDS_SUPP_1] & 0x0008)) { @@ -989,23 +925,22 @@ static void identify(uint16_t *id_supplied) else printf("unknown setting (0x%04x)\n", val[ADV_PWR]); } - if (like_std > 5) - { - if_printf(val[ACOUSTIC],"\tRecommended acoustic management value: %u, current value: %u\n", + if (like_std > 5 && val[ACOUSTIC]) { + printf("\tRecommended acoustic management value: %u, current value: %u\n", (val[ACOUSTIC] >> 8) & 0x00ff, val[ACOUSTIC] & 0x00ff); } } else { /* ATAPI */ - if (eqpt != CDROM) - if_printf((val[CAPAB_0] & SWRST_REQ),"\tATA sw reset required\n"); + if (eqpt != CDROM && (val[CAPAB_0] & SWRST_REQ)) + printf("\tATA sw reset required\n"); if (val[PKT_REL] || val[SVC_NBSY]) { printf("\tOverlap support:"); - if_printf(val[PKT_REL]," %uus to release bus.",val[PKT_REL]); - if_printf(val[SVC_NBSY]," %uus to clear BSY after SERVICE cmd.",val[SVC_NBSY]); + if (val[PKT_REL]) printf(" %uus to release bus.",val[PKT_REL]); + if (val[SVC_NBSY]) printf(" %uus to clear BSY after SERVICE cmd.",val[SVC_NBSY]); printf("\n"); } } @@ -1016,8 +951,8 @@ static void identify(uint16_t *id_supplied) printf("not supported\n"); else { - if_printf((val[DMA_MODE] && !val[SINGLE_DMA] && !val[MULTI_DMA]), - " sdma%u\n",(val[DMA_MODE] & MODE) >> 8); + if (val[DMA_MODE] && !val[SINGLE_DMA] && !val[MULTI_DMA]) + printf(" sdma%u\n",(val[DMA_MODE] & MODE) >> 8); if (val[SINGLE_DMA]) { jj = val[SINGLE_DMA]; @@ -1036,18 +971,18 @@ static void identify(uint16_t *id_supplied) kk = val[ULTRA_DMA] >> 8; err_dma += mode_loop(jj,kk,'u',&have_mode); } - if_printf((err_dma || !have_mode),"(?)"); + if (err_dma || !have_mode) printf("(?)"); printf("\n"); - if_printf(((dev == ATAPI_DEV) && (eqpt != CDROM) && (val[CAPAB_0] & DMA_IL_SUP)), - "\t Interleaved DMA support\n"); + if ((dev == ATAPI_DEV) && (eqpt != CDROM) && (val[CAPAB_0] & DMA_IL_SUP)) + printf("\t\tInterleaved DMA support\n"); if ((val[WHATS_VALID] & OK_W64_70) && (val[DMA_TIME_MIN] || val[DMA_TIME_NORM])) { - printf("\t Cycle time:"); - if_printf(val[DMA_TIME_MIN]," min=%uns",val[DMA_TIME_MIN]); - if_printf(val[DMA_TIME_NORM]," recommended=%uns",val[DMA_TIME_NORM]); + printf("\t\tCycle time:"); + if (val[DMA_TIME_MIN]) printf(" min=%uns",val[DMA_TIME_MIN]); + if (val[DMA_TIME_NORM]) printf(" recommended=%uns",val[DMA_TIME_NORM]); printf("\n"); } } @@ -1061,7 +996,7 @@ static void identify(uint16_t *id_supplied) jj = ((val[ADV_PIO_MODES] & PIO_SUP) << 3) | 0x0007; for (ii = 0; ii <= PIO_MODE_MAX ; ii++) { - if_printf((jj & 0x0001),"pio%d ",ii); + if (jj & 0x0001) printf("pio%d ",ii); jj >>=1; } printf("\n"); @@ -1079,9 +1014,9 @@ static void identify(uint16_t *id_supplied) { if (val[PIO_NO_FLOW] || val[PIO_FLOW]) { - printf("\t Cycle time:"); - if_printf(val[PIO_NO_FLOW]," no flow control=%uns", val[PIO_NO_FLOW]); - if_printf(val[PIO_FLOW]," IORDY flow control=%uns", val[PIO_FLOW]); + printf("\t\tCycle time:"); + if (val[PIO_NO_FLOW]) printf(" no flow control=%uns", val[PIO_NO_FLOW]); + if (val[PIO_FLOW]) printf(" IORDY flow control=%uns", val[PIO_FLOW]); printf("\n"); } } @@ -1095,8 +1030,7 @@ static void identify(uint16_t *id_supplied) { if ((jj & 0x8000) && (*cmd_feat_str[ii] != '\0')) { - if_else_printf((kk & 0x8000),"\t *","\t"); - printf("\t%s\n",cmd_feat_str[ii]); + printf("\t%s\t%s\n", (kk & 0x8000) ? " *" : "", cmd_feat_str[ii]); } jj <<=1; kk<<=1; if (ii%16 == 15) @@ -1111,8 +1045,9 @@ static void identify(uint16_t *id_supplied) } } } - if_printf(((val[RM_STAT] & RM_STAT_BITS) == RM_STAT_SUP), - "\tRemovable Media Status Notification feature set supported\n"); + /* Removable Media Status Notification feature set */ + if((val[RM_STAT] & RM_STAT_BITS) == RM_STAT_SUP) + printf("\t%s supported\n", cmd_feat_str[27]); /* security */ @@ -1120,20 +1055,19 @@ static void identify(uint16_t *id_supplied) (val[SECU_STATUS] || val[ERASE_TIME] || val[ENH_ERASE_TIME])) { printf("Security: \n"); - if_printf((val[PSWD_CODE] && (val[PSWD_CODE] != NOVAL_1)), - "\tMaster password revision code = %u\n",val[PSWD_CODE]); + if (val[PSWD_CODE] && (val[PSWD_CODE] != NOVAL_1)) + printf("\tMaster password revision code = %u\n",val[PSWD_CODE]); jj = val[SECU_STATUS]; if (jj) { for (ii = 0; ii < NUM_SECU_STR; ii++) { - if_else_printf((!(jj & 0x0001)),"\tnot\t%s\n", "\t\t%s\n", secu_str[ii]); + printf("\t%s\t%s\n", (!(jj & 0x0001)) ? "not" : "", secu_str[ii]); jj >>=1; } if (val[SECU_STATUS] & SECU_ENABLED) { - printf("\tSecurity level "); - if_else_printf((val[SECU_STATUS] & SECU_LEVEL),"maximum\n","high\n"); + printf("\tSecurity level %s\n", (val[SECU_STATUS] & SECU_LEVEL) ? "maximum" : "high"); } } jj = val[ERASE_TIME] & ERASE_BITS; @@ -1141,34 +1075,26 @@ static void identify(uint16_t *id_supplied) if (jj || kk) { printf("\t"); - if_printf(jj,"%umin for SECURITY ERASE UNIT. ", jj==ERASE_BITS ? 508 : jj<<1); - if_printf(kk,"%umin for ENHANCED SECURITY ERASE UNIT.", kk==ERASE_BITS ? 508 : kk<<1); + if (jj) printf("%umin for SECURITY ERASE UNIT. ", jj==ERASE_BITS ? 508 : jj<<1); + if (kk) printf("%umin for ENHANCED SECURITY ERASE UNIT.", kk==ERASE_BITS ? 508 : kk<<1); printf("\n"); } } /* reset result */ - if ((val[HWRST_RSLT] & VALID) == VALID_VAL) + jj = val[HWRST_RSLT]; + if ((jj & VALID) == VALID_VAL) { - printf("HW reset results:\n"); - if_else_printf((val[HWRST_RSLT] & CBLID),"\tCBLID- above Vih\n","\tCBLID- below Vih\n"); - - if (val[HWRST_RSLT] & RST0) - { - printf("\tDevice num = 0"); - jj = val[HWRST_RSLT]; - } - else - { - printf("\tDevice num = 1"); - jj = val[HWRST_RSLT] >> 8; - } - + if (!(oo = (jj & RST0))) + jj >>= 8; if ((jj & DEV_DET) == JUMPER_VAL) - printf(" determined by the jumper"); + strng = " determined by the jumper"; else if ((jj & DEV_DET) == CSEL_VAL) - printf(" determined by CSEL"); - printf("\n"); + strng = " determined by CSEL"; + else + strng = ""; + printf("HW reset results:\n\tCBLID- %s Vih\n\tDevice num = %i%s\n", + (val[HWRST_RSLT] & CBLID) ? "above" : "below", !(oo), strng); } /* more stuff from std 5 */ @@ -1176,23 +1102,18 @@ static void identify(uint16_t *id_supplied) { if (val[CFA_PWR_MODE] & VALID_W160) { - printf("CFA power mode 1:\n\t"); - if_else_printf((val[CFA_PWR_MODE] & PWR_MODE_OFF),"disabled","enabled"); - - if_printf((val[CFA_PWR_MODE] & PWR_MODE_REQ)," and required by some commands"); - printf("\n"); + printf("CFA power mode 1:\n\t%s%s\n", (val[CFA_PWR_MODE] & PWR_MODE_OFF) ? "disabled" : "enabled", + (val[CFA_PWR_MODE] & PWR_MODE_REQ) ? " and required by some commands" : ""); - if_printf((val[CFA_PWR_MODE] & MAX_AMPS),"\tMaximum current = %uma\n",val[CFA_PWR_MODE] & MAX_AMPS); + if (val[CFA_PWR_MODE] & MAX_AMPS) printf("\tMaximum current = %uma\n",val[CFA_PWR_MODE] & MAX_AMPS); } if ((val[INTEGRITY] & SIG) == SIG_VAL) { - printf("Checksum: "); - if_printf(chksum,"in"); - printf("correct\n"); + printf("Checksum: %scorrect\n", chksum ? "in" : ""); } } - exit(0); + exit(EXIT_SUCCESS); } #endif @@ -1267,13 +1188,13 @@ static unsigned long perform_tristate, tristate; #ifdef CONFIG_FEATURE_HDPARM_GET_IDENTITY static const char * const cfg_str[] = -{ "", " HardSect", " SoftSect", " NotMFM", - " HdSw>15uSec", " SpinMotCtl", " Fixed", " Removeable", - " DTR<=5Mbs", " DTR>5Mbs", " DTR>10Mbs", " RotSpdTol>.5%", - " dStbOff", " TrkOff", " FmtGapReq", " nonMagnetic" +{ "", "HardSect", "SoftSect", "NotMFM", + "HdSw>15uSec", "SpinMotCtl", "Fixed", "Removeable", + "DTR<=5Mbs", "DTR>5Mbs", "DTR>10Mbs", "RotSpdTol>.5%", + "dStbOff", "TrkOff", "FmtGapReq", "nonMagnetic" }; -static const char * const BuffType[] = {"unknown", "1Sect", "DualPort", "DualPortCache"}; +static const char * const BuffType[] = {"Unknown", "1Sect", "DualPort", "DualPortCache"}; static void dump_identity(const struct hd_driveid *id) { @@ -1286,19 +1207,16 @@ static void dump_identity(const struct hd_driveid *id) printf("\n Model=%.40s, FwRev=%.8s, SerialNo=%.20s\n Config={", id->model, id->fw_rev, id->serial_no); - for (i=0; i<=15; i++) - if_printf((id->config & (1<config & (1<cyls, id->heads, id->sectors, id->track_bytes, - id->sector_bytes, id->ecc_bytes); - - if (id->buf_type > 3) - printf("%s%u", " BuffType=", id->buf_type); - else - printf("%s%s", " BuffType=", BuffType[id->buf_type]); - - printf(", BuffSize=%ukB, MaxMultSect=%u", id->buf_size/2, id->max_multsect); + id->sector_bytes, id->ecc_bytes, + id->buf_type, BuffType[(id->buf_type > 3) ? 0 : id->buf_type], + id->buf_size/2, id->max_multsect); if (id->max_multsect) { printf(", MultSect="); @@ -1317,14 +1235,14 @@ static void dump_identity(const struct hd_driveid *id) if_strcat((id->tPIO >= 2), pmodes, "pio2 "); } - if_printf((!(id->field_valid&1))," (maybe):"); + if(!(id->field_valid&1)) printf(" (maybe):"); if (BB_BIG_ENDIAN) capacity = (id->cur_capacity0 << 16) | id->cur_capacity1; else capacity = (id->cur_capacity1 << 16) | id->cur_capacity0; printf(" CurCHS=%u/%u/%u, CurSects=%lu, LBA=%s",id->cur_cyls, id->cur_heads, id->cur_sectors, capacity , ((id->capability&2)==0)?"no":"yes"); - if_printf((id->capability&2),", LBAsects=%u", id->lba_capacity); + if (id->capability&2) printf(", LBAsects=%u", id->lba_capacity); if (id->capability&1) { @@ -1391,11 +1309,11 @@ static void dump_identity(const struct hd_driveid *id) } } } - if_printf(((id->capability&1) && (id->field_valid&2)), - ", tDMA={min:%u,rec:%u}", id->eide_dma_min, id->eide_dma_time); + if ((id->capability&1) && (id->field_valid&2)) + printf(", tDMA={min:%u,rec:%u}", id->eide_dma_min, id->eide_dma_time); printf("\n PIO modes: %s", pmodes); - if_printf((*dmodes),"\n DMA modes: %s", dmodes); - if_printf((*umodes),"\n UDMA modes: %s", umodes); + if (*dmodes) printf("\n DMA modes: %s", dmodes); + if (*umodes) printf("\n UDMA modes: %s", umodes); printf("\n AdvancedPM=%s",((id_regs[83]&8)==0)?"no":"yes"); if (id_regs[83] & 8) @@ -1407,16 +1325,17 @@ static void dump_identity(const struct hd_driveid *id) else printf(": mode=0x%02X (%u)",id_regs[91]&0xFF,id_regs[91]&0xFF); } - if_printf( (id_regs[82]&0x20)," WriteCache=%s",(id_regs[85]&0x20) ? "enabled" : "disabled"); + if (id_regs[82]&0x20) printf(" WriteCache=%s",(id_regs[85]&0x20) ? "enabled" : "disabled"); #ifdef __NEW_HD_DRIVE_ID if ((id->minor_rev_num && id->minor_rev_num <= 31) || (id->major_rev_num && id->minor_rev_num <= 31)) { - printf("\n Drive conforms to: "); - if_else_printf((id->minor_rev_num <= 31),"%s: ","unknown: ", minor_str[id->minor_rev_num]); + printf("\n Drive conforms to: %s: ", (id->minor_rev_num <= 31) ? minor_str[id->minor_rev_num] : "Unknown"); if (id->major_rev_num != 0x0000 && /* NOVAL_0 */ id->major_rev_num != 0xFFFF) { /* NOVAL_1 */ - for (i=0; i <= 15; i++) - if_printf((id->major_rev_num & (1<major_rev_num & (1<= e) /* more than 1MB/s */ + printf("%2d MB in %5.2f seconds =%6.2f MB/sec\n", t, e, t / e); + else + printf("%2d MB in %5.2f seconds =%6.2f kB/sec\n", t, e, t / e * 1024); +} + +static int do_blkgetsize (int fd, unsigned long long *blksize64) +{ + int rc; + unsigned int blksize32 = 0; + + if (0 == ioctl(fd, BLKGETSIZE64, blksize64)) { // returns bytes + *blksize64 /= 512; + return 0; + } + rc = ioctl(fd, BLKGETSIZE, &blksize32); // returns sectors + if (rc) + bb_perror_msg("BLKGETSIZE"); + *blksize64 = blksize32; + return rc; +} static void do_time(int flag, int fd) /* @@ -1477,91 +1406,78 @@ static void do_time(int flag, int fd) flag = 1 time_device */ { - int i; char *buf; - double elapsed; struct itimerval e1, e2; int shmid; - int timing_MB = TIMING_MB; + double elapsed, elapsed2; + unsigned int max_iterations = 1024, total_MB, iterations; + unsigned long long blksize; + + if (0 == do_blkgetsize(fd, &blksize)) { + max_iterations = blksize / (2 * 1024) / TIMING_BUF_MB; + } if ((shmid = shmget(IPC_PRIVATE, TIMING_BUF_BYTES, 0600)) == -1) { - bb_error_msg(bb_msg_shared_mem,"allocate"); /*"could not allocate sharedmem buf"*/ + bb_perror_msg("shmget"); /*"could not allocate sharedmem buf"*/ return; } if (shmctl(shmid, SHM_LOCK, NULL) == -1) { - bb_error_msg(bb_msg_shared_mem,"lock"); /*"could not lock sharedmem buf"*/ + bb_perror_msg("shmctl"); /*"could not lock sharedmem buf"*/ (void) shmctl(shmid, IPC_RMID, NULL); return; } if ((buf = shmat(shmid, (char *) 0, 0)) == (char *) -1) { - bb_error_msg(bb_msg_shared_mem,"attach"); /*"could not attach sharedmem buf"*/ + bb_perror_msg("shmat"); /*"could not attach sharedmem buf"*/ (void) shmctl(shmid, IPC_RMID, NULL); return; } if (shmctl(shmid, IPC_RMID, NULL) == -1) - bb_error_msg("shmctl(,IPC_RMID,)"); + bb_perror_msg("shmctl"); /* Clear out the device request queues & give them time to complete */ - sync_and_sleep(3); + sync(); + sleep(3); + + setitimer(ITIMER_REAL, &(struct itimerval){{1000,0},{1000,0}}, NULL); if (flag == 0) /* Time cache */ { - /* Calculate a correction factor for the basic - * overhead of doing a read() from the buffer cache. - * To do this, we read the data once to "cache it" and - * to force full preallocation of our timing buffer, - * and then we re-read it 10 times while timing it. - * - * getitimer() is used rather than gettimeofday() because - * it is much more consistent (on my machine, at least). - */ - setitimer(ITIMER_REAL, &(struct itimerval){{1000,0},{1000,0}}, NULL); - if (seek_to_zero(fd)) - return; - if (read_big_block(fd, buf)) - return; - printf(" Timing buffer-cache reads: "); + if (seek_to_zero (fd)) return; + if (read_big_block (fd, buf)) return; + printf(" Timing cached reads: "); fflush(stdout); - - /* Clear out the device request queues & give them time to complete */ - sync_and_sleep(1); - - /* Time re-reading from the buffer-cache */ + + /* Now do the timing */ + iterations = 0; getitimer(ITIMER_REAL, &e1); - for (i = (BUFCACHE_FACTOR * TIMING_BUF_COUNT) ; i > 0; --i) - { - if (seek_to_zero(fd)) + do { + ++iterations; + if (seek_to_zero (fd) || read_big_block (fd, buf)) goto quit; - if (read_big_block(fd, buf)) - goto quit; - } - getitimer(ITIMER_REAL, &e2); - correction = (e1.it_value.tv_sec - e2.it_value.tv_sec) + ((e1.it_value.tv_usec - e2.it_value.tv_usec) / 1000000.0); - - /* Now remove the lseek() from the correction factor */ + getitimer(ITIMER_REAL, &e2); + elapsed = (e1.it_value.tv_sec - e2.it_value.tv_sec) + + ((e1.it_value.tv_usec - e2.it_value.tv_usec) / 1000000.0); + } while (elapsed < 2.0); + total_MB = iterations * TIMING_BUF_MB; + + elapsed = (e1.it_value.tv_sec - e2.it_value.tv_sec) + + ((e1.it_value.tv_usec - e2.it_value.tv_usec) / 1000000.0); + + /* Now remove the lseek() and getitimer() overheads from the elapsed time */ getitimer(ITIMER_REAL, &e1); - for (i = (BUFCACHE_FACTOR * TIMING_BUF_COUNT) ; i > 0; --i) - { - if (seek_to_zero(fd)) + do { + if (seek_to_zero (fd)) goto quit; - } - getitimer(ITIMER_REAL, &e2); - correction -= (e1.it_value.tv_sec - e2.it_value.tv_sec) - + ((e1.it_value.tv_usec - e2.it_value.tv_usec) / 1000000.0); - - if ((BUFCACHE_FACTOR * timing_MB) >= correction) /* more than 1MB/s */ - printf("%2d MB in %5.2f seconds =%6.2f MB/sec\n", - (BUFCACHE_FACTOR * timing_MB), correction, - (BUFCACHE_FACTOR * timing_MB) / correction); - else - printf("%2d MB in %5.2f seconds =%6.2f kB/sec\n", - (BUFCACHE_FACTOR * timing_MB), correction, - (BUFCACHE_FACTOR * timing_MB) / correction * 1024); - correction /= BUFCACHE_FACTOR; - + getitimer(ITIMER_REAL, &e2); + elapsed2 = (e1.it_value.tv_sec - e2.it_value.tv_sec) + + ((e1.it_value.tv_usec - e2.it_value.tv_usec) / 1000000.0); + } while (--iterations); + + elapsed -= elapsed2; + print_timing(BUFCACHE_FACTOR * total_MB, elapsed); flush_buffer_cache(fd); sleep(1); } @@ -1569,49 +1485,28 @@ static void do_time(int flag, int fd) { printf(" Timing buffered disk reads: "); fflush(stdout); - /* * getitimer() is used rather than gettimeofday() because * it is much more consistent (on my machine, at least). */ - setitimer(ITIMER_REAL, &(struct itimerval){{1000,0},{1000,0}}, NULL); - /* Now do the timings for real */ + iterations = 0; getitimer(ITIMER_REAL, &e1); - for (i = TIMING_BUF_COUNT; i > 0; --i) - { - if (read_big_block(fd, buf)) + do { + ++iterations; + if (read_big_block (fd, buf)) goto quit; - } - getitimer(ITIMER_REAL, &e2); - - elapsed = (e1.it_value.tv_sec - e2.it_value.tv_sec) + ((e1.it_value.tv_usec - e2.it_value.tv_usec) / 1000000.0); - - if (timing_MB >= elapsed) /* more than 1MB/s */ - printf("%2d MB in %5.2f seconds =%6.2f MB/sec\n",timing_MB, elapsed, timing_MB / elapsed); - else - printf("%2d MB in %5.2f seconds =%6.2f kB/sec\n",timing_MB, elapsed, timing_MB / elapsed * 1024); - - /*"Hmm.. suspicious results: probably not enough free memory for a proper test.");*/ - if (elapsed <= (correction * 2)) - bb_error_msg(bb_msg_memory_exhausted); - -#if 0 /* the "estimate" is just plain wrong for many systems.. */ - else if (correction != 0.0) { - printf(" Estimating raw driver speed: "); - elapsed -= correction; - if (timing_MB >= elapsed) /* more than 1MB/s */ - printf("%2d MB in %5.2f seconds =%6.2f MB/sec\n", - timing_MB, elapsed, timing_MB / elapsed); - else - printf("%2d MB in %5.2f seconds =%6.2f kB/sec\n", - timing_MB, elapsed, timing_MB / elapsed * 1024); - } -#endif + getitimer(ITIMER_REAL, &e2); + elapsed = (e1.it_value.tv_sec - e2.it_value.tv_sec) + + ((e1.it_value.tv_usec - e2.it_value.tv_usec) / 1000000.0); + } while (elapsed < 3.0 && iterations < max_iterations); + + total_MB = iterations * TIMING_BUF_MB; + print_timing(total_MB, elapsed); } quit: if (-1 == shmdt(buf)) - bb_error_msg(bb_msg_shared_mem,"detach"); /*"could not detach sharedmem buf"*/ + bb_perror_msg("shmdt"); /*"could not detach sharedmem buf"*/ } static void on_off (unsigned int value) @@ -1622,70 +1517,42 @@ static void on_off (unsigned int value) #ifdef CONFIG_FEATURE_HDPARM_HDIO_TRISTATE_HWIF static void bus_state_value(unsigned int value) { - const char *string; - - switch (value) - { - case BUSSTATE_ON: - string = " (on)\n"; - break; - case BUSSTATE_OFF: - string = " (off)\n"; - break; - case BUSSTATE_TRISTATE: - string = " (tristate)\n"; - break; - default: - string = " (unknown: %d)\n"; - break; - } - printf(string, value); + if (value == BUSSTATE_ON) + on_off(1); + else if (value == BUSSTATE_OFF) + on_off(0); + else if (value == BUSSTATE_TRISTATE) + printf(" (tristate)\n"); + else + printf(" (unknown: %d)\n", value); } #endif #ifdef HDIO_DRIVE_CMD static void interpret_standby(unsigned int standby) { - printf(" ("); - switch(standby) - { - case 0: - printf("off"); - break; - case 252: - printf("21 minutes"); - break; - case 253: - printf("vendor-specific"); - break; - case 254: - printf("?reserved"); - break; - case 255: - printf("21 minutes + 15 seconds"); - break; - default: - if (standby <= 240) - { - unsigned int secs = standby * 5; - unsigned int mins = secs / 60; - secs %= 60; - if_printf(mins,"%u minutes", mins); - if_printf((mins && secs)," + "); - if_printf(secs,"%u seconds", secs); - } - else if (standby <= 251) - { - unsigned int mins = (standby - 240) * 30; - unsigned int hrs = mins / 60; - mins %= 60; - if_printf(hrs,"%u hours", hrs); - if_printf((hrs && mins)," + "); - if_printf(mins,"%u minutes", mins); - } - else - printf("illegal value"); - break; + unsigned int t; + + + if (standby == 0) + printf("off"); + else if (standby == 252) + printf("21 minutes"); + else if (standby == 253) + printf("vendor-specific"); + else if (standby == 254) + printf("Reserved"); + else if (standby == 255) + printf("21 minutes + 15 seconds"); + else { + if (standby <= 240) { + t = standby * 5; + printf("%u minutes + %u seconds", t / 60, t % 60); + } else if (standby <= 251) { + t = (standby - 240) * 30; + printf("%u hours + %u minutes", t / 60, t % 60); + } else + printf("illegal value"); } printf(")\n"); } @@ -1742,7 +1609,6 @@ static int translate_xfermode(char * name) { if (!strcmp(name, tmp->name)) return tmp->val; - } val = strtol(name, &endptr, 10); @@ -1755,61 +1621,30 @@ static int translate_xfermode(char * name) static void interpret_xfermode(unsigned int xfermode) { printf(" ("); - switch(xfermode) { - case 0: - printf("default PIO mode"); - break; - case 1: - printf("default PIO mode, disable IORDY"); - break; - case 8: - case 9: - case 10: - case 11: - case 12: - case 13: - case 14: - case 15: - printf("PIO flow control mode%u", xfermode-8); - break; - case 16: - case 17: - case 18: - case 19: - case 20: - case 21: - case 22: - case 23: - printf("singleword DMA mode%u", xfermode-16); - break; - case 32: - case 33: - case 34: - case 35: - case 36: - case 37: - case 38: - case 39: - printf("multiword DMA mode%u", xfermode-32); - break; - case 64: - case 65: - case 66: - case 67: - case 68: - case 69: - case 70: - case 71: - printf("UltraDMA mode%u", xfermode-64); - break; - default: - printf("unknown, probably not valid"); - break; - } + if (xfermode == 0) + printf("default PIO mode"); + else if (xfermode == 1) + printf("default PIO mode, disable IORDY"); + else if (xfermode >= 8 && xfermode <= 15) + printf("PIO flow control mode%u", xfermode-8); + else if (xfermode >= 16 && xfermode <= 23) + printf("singleword DMA mode%u", xfermode-16); + else if (xfermode >= 32 && xfermode <= 39) + printf("multiword DMA mode%u", xfermode-32); + else if (xfermode >= 64 && xfermode <= 71) + printf("UltraDMA mode%u", xfermode-64); + else + printf("Unknown"); printf(")\n"); } #endif /* HDIO_DRIVE_CMD */ +static void print_flag(unsigned long flag, char *s, unsigned long value) +{ + if(flag) + printf(" setting %s to %ld\n", s, value); +} + static void process_dev(char *devname) { int fd; @@ -1820,13 +1655,14 @@ static void process_dev(char *devname) /* Please restore args[n] to these values after each ioctl except for args[2] */ unsigned char args[4] = {WIN_SETFEATURES,0,0,0}; + const char *fmt = " %s\t= %2ld"; fd = bb_xopen(devname, O_RDONLY|O_NONBLOCK); - if_printf(!quiet, "\n%s:\n", devname); + if (!quiet) printf("\n%s:\n", devname); if (set_readahead) { - if_printf(get_readahead," setting fs readahead to %ld\n", Xreadahead); + print_flag(get_readahead,"fs readahead", Xreadahead); bb_ioctl(fd, BLKRASET,(int *)Xreadahead,"BLKRASET"); } #ifdef CONFIG_FEATURE_HDPARM_HDIO_UNREGISTER_HWIF @@ -1866,12 +1702,12 @@ static void process_dev(char *devname) } if (set_io32bit) { - if_printf(get_io32bit," setting 32-bit IO_support flag to %ld\n", io32bit); + print_flag(get_io32bit,"32-bit IO_support flag", io32bit); bb_ioctl(fd, HDIO_SET_32BIT, (int *)io32bit, "HDIO_SET_32BIT"); } if (set_mult) { - if_printf(get_mult, " setting multcount to %ld\n", mult); + print_flag(get_mult, "multcount", mult); if (ioctl(fd, HDIO_SET_MULTCOUNT, mult)) bb_perror_msg("HDIO_SET_MULTCOUNT"); #ifndef HDIO_DRIVE_CMD @@ -1881,34 +1717,34 @@ static void process_dev(char *devname) } if (set_readonly) { - if_printf_on_off(get_readonly," setting readonly to %ld", readonly); + print_flag_on_off(get_readonly,"readonly", readonly); bb_ioctl(fd, BLKROSET, &readonly, "BLKROSET"); } if (set_unmask) { - if_printf_on_off(get_unmask," setting unmaskirq to %ld", unmask); + print_flag_on_off(get_unmask,"unmaskirq", unmask); bb_ioctl(fd, HDIO_SET_UNMASKINTR, (int *)unmask, "HDIO_SET_UNMASKINTR"); } #ifdef CONFIG_FEATURE_HDPARM_HDIO_GETSET_DMA if (set_dma) { - if_printf_on_off(get_dma," setting using_dma to %ld", dma); + print_flag_on_off(get_dma, "using_dma", dma); bb_ioctl(fd, HDIO_SET_DMA, (int *)dma, "HDIO_SET_DMA"); } #endif /* CONFIG_FEATURE_HDPARM_HDIO_GETSET_DMA */ if (set_dma_q) { - if_printf_on_off(get_dma_q," setting DMA queue_depth to %ld", dma_q); + print_flag_on_off(get_dma_q,"DMA queue_depth", dma_q); bb_ioctl(fd, HDIO_SET_QDMA, (int *)dma_q, "HDIO_SET_QDMA"); } if (set_nowerr) { - if_printf_on_off(get_nowerr," setting nowerr to %ld", nowerr); + print_flag_on_off(get_nowerr,"nowerr", nowerr); bb_ioctl(fd, HDIO_SET_NOWERR, (int *)nowerr,"HDIO_SET_NOWERR"); } if (set_keep) { - if_printf_on_off(get_keep," setting keep_settings to %ld", keep); + print_flag_on_off(get_keep,"keep_settings", keep); bb_ioctl(fd, HDIO_SET_KEEPSETTINGS, (int *)keep,"HDIO_SET_KEEPSETTINGS"); } #ifdef HDIO_DRIVE_CMD @@ -1916,28 +1752,28 @@ static void process_dev(char *devname) { args[0] = doorlock ? WIN_DOORLOCK : WIN_DOORUNLOCK; args[2] = 0; - if_printf_on_off(get_doorlock," setting drive doorlock to %ld", doorlock); + print_flag_on_off(get_doorlock,"drive doorlock", doorlock); bb_ioctl(fd, HDIO_DRIVE_CMD, &args,"HDIO_DRIVE_CMD(doorlock)"); args[0] = WIN_SETFEATURES; } if (set_dkeep) { /* lock/unlock the drive's "feature" settings */ - if_printf_on_off(get_dkeep," setting drive keep features to %ld", dkeep); + print_flag_on_off(get_dkeep,"drive keep features", dkeep); args[2] = dkeep ? 0x66 : 0xcc; bb_ioctl(fd, HDIO_DRIVE_CMD, &args,"HDIO_DRIVE_CMD(keepsettings)"); } if (set_defects) { args[2] = defects ? 0x04 : 0x84; - if_printf(get_defects," setting drive defect-mgmt to %ld\n", defects); + print_flag(get_defects,"drive defect-mgmt", defects); bb_ioctl(fd, HDIO_DRIVE_CMD, &args,"HDIO_DRIVE_CMD(defectmgmt)"); } if (set_prefetch) { args[1] = prefetch; args[2] = 0xab; - if_printf(get_prefetch," setting drive prefetch to %ld\n", prefetch); + print_flag(get_prefetch,"drive prefetch", prefetch); bb_ioctl(fd, HDIO_DRIVE_CMD, &args, "HDIO_DRIVE_CMD(setprefetch)"); args[1] = 0; } @@ -1947,7 +1783,7 @@ static void process_dev(char *devname) args[2] = 3; if (get_xfermode) { - printf(" setting xfermode to %d", xfermode_requested); + print_flag(1,/*" setting */"xfermode"/* to %d"*/, xfermode_requested); interpret_xfermode(xfermode_requested); } bb_ioctl(fd, HDIO_DRIVE_CMD, &args,"HDIO_DRIVE_CMD(setxfermode)"); @@ -1956,27 +1792,15 @@ static void process_dev(char *devname) if (set_lookahead) { args[2] = lookahead ? 0xaa : 0x55; - if_printf_on_off(get_lookahead," setting drive read-lookahead to %ld", lookahead); + print_flag_on_off(get_lookahead,"drive read-lookahead", lookahead); bb_ioctl(fd, HDIO_DRIVE_CMD, &args, "HDIO_DRIVE_CMD(setreadahead)"); } if (set_apmmode) { - apmmode=check_if_min_and_set_val(apmmode,1); - apmmode=check_if_maj_and_set_val(apmmode,255); - if_printf(get_apmmode," setting APM level to"); - if (apmmode==255) - { - /* disable Advanced Power Management */ - args[2] = 0x85; /* feature register */ - if_printf(get_apmmode," disabled\n"); - } - else - { - /* set Advanced Power Management mode */ - args[2] = 0x05; /* feature register */ - args[1] = apmmode; /* sector count register */ - if_printf(get_apmmode," 0x%02lX (%ld)\n",apmmode,apmmode); - } + args[2] = (apmmode == 255) ? 0x85 /* disable */ : 0x05 /* set */; /* feature register */ + args[1] = apmmode; /* sector count register 1-255 */ + if (get_apmmode) + printf(" setting APM level to %s 0x%02lX (%ld)\n", (apmmode == 255) ? "disabled" : "", apmmode, apmmode); bb_ioctl(fd, HDIO_DRIVE_CMD, &args,"HDIO_DRIVE_CMD"); args[1] = 0; } @@ -1989,7 +1813,7 @@ static void process_dev(char *devname) static unsigned char flushcache[4] = {WIN_FLUSHCACHE,0,0,0}; #endif /* DO_FLUSHCACHE */ args[2] = wcache ? 0x02 : 0x82; - if_printf_on_off(get_wcache," setting drive write-caching to %ld", wcache); + print_flag_on_off(get_wcache,"drive write-caching", wcache); #ifdef DO_FLUSHCACHE if (!wcache) bb_ioctl(fd, HDIO_DRIVE_CMD, &flushcache, "HDIO_DRIVE_CMD(flushcache)"); @@ -2013,7 +1837,7 @@ static void process_dev(char *devname) #ifndef WIN_STANDBYNOW2 #define WIN_STANDBYNOW2 0x94 #endif - if_printf(get_standbynow," issuing standby command\n"); + if (get_standbynow) printf(" issuing standby command\n"); args[0] = WIN_STANDBYNOW1; bb_ioctl_alt(fd, HDIO_DRIVE_CMD, args, WIN_STANDBYNOW2, "HDIO_DRIVE_CMD(standby)"); } @@ -2025,14 +1849,14 @@ static void process_dev(char *devname) #ifndef WIN_SLEEPNOW2 #define WIN_SLEEPNOW2 0x99 #endif - if_printf(get_sleepnow," issuing sleep command\n"); + if (get_sleepnow) printf(" issuing sleep command\n"); args[0] = WIN_SLEEPNOW1; bb_ioctl_alt(fd, HDIO_DRIVE_CMD, args, WIN_SLEEPNOW2, "HDIO_DRIVE_CMD(sleep)"); } if (set_seagate) { args[0] = 0xfb; - if_printf(get_seagate," disabling Seagate auto powersaving mode\n"); + if (get_seagate) printf(" disabling Seagate auto powersaving mode\n"); bb_ioctl(fd, HDIO_DRIVE_CMD, &args, "HDIO_DRIVE_CMD(seagatepwrsave)"); } if (set_standby) @@ -2041,7 +1865,7 @@ static void process_dev(char *devname) args[1] = standby_requested; if (get_standby) { - printf(" setting standby to %lu", standby_requested); + print_flag(1, /*" setting */"standby"/* to %lu"*/, standby_requested); interpret_standby(standby_requested); } bb_ioctl(fd, HDIO_DRIVE_CMD, &args, "HDIO_DRIVE_CMD(setidle1)"); @@ -2053,7 +1877,7 @@ static void process_dev(char *devname) char buf[512]; flush_buffer_cache(fd); if (-1 == read(fd, buf, sizeof(buf))) - bb_error_msg("access failed"); + bb_perror_msg("read(%d bytes) failed (rc=%d)", sizeof(buf), -1); } #endif /* HDIO_DRIVE_CMD */ @@ -2070,7 +1894,7 @@ static void process_dev(char *devname) } else if (verbose | get_mult) { - printf(" multcount = %2ld", multcount); + printf(fmt, "multcount", multcount); on_off(multcount); } } @@ -2078,33 +1902,25 @@ static void process_dev(char *devname) { if (!bb_ioctl(fd, HDIO_GET_32BIT, &parm, "HDIO_GET_32BIT")) { - printf(" IO_support =%3ld (", parm); - switch (parm) - { - case 0: - printf("default "); - case 2: - printf("16-bit)\n"); - break; - case 1: - printf("32-bit)\n"); - break; - case 3: - printf("32-bit w/sync)\n"); - break; - case 8: - printf("Request-Queue-Bypass)\n"); - break; - default: - printf("\?\?\?)\n"); - /*esac*/ - } + printf(" IO_support\t=%3ld (", parm); + if (parm == 0) + printf("default 16-bit)\n"); + else if (parm == 2) + printf("16-bit)\n"); + else if (parm == 1) + printf("32-bit)\n"); + else if (parm == 3) + printf("32-bit w/sync)\n"); + else if (parm == 8) + printf("Request-Queue-Bypass)\n"); + else + printf("\?\?\?)\n"); } } if (verbose || get_unmask) { bb_ioctl_on_off(fd, HDIO_GET_UNMASKINTR,(unsigned long *)parm, - "HDIO_GET_UNMASKINTR"," unmaskirq = %2ld"); + "HDIO_GET_UNMASKINTR","unmaskirq"); } @@ -2112,7 +1928,7 @@ static void process_dev(char *devname) if (verbose || get_dma) { if (!bb_ioctl(fd, HDIO_GET_DMA, &parm, "HDIO_GET_DMA")) { - printf(" using_dma = %2ld", parm); + printf(fmt, "using_dma", parm); if (parm == 8) printf(" (DMA-Assisted-PIO)\n"); else @@ -2123,28 +1939,28 @@ static void process_dev(char *devname) if (get_dma_q) { bb_ioctl_on_off (fd, HDIO_GET_QDMA,(unsigned long *)parm, - "HDIO_GET_QDMA"," queue_depth = %2ld"); + "HDIO_GET_QDMA","queue_depth"); } if (verbose || get_keep) { bb_ioctl_on_off (fd, HDIO_GET_KEEPSETTINGS,(unsigned long *)parm, - "HDIO_GET_KEEPSETTINGS"," keepsettings = %2ld"); + "HDIO_GET_KEEPSETTINGS","keepsettings"); } if (get_nowerr) { bb_ioctl_on_off (fd, HDIO_GET_NOWERR,(unsigned long *)&parm, - " HDIO_GET_NOWERR"," nowerr = %2ld"); + "HDIO_GET_NOWERR","nowerr"); } if (verbose || get_readonly) { bb_ioctl_on_off(fd, BLKROGET,(unsigned long *)parm, - " BLKROGET"," readonly = %2ld"); + "BLKROGET","readonly"); } if (verbose || get_readahead) { bb_ioctl_on_off (fd, BLKRAGET, (unsigned long *) parm, - " BLKRAGET"," readahead = %2ld"); + "BLKRAGET","readahead"); } if (verbose || get_geom) { @@ -2153,7 +1969,7 @@ static void process_dev(char *devname) struct hd_geometry g; if (!bb_ioctl(fd, HDIO_GETGEO, &g, "HDIO_GETGEO")) - printf(" geometry = %u/%u/%u, sectors = %ld, start = %ld\n", + printf(" geometry\t= %u/%u/%u, sectors = %ld, start = %ld\n", g.cylinders, g.heads, g.sectors, parm, g.start); } } @@ -2172,7 +1988,7 @@ static void process_dev(char *devname) if (bb_ioctl_alt(fd, HDIO_DRIVE_CMD, args, WIN_CHECKPOWERMODE2, 0)) { if (errno != EIO || args[0] != 0 || args[1] != 0) - state = "unknown"; + state = "Unknown"; else state = "sleeping"; } @@ -2239,7 +2055,7 @@ static void process_dev(char *devname) { if (get_busstate) { - printf(" setting bus state to %ld", busstate); + print_flag(1, "bus state", busstate); bus_state_value(busstate); } bb_ioctl(fd, HDIO_SET_BUSSTATE, (int *)(unsigned long)busstate, "HDIO_SET_BUSSTATE"); @@ -2250,7 +2066,7 @@ static void process_dev(char *devname) { if (!bb_ioctl(fd, HDIO_GET_BUSSTATE, &parm, "HDIO_GET_BUSSTATE")) { - printf(" busstate = %2ld", parm); + printf(fmt, "bus state", parm); bus_state_value(parm); } } @@ -2285,7 +2101,7 @@ static void identify_from_stdin(void) int i, count = read(0, buf, 1280); if (count != 1280) - bb_error_msg_and_die("read(1280 bytes) failed (rc=%d)", count); + bb_error_msg_and_die("read(%d bytes) failed (rc=%d)", 1280, count); for (i = 0; count >= 4; ++i) { sbuf[i] = (fromhex(b[0]) << 12) | (fromhex(b[1]) << 8) | (fromhex(b[2]) << 4) | fromhex(b[3]); @@ -2386,11 +2202,11 @@ int hdparm_main(int argc, char **argv) exit(EXIT_SUCCESS); } - verbose = (c == 'v'); + verbose |= (c == 'v'); USE_FEATURE_HDPARM_GET_IDENTITY(get_IDentity = (c == 'I')); USE_FEATURE_HDPARM_GET_IDENTITY(get_identity = (c == 'i')); - get_geom = (c == 'g'); - do_flush = (c == 'f'); + get_geom |= (c == 'g'); + do_flush |= (c == 'f'); if (c == 'q') { quiet = 1; noisy = 0; @@ -2405,8 +2221,8 @@ int hdparm_main(int argc, char **argv) if (c == 'k') parse_opts(&get_keep, &set_keep, &keep, 0, 1); if (c == 'a') parse_opts(&get_readahead, &set_readahead, &Xreadahead, 0, INT_MAX); if (c == 'B') parse_opts(&get_apmmode, &set_apmmode, &apmmode, 1, 255); - do_flush |= do_timings = (c == 't'); - do_flush |= do_ctimings = (c == 'T'); + do_flush |= do_timings |= (c == 't'); + do_flush |= do_ctimings |= (c == 'T'); #ifdef HDIO_DRIVE_CMD if (c == 'S') parse_opts(&get_standby, &set_standby, &standby_requested, 0, INT_MAX); if (c == 'D') parse_opts(&get_defects, &set_defects, &defects, 0, INT_MAX); @@ -2419,7 +2235,7 @@ int hdparm_main(int argc, char **argv) parse_opts_v3((c == 'C'), &get_powermode, NULL, NULL); parse_opts_v2((c == 'y'), &get_standbynow, &set_standbynow); parse_opts_v2((c == 'Y'), &get_sleepnow, &set_sleepnow); - reread_partn = (c == 'z'); + reread_partn |= (c == 'z'); parse_opts_v2((c == 'Z'), &get_seagate, &set_seagate); #endif USE_FEATURE_HDPARM_HDIO_UNREGISTER_HWIF(if (c == 'U') parse_opts(NULL, &unregister_hwif, &hwif, 0, INT_MAX)); -- 2.25.1