From: Alex S. <ml...@os...> - 2012-02-28 19:49:40
|
Hi Artem, Initially it was done because i found that SAT can easily degrade raid and in the worst case - destroy the array. Later i identified "unsafe" commands and blocked them in the code (for Linux and FreeBSD) and now on all my arrays (about ~30 i guess) "smatctl -x /dev/sda" do not cause any problems. So its _probably_ safe to remove this. It would be great to see feedback from other users/developers before final decision. On 02/28/2012 06:13 PM, Bokhan Artem wrote: > Alex can you return the same function for megaraid? It's quite annoying > to support mixed sas/sata enviroment for large installations. May be > there is some option to force old behavior which I do not see. > > 28.02.2012 5:27, Christian Franke пишет: >> Brace, Don wrote: >>> We have many customers who have hundreds of disk drives that are using smartmontools to monitor drive health. >>> >>> But they want to see an enhancement to smartmontols to make it easier to use. >>> >>> Typically a customer has a mixture of SAS and SATA drives installed and would like to be able to run >>> smartmontools without having to know in advance if the drive is SAS or SATA. >> Alex Samorukov added a ticket for your patch in Nov, 2011: >> http://sourceforge.net/apps/trac/smartmontools/ticket/202 >> >> There are still open issues, see the comments in the ticket and the >> questions below. >> >> >>> Currently if the drive is SATA they have to use -device=sat+cciss,N and -device=cciss,N for SAS drives. >> Does --device=sat+cciss,N work for all commands, including DATA OUT and >> 48-bit ATA commands (Try smartctl -x ...) ? >> >> >>> In the past I had added code to smartmontools that enabled it to determine if the drive was SAS or SATA and issue the appropriate command automatically. >> Does your code work with all commands, including DATA OUT and 48-bit ATA >> commands (Try smartctl -x ...) ? >> >> >>> Would this enhancement be acceptable? >> Yes - this enhancements makes sense. >> No - the current patch likely includes an unneeded (and incomplete) >> re-implementation of SAT. >> >> >>> If so, who would make these enhancements? >> I could commit the fixed patch for the next release. >> >> Thanks, >> Christian >> >> >> ------------------------------------------------------------------------------ >> Try before you buy = See our experts in action! >> The most comprehensive online learning library for Microsoft developers >> is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3, >> Metro Style Apps, more. Free future releases when you subscribe now! >> http://p.sf.net/sfu/learndevnow-dev2 >> _______________________________________________ >> Smartmontools-support mailing list >> Sma...@li... >> https://lists.sourceforge.net/lists/listinfo/smartmontools-support > > ------------------------------------------------------------------------------ > Keep Your Developer Skills Current with LearnDevNow! > The most comprehensive online learning library for Microsoft developers > is just $99.99! Visual Studio, SharePoint, SQL - plus HTML5, CSS3, MVC3, > Metro Style Apps, more. Free future releases when you subscribe now! > http://p.sf.net/sfu/learndevnow-d2d > _______________________________________________ > Smartmontools-support mailing list > Sma...@li... > https://lists.sourceforge.net/lists/listinfo/smartmontools-support |