lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 29 Mar 2013 20:34:43 +0000 From: Ken Moffat <zarniwhoop@...world.com> To: Gwendal Grignou <gwendal@...gle.com> Cc: linux-kernel@...r.kernel.org, jgarzik@...hat.com Subject: Re: [PATCH] [libata] Fix HDIO_DRIVE_CMD ioctl sense data check On Fri, Mar 29, 2013 at 06:31:03PM +0000, Ken Moffat wrote: > On Thu, Mar 28, 2013 at 10:56:49PM -0700, Gwendal Grignou wrote: > > Hmm, not sure. Smartd started and was happy to monitor the disk, > but I got two new messages between 'found in smartd database' and > 'is SMART capable. Adding to "monitor" list' - > > Mar 29 17:26:42 ac4tv smartd[2481]: Device: /dev/sda, not capable of > SMART Health Status check > Mar 29 17:26:42 ac4tv smartd[2481]: Device: /dev/sda, enable SMART > Automatic Offline Testing failed. > > I've seen the first (intermittently) when a drive was starting to > fail, and apparently there was a taskfile issue in the days of 2.6.22 > which also caused it to appear. I don't think I've seen the second > of these before. > > After going back and forth between the kernel where I reverted your > original patch, and regular rc4 plus this new patch the output from > running smartctl as root all seems to be consistent (including > 'Passed' for the health check). > > I'm now running with the patch again, and I've started a manual > 'long' test (which will take 85 minutes, the default 'offline' is > about 150 minutes). > Looks like the problem is confined to smartd, smartctl is different and working fine. The new messages only come from smartd.cpp : (sorry, long lines to avoid word wrapping) // capability check: SMART status if (cfg.smartcheck && ataSmartStatus2(atadev) == -1) { PrintOut(LOG_INFO,"Device: %s, not capable of SMART Health Status check\n",name); cfg.smartcheck = false; } and // enable/disable automatic on-line testing if (cfg.autoofflinetest) { // is this an enable or disable request? const char *what=(cfg.autoofflinetest==1)?"disable":"enable"; if (!smart_val_ok) PrintOut(LOG_INFO,"Device: %s, could not %s SMART Automatic Offline Testing.\n",name, what); else { // if command appears unsupported, issue a warning... if (!isSupportAutomaticTimer(&state.smartval)) PrintOut(LOG_INFO,"Device: %s, SMART Automatic Offline Testing unsupported...\n",name); // ... but then try anyway if ((cfg.autoofflinetest==1)?ataDisableAutoOffline(atadev):ataEnableAutoOffline(atadev)) PrintOut(LOG_INFO,"Device: %s, %s SMART Automatic Offline Testing failed.\n", name, what); else PrintOut(LOG_INFO,"Device: %s, %sd SMART Automatic Offline Testing.\n", name, what); } } I've no idea about the details, but it looks to me as if smartd is still getting different values returned to it. The capability check normally was ok (silent), the automatic testing normally showed as 'enabled'd. ken -- das eine Mal als Tragödie, das andere Mal als Farce -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists