On 11-04-21 10:34, Arnaud Quette wrote:
Hi Lupe,
since we now have an apcsmart maintainer, I'm forwarding this issue to him.
@Michal: could you please have a look at this issue [1], and give us
your feeling?
cheers,
Arnaud
--
[1] https://bugs.launchpad.net/bugs/535583
2011/2/15 Lupe Christoph <[email protected]
<mailto:[email protected]>>
The suggestions are pretty fine.
- flushing stale input (though at driver level)
Certainly. I even added some flushes earlier, but haven't touched the
updateinfo and/or the functions it calls yet. I'll add it along with
forthcoming patches (icanon mode and the rest). Looking at the strace,
flushing post-failure might be good idea in certain cases as well.
- reopening serial port
If the upper layers of nut don't disallow this kind of behaviour for
some reason - it's good idea as well. Should be helpful in weird cases,
and at the very least wouldn't hurt at all. If it would help in this
particular case, hard to say.
- smartmode()
TBH, I'm not sure why it diligently tries to enter SM 5 times.
Pre-emptive flush + 'Y' + reasonable delay (icanon or not) should be all
that is necessary. If we don't succeed, next attempt shouldn't
miraculously (in theory) make much of a difference 1 second later ...
Thanks for pointing out those issues.
_______________________________________________
Nut-upsdev mailing list
[email protected]
http://lists.alioth.debian.org/mailman/listinfo/nut-upsdev