From: Erwan V. <er...@se...> - 2008-02-28 15:39:32
|
As many other distro, we've been switching to the current trunk. Two users already reports this bug https://qa.mandriva.com/show_bug.cgi?id=37887 It means that some users now needs "-d sat" to make smartmontools working. I'm proving a default configuration file that do : DEVICESCAN -a -o on -S on -s L/../../6/03 Since "-d sat" is needed on some devices, it makes the demon failing to start. Is there any possibility to make smartmontools trying "-d sat" automatically when the defaut behaviour fails ? Erwan, -------------------------------------------------------------------------------- Les opinions et prises de position emises par le signataire du present message lui sont propres et ne sauraient engager la responsabilite de la societe SEANODES. Ce message ainsi que les eventuelles pieces jointes constituent une correspondance privee et confidentielle a l'attention exclusive du destinataire designe ci-dessus. Si vous n'etes pas le destinataire du present message ou une personne susceptible de pouvoir le lui delivrer, il vous est signifie que toute divulgation, distribution ou copie de cette transmission est strictement interdite. Si vous avez recu ce message par erreur, nous vous remercions d'en informer l'expediteur par telephone ou de lui retourner le present message, puis d'effacer immediatement ce message de votre systeme. The views and opinions expressed by the author of this message are personal. SEANODES shall assume no liability, express or implied for such message. This e-mail and any attachments is a confidential correspondence intended only for use of the individual or entity named above. If you are not the intended recipient or the agent responsible for delivering the message to the intended recipient, you are hereby notified that any disclosure, distribution or copying of this communication is strictly prohibited. If you have received this communication in error, please notify the sender by phone or by replying this message, and then delete this message from your system. |
From: Bruce A. <ba...@gr...> - 2008-02-29 01:22:13
|
Hi Erwan, There have been a lot of requests to have DEVICESCAN automatically detect and monitor SATA drives behind the libata layer. This was (at some point) attempted but unfortunately the code fix had some unexpected side effects and so it was reverted by Doug Gilbert: http://smartmontools.cvs.sourceforge.net/smartmontools/sm5/smartd.cpp?r1=1.386&r2=1.387 I spent a couple of hours hacking and came up with the attached patch to CVS HEAD. WARNING -- UNTESTED! Please don't put this into your distro!! But if you test it on a few machines and it works OK, then I'll consider checking it into CVS. Doug: I think I have fixed this 'properly' (to the degree that this word can ever be applied to smartd!). I did this by combining the ATA and SCSI device lists into a single list, which makes it trivial to retag a 'scsi' device as an 'ata' or 'sat' one. Could you have a look at the diff? I think it should now properly work on IDE, SCSI and SATA. Unfortunately this work was done between 00:00 and 02:00 so I may have done something really dumb. But the idea is pretty simple, I think. Christian: I am looking forward to a C++ rewrite of smartd with lots of data hiding. It will be a much simpler bit of code! Cheers, Bruce On Thu, 28 Feb 2008, Erwan Velu wrote: > As many other distro, we've been switching to the current trunk. > Two users already reports this bug > https://qa.mandriva.com/show_bug.cgi?id=37887 > It means that some users now needs "-d sat" to make smartmontools working. > > I'm proving a default configuration file that do : DEVICESCAN -a -o on > -S on -s L/../../6/03 > Since "-d sat" is needed on some devices, it makes the demon failing to > start. > > Is there any possibility to make smartmontools trying "-d sat" > automatically when the defaut behaviour fails ? > > Erwan, > > > -------------------------------------------------------------------------------- > Les opinions et prises de position emises par le signataire du present > message lui sont propres et ne sauraient engager la responsabilite de la > societe SEANODES. > > Ce message ainsi que les eventuelles pieces jointes constituent une > correspondance privee et confidentielle a l'attention exclusive du > destinataire designe ci-dessus. Si vous n'etes pas le destinataire du > present message ou une personne susceptible de pouvoir le lui delivrer, il > vous est signifie que toute divulgation, distribution ou copie de cette > transmission est strictement interdite. Si vous avez recu ce message par > erreur, nous vous remercions d'en informer l'expediteur par telephone ou de > lui retourner le present message, puis d'effacer immediatement ce message de > votre systeme. > > > The views and opinions expressed by the author of this message are personal. > SEANODES shall assume no liability, express or implied for such message. > > This e-mail and any attachments is a confidential correspondence intended > only for use of the individual or entity named above. If you are not the > intended recipient or the agent responsible for delivering the message to > the intended recipient, you are hereby notified that any disclosure, > distribution or copying of this communication is strictly prohibited. If you > have received this communication in error, please notify the sender by phone > or by replying this message, and then delete this message from your system. > > ------------------------------------------------------------------------- > This SF.net email is sponsored by: Microsoft > Defy all challenges. Microsoft(R) Visual Studio 2008. > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > _______________________________________________ > Smartmontools-support mailing list > Sma...@li... > https://lists.sourceforge.net/lists/listinfo/smartmontools-support > |
From: Erwan V. <er...@se...> - 2008-02-29 10:08:04
|
Bruce Allen wrote: > Hi Erwan, > > There have been a lot of requests to have DEVICESCAN automatically > detect and monitor SATA drives behind the libata layer. This was (at > some point) attempted but unfortunately the code fix had some > unexpected side effects and so it was reverted by Doug Gilbert: > http://smartmontools.cvs.sourceforge.net/smartmontools/sm5/smartd.cpp?r1=1.386&r2=1.387 > > > I spent a couple of hours hacking and came up with the attached patch > to CVS HEAD. WARNING -- UNTESTED! Please don't put this into your > distro!! But if you test it on a few machines and it works OK, then > I'll consider checking it into CVS. I'm asking users that reported this if they want to "beta-test" this patch. I'll keep you on touch with the results. Thanks Bruce, Erwan, -------------------------------------------------------------------------------- Les opinions et prises de position emises par le signataire du present message lui sont propres et ne sauraient engager la responsabilite de la societe SEANODES. Ce message ainsi que les eventuelles pieces jointes constituent une correspondance privee et confidentielle a l'attention exclusive du destinataire designe ci-dessus. Si vous n'etes pas le destinataire du present message ou une personne susceptible de pouvoir le lui delivrer, il vous est signifie que toute divulgation, distribution ou copie de cette transmission est strictement interdite. Si vous avez recu ce message par erreur, nous vous remercions d'en informer l'expediteur par telephone ou de lui retourner le present message, puis d'effacer immediatement ce message de votre systeme. The views and opinions expressed by the author of this message are personal. SEANODES shall assume no liability, express or implied for such message. This e-mail and any attachments is a confidential correspondence intended only for use of the individual or entity named above. If you are not the intended recipient or the agent responsible for delivering the message to the intended recipient, you are hereby notified that any disclosure, distribution or copying of this communication is strictly prohibited. If you have received this communication in error, please notify the sender by phone or by replying this message, and then delete this message from your system. |
From: Bruce A. <ba...@gr...> - 2008-02-29 10:27:13
|
Thanks! On Fri, 29 Feb 2008, Erwan Velu wrote: > Bruce Allen wrote: >> Hi Erwan, >> >> There have been a lot of requests to have DEVICESCAN automatically >> detect and monitor SATA drives behind the libata layer. This was (at >> some point) attempted but unfortunately the code fix had some >> unexpected side effects and so it was reverted by Doug Gilbert: >> http://smartmontools.cvs.sourceforge.net/smartmontools/sm5/smartd.cpp?r1=1.386&r2=1.387 >> >> >> I spent a couple of hours hacking and came up with the attached patch >> to CVS HEAD. WARNING -- UNTESTED! Please don't put this into your >> distro!! But if you test it on a few machines and it works OK, then >> I'll consider checking it into CVS. > I'm asking users that reported this if they want to "beta-test" this > patch. I'll keep you on touch with the results. > Thanks Bruce, > > Erwan, > > -------------------------------------------------------------------------------- > Les opinions et prises de position emises par le signataire du present > message lui sont propres et ne sauraient engager la responsabilite de la > societe SEANODES. > > Ce message ainsi que les eventuelles pieces jointes constituent une > correspondance privee et confidentielle a l'attention exclusive du > destinataire designe ci-dessus. Si vous n'etes pas le destinataire du > present message ou une personne susceptible de pouvoir le lui delivrer, il > vous est signifie que toute divulgation, distribution ou copie de cette > transmission est strictement interdite. Si vous avez recu ce message par > erreur, nous vous remercions d'en informer l'expediteur par telephone ou de > lui retourner le present message, puis d'effacer immediatement ce message de > votre systeme. > > > The views and opinions expressed by the author of this message are personal. > SEANODES shall assume no liability, express or implied for such message. > > This e-mail and any attachments is a confidential correspondence intended > only for use of the individual or entity named above. If you are not the > intended recipient or the agent responsible for delivering the message to > the intended recipient, you are hereby notified that any disclosure, > distribution or copying of this communication is strictly prohibited. If you > have received this communication in error, please notify the sender by phone > or by replying this message, and then delete this message from your system. > > ------------------------------------------------------------------------- > This SF.net email is sponsored by: Microsoft > Defy all challenges. Microsoft(R) Visual Studio 2008. > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > _______________________________________________ > Smartmontools-support mailing list > Sma...@li... > https://lists.sourceforge.net/lists/listinfo/smartmontools-support > |
From: Erwan V. <er...@se...> - 2008-02-29 10:31:37
|
Bruce Allen wrote: > Thanks! Oh by the way, we keep such patch in our version to keep it lsb compliant. Would you mind to get it upstream ? Erwan, --- smartmontools-5.33/smartd.initd.in.lsb 2004-08-31 09:37:07.000000000 +0200 +++ smartmontools-5.33/smartd.initd.in 2006-01-09 03:19:44.000000000 +0100 @@ -13,11 +13,9 @@ ### BEGIN INIT INFO # Provides: smartd # Required-Start: $syslog -# X-UnitedLinux-Should-Start: $sendmail +# Should-Start: $sendmail # Required-Stop: $syslog -# X-UnitedLinux-Should-Stop: # Default-Start: 2 3 5 -# Default-Stop: # Short-Description: Monitors disk and tape health via S.M.A.R.T. # Description: Start S.M.A.R.T. disk and tape monitor. ### END INIT INFO -------------------------------------------------------------------------------- Les opinions et prises de position emises par le signataire du present message lui sont propres et ne sauraient engager la responsabilite de la societe SEANODES. Ce message ainsi que les eventuelles pieces jointes constituent une correspondance privee et confidentielle a l'attention exclusive du destinataire designe ci-dessus. Si vous n'etes pas le destinataire du present message ou une personne susceptible de pouvoir le lui delivrer, il vous est signifie que toute divulgation, distribution ou copie de cette transmission est strictement interdite. Si vous avez recu ce message par erreur, nous vous remercions d'en informer l'expediteur par telephone ou de lui retourner le present message, puis d'effacer immediatement ce message de votre systeme. The views and opinions expressed by the author of this message are personal. SEANODES shall assume no liability, express or implied for such message. This e-mail and any attachments is a confidential correspondence intended only for use of the individual or entity named above. If you are not the intended recipient or the agent responsible for delivering the message to the intended recipient, you are hereby notified that any disclosure, distribution or copying of this communication is strictly prohibited. If you have received this communication in error, please notify the sender by phone or by replying this message, and then delete this message from your system. |
From: Bruce A. <ba...@gr...> - 2008-03-03 14:38:42
|
Hi Erwan, Would it work OK if I just added the +# Should-Start: $sendmail line? I am afraid that removing the other lines might break something somewhere else. PS: do you want CVS write access? I would be happy to add you as a developer. You've been a smartmontools package maintainer for a few years now. Cheers, Bruce On Fri, 29 Feb 2008, Erwan Velu wrote: > Bruce Allen wrote: >> Thanks! > > Oh by the way, we keep such patch in our version to keep it lsb compliant. > Would you mind to get it upstream ? > Erwan, > > --- smartmontools-5.33/smartd.initd.in.lsb 2004-08-31 > 09:37:07.000000000 +0200 > +++ smartmontools-5.33/smartd.initd.in 2006-01-09 03:19:44.000000000 > +0100 > @@ -13,11 +13,9 @@ > ### BEGIN INIT INFO > # Provides: smartd > # Required-Start: $syslog > -# X-UnitedLinux-Should-Start: $sendmail > +# Should-Start: $sendmail > # Required-Stop: $syslog > -# X-UnitedLinux-Should-Stop: > # Default-Start: 2 3 5 > -# Default-Stop: > # Short-Description: Monitors disk and tape health via S.M.A.R.T. > # Description: Start S.M.A.R.T. disk and tape monitor. > ### END INIT INFO > > > -------------------------------------------------------------------------------- > Les opinions et prises de position emises par le signataire du present > message lui sont propres et ne sauraient engager la responsabilite de la > societe SEANODES. > > Ce message ainsi que les eventuelles pieces jointes constituent une > correspondance privee et confidentielle a l'attention exclusive du > destinataire designe ci-dessus. Si vous n'etes pas le destinataire du > present message ou une personne susceptible de pouvoir le lui delivrer, il > vous est signifie que toute divulgation, distribution ou copie de cette > transmission est strictement interdite. Si vous avez recu ce message par > erreur, nous vous remercions d'en informer l'expediteur par telephone ou de > lui retourner le present message, puis d'effacer immediatement ce message de > votre systeme. > > > The views and opinions expressed by the author of this message are personal. > SEANODES shall assume no liability, express or implied for such message. > > This e-mail and any attachments is a confidential correspondence intended > only for use of the individual or entity named above. If you are not the > intended recipient or the agent responsible for delivering the message to > the intended recipient, you are hereby notified that any disclosure, > distribution or copying of this communication is strictly prohibited. If you > have received this communication in error, please notify the sender by phone > or by replying this message, and then delete this message from your system. > > ------------------------------------------------------------------------- > This SF.net email is sponsored by: Microsoft > Defy all challenges. Microsoft(R) Visual Studio 2008. > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > _______________________________________________ > Smartmontools-support mailing list > Sma...@li... > https://lists.sourceforge.net/lists/listinfo/smartmontools-support > |
From: Erwan V. <er...@se...> - 2008-03-03 15:04:20
|
Bruce Allen wrote: > Hi Erwan, > > Would it work OK if I just added the +# Should-Start: $sendmail line? I > am afraid that removing the other lines might break something somewhere > else. > Ok, could be enough. > PS: do you want CVS write access? I would be happy to add you as a > developer. You've been a smartmontools package maintainer for a few years > now. > Thanks, Bruce. I accept it with pleasure ;o) Erwan, -------------------------------------------------------------------------------- Les opinions et prises de position emises par le signataire du present message lui sont propres et ne sauraient engager la responsabilite de la societe SEANODES. Ce message ainsi que les eventuelles pieces jointes constituent une correspondance privee et confidentielle a l'attention exclusive du destinataire designe ci-dessus. Si vous n'etes pas le destinataire du present message ou une personne susceptible de pouvoir le lui delivrer, il vous est signifie que toute divulgation, distribution ou copie de cette transmission est strictement interdite. Si vous avez recu ce message par erreur, nous vous remercions d'en informer l'expediteur par telephone ou de lui retourner le present message, puis d'effacer immediatement ce message de votre systeme. The views and opinions expressed by the author of this message are personal. SEANODES shall assume no liability, express or implied for such message. This e-mail and any attachments is a confidential correspondence intended only for use of the individual or entity named above. If you are not the intended recipient or the agent responsible for delivering the message to the intended recipient, you are hereby notified that any disclosure, distribution or copying of this communication is strictly prohibited. If you have received this communication in error, please notify the sender by phone or by replying this message, and then delete this message from your system. |
From: Erwan V. <er...@se...> - 2008-02-29 10:37:50
|
Bruce Allen wrote: > Hi Erwan, > > There have been a lot of requests to have DEVICESCAN automatically > detect and monitor SATA drives behind the libata layer. This was (at > some point) attempted but unfortunately the code fix had some > unexpected side effects and so it was reverted by Doug Gilbert: > http://smartmontools.cvs.sourceforge.net/smartmontools/sm5/smartd.cpp?r1=1.386&r2=1.387 > > > I spent a couple of hours hacking and came up with the attached patch > to CVS HEAD. WARNING -- UNTESTED! Please don't put this into your > distro!! But if you test it on a few machines and it works OK, then > I'll consider checking it into CVS. Hum, this patches doesn't apply to the 5.38 tag. Is it possible to have patches for this tag ? Thx, Erwan -------------------------------------------------------------------------------- Les opinions et prises de position emises par le signataire du present message lui sont propres et ne sauraient engager la responsabilite de la societe SEANODES. Ce message ainsi que les eventuelles pieces jointes constituent une correspondance privee et confidentielle a l'attention exclusive du destinataire designe ci-dessus. Si vous n'etes pas le destinataire du present message ou une personne susceptible de pouvoir le lui delivrer, il vous est signifie que toute divulgation, distribution ou copie de cette transmission est strictement interdite. Si vous avez recu ce message par erreur, nous vous remercions d'en informer l'expediteur par telephone ou de lui retourner le present message, puis d'effacer immediatement ce message de votre systeme. The views and opinions expressed by the author of this message are personal. SEANODES shall assume no liability, express or implied for such message. This e-mail and any attachments is a confidential correspondence intended only for use of the individual or entity named above. If you are not the intended recipient or the agent responsible for delivering the message to the intended recipient, you are hereby notified that any disclosure, distribution or copying of this communication is strictly prohibited. If you have received this communication in error, please notify the sender by phone or by replying this message, and then delete this message from your system. |
From: Bruce A. <ba...@gr...> - 2008-03-01 08:48:24
|
On Fri, 29 Feb 2008, Erwan Velu wrote: > Bruce Allen wrote: >> Hi Erwan, >> >> There have been a lot of requests to have DEVICESCAN automatically >> detect and monitor SATA drives behind the libata layer. This was (at >> some point) attempted but unfortunately the code fix had some >> unexpected side effects and so it was reverted by Doug Gilbert: >> http://smartmontools.cvs.sourceforge.net/smartmontools/sm5/smartd.cpp?r1=1.386&r2=1.387 >> >> >> I spent a couple of hours hacking and came up with the attached patch >> to CVS HEAD. WARNING -- UNTESTED! Please don't put this into your >> distro!! But if you test it on a few machines and it works OK, then >> I'll consider checking it into CVS. > Hum, this patches doesn't apply to the 5.38 tag. > Is it possible to have patches for this tag ? I think it was a patch off CVS HEAD. I will try to test it as soon as possible, and fix it if needed. Did you reproduce the problem youself? Cheers, Bruce |
From: Erwan <er...@se...> - 2008-03-01 10:26:29
|
Bruce Allen wrote: > [..] > I think it was a patch off CVS HEAD. I will try to test it as soon as > possible, and fix it if needed. Did you reproduce the problem youself? > I didn't myself but I have several bug reporters ready to test a "beta" build of smartmontools + patch. As they did report the failure, I'm pretty sure they'll report success or comments ;o) Erwan -------------------------------------------------------------------------------- Les opinions et prises de position emises par le signataire du present message lui sont propres et ne sauraient engager la responsabilite de la societe SEANODES. Ce message ainsi que les eventuelles pieces jointes constituent une correspondance privee et confidentielle a l'attention exclusive du destinataire designe ci-dessus. Si vous n'etes pas le destinataire du present message ou une personne susceptible de pouvoir le lui delivrer, il vous est signifie que toute divulgation, distribution ou copie de cette transmission est strictement interdite. Si vous avez recu ce message par erreur, nous vous remercions d'en informer l'expediteur par telephone ou de lui retourner le present message, puis d'effacer immediatement ce message de votre systeme. The views and opinions expressed by the author of this message are personal. SEANODES shall assume no liability, express or implied for such message. This e-mail and any attachments is a confidential correspondence intended only for use of the individual or entity named above. If you are not the intended recipient or the agent responsible for delivering the message to the intended recipient, you are hereby notified that any disclosure, distribution or copying of this communication is strictly prohibited. If you have received this communication in error, please notify the sender by phone or by replying this message, and then delete this message from your system. |
From: Bruce A. <ba...@gr...> - 2008-03-03 13:43:00
|
Hi Erwan, >> I think it was a patch off CVS HEAD. I will try to test it as soon as >> possible, and fix it if needed. Did you reproduce the problem youself? > I didn't myself but I have several bug reporters ready to test a "beta" > build of smartmontools + patch. As they did report the failure, I'm > pretty sure they'll report success or comments ;o) I can not reproduce the failure of the patches to apply cleanly to the smartmontools CVS. If I understand correctly, the users were reporting failure to patch the files properly. I just had a user report that they could do the patching, and also I just did this successfully: cvs co sm5 cd sm5 patch < patch1 patch < patch2 If the patches did not apply cleanly, could you please try to reproduce this and send me the error transcript? It would also be helpful to have the output of the command: 'ident smartd.cpp smartd.h'. Or get these from the users please! Cheers, Bruce > > Erwan > > -------------------------------------------------------------------------------- > Les opinions et prises de position emises par le signataire du present > message lui sont propres et ne sauraient engager la responsabilite de la > societe SEANODES. > > Ce message ainsi que les eventuelles pieces jointes constituent une > correspondance privee et confidentielle a l'attention exclusive du > destinataire designe ci-dessus. Si vous n'etes pas le destinataire du > present message ou une personne susceptible de pouvoir le lui delivrer, il > vous est signifie que toute divulgation, distribution ou copie de cette > transmission est strictement interdite. Si vous avez recu ce message par > erreur, nous vous remercions d'en informer l'expediteur par telephone ou de > lui retourner le present message, puis d'effacer immediatement ce message de > votre systeme. > > > The views and opinions expressed by the author of this message are personal. > SEANODES shall assume no liability, express or implied for such message. > > This e-mail and any attachments is a confidential correspondence intended > only for use of the individual or entity named above. If you are not the > intended recipient or the agent responsible for delivering the message to > the intended recipient, you are hereby notified that any disclosure, > distribution or copying of this communication is strictly prohibited. If you > have received this communication in error, please notify the sender by phone > or by replying this message, and then delete this message from your system. > > ------------------------------------------------------------------------- > This SF.net email is sponsored by: Microsoft > Defy all challenges. Microsoft(R) Visual Studio 2008. > http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ > _______________________________________________ > Smartmontools-support mailing list > Sma...@li... > https://lists.sourceforge.net/lists/listinfo/smartmontools-support > |