Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 16 Oct 2008 13:15:46 +0400
From:      "Andrey V. Elsukov" <bu7cher@yandex.ru>
To:        Jeremy Chadwick <koitsu@FreeBSD.org>
Cc:        kib@freebsd.org, freebsd-stable@freebsd.org, sos@freebsd.org
Subject:   Re: Request for testing: ata(4) MFC
Message-ID:  <48F70642.7080701@yandex.ru>
In-Reply-To: <20081016071700.GA2793@icarus.home.lan>
References:  <676151223134689@webmail38.yandex.ru> <20081005004808.GA70137@icarus.home.lan> <48E99C18.6070602@yandex.ru> <20081006051211.GA10542@icarus.home.lan> <20081010115855.GA31707@icarus.home.lan> <20081016071700.GA2793@icarus.home.lan>

next in thread | previous in thread | raw e-mail | index | archive | help
Jeremy Chadwick wrote:
> - Could someone provide an explanation of the 48-bit LBA addressing
>   changes (see lines 988-1003 in the patch)?  I'd like to know what they
>   do, and if further QA/testing is needed with this.

I think this is for debug purposes.

> - Can we please see about adding the FreeNAS project's ata timeout
>   sysctls?  I see lots of delays/sleeps in numerous pieces of code that
>   pertain to soft or hard resets of AHCI controllers, and I often worry
>   about the implications of hard-coded timeouts.
> 
>   http://freenas.svn.sourceforge.net/viewvc/freenas/trunk/build/kernel-patches/ata/files/patch-ata.diff?view=markup

As i remember changes in FreeNAS don't change timeouts for resets.
In any case, these changes can not be in 7.1-RELEASE. My patch
targeted to move changes from CURRENT to 7.1. But it seems there
are too few testers and patch can not be commited.

-- 
WBR, Andrey V. Elsukov



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?48F70642.7080701>