[SeaBIOS] [Qemu-devel] What's the impact of enlarging IDE_TIMEOUT ?

[Fred .]

Sounds like something that should be commented in the source code.

IDE_TIMEOUT = 32; // 30 seconds per ATA specification +2 seconds thrown in
for safety


On Sat, Jun 1, 2013 at 3:21 PM, Kevin O'Connor <kevin at koconnor.net> wrote:

> On Fri, May 31, 2013 at 03:18:56AM +0000, Gonglei (Arei) wrote:
> >    IDE_TIMEOUT is defined 32s. But we encountered its timeout in
> >    some cases, and then loading disk failed in VM.  In order to
> >    reduce the probability of timeout, we want to enlarge the
> >    IDE_TIMEOUT, such as 120s.  We verified this modification worked
> >    for us. But we are wondering if this modification may cause other
> >    potential issues.  Why set IDE_TIMEOUT to 32s? Thanks!
>
> The ATA specification specifies the maximum response time for many
> commands to be 30 seconds.  The extra 2 seconds were thrown in for
> safety.
>
> -Kevin
>
> _______________________________________________
> SeaBIOS mailing list
> SeaBIOS at seabios.org
> http://www.seabios.org/mailman/listinfo/seabios
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.seabios.org/pipermail/seabios/attachments/20130601/b6380446/attachment.html>