From: Bokhan A. <ap...@ng...> - 2012-02-28 21:33:52
|
29.02.2012 2:49, Alex Samorukov пишет: > 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. I do not have problems, about 50 arrays of lsi 9240/9260/9280 > > 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 > |