Re: Request for testing: ata(4) MFC

2008-10-17 Thread Ruben van Staveren
Hi, On 17 Oct 2008, at 1:15, Steven Hartland wrote: You must be very careful with using less than 48bit addressing when both the drive and the controller supports it as some disks report errors at none standard crossover points. I saw this first hand with the highpoint driver where it

Re: Request for testing: ata(4) MFC

2008-10-17 Thread Steven Hartland
Highly likely, as it was seagate disks I had this issue with. The comment in the hptmv driver:- + /* +* always use 48bit LBA if drive supports it. +* Some Seagate drives report error if you use a 28-bit command +* to access sector 0xfff. +*/ - Original Message -

Re: Request for testing: ata(4) MFC

2008-10-16 Thread Jeremy Chadwick
On Fri, Oct 10, 2008 at 04:58:55AM -0700, Jeremy Chadwick wrote: On Sun, Oct 05, 2008 at 10:12:11PM -0700, Jeremy Chadwick wrote: On Mon, Oct 06, 2008 at 09:03:20AM +0400, Andrey V. Elsukov wrote: Jeremy Chadwick wrote: Also, does your patch include any fixes (intentional or inadvertent)

Re: Request for testing: ata(4) MFC

2008-10-16 Thread Jeremy Chadwick
On Thu, Oct 16, 2008 at 01:15:46PM +0400, Andrey V. Elsukov wrote: - 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

Re: Request for testing: ata(4) MFC

2008-10-16 Thread Ruben van Staveren
I'm getting this on FreeBSD/amd64 with the ata patch. It also blew one part of the gmirror the system is running on. The machine is a Sun Fire X2100-M2 with 4gb memory, running from a RELENG_7 csup of a couple of hours ago. [...] atapci1: nVidia nForce MCP55 SATA300 controller port

Re: Request for testing: ata(4) MFC

2008-10-16 Thread Andrey V. Elsukov
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

Re: Request for testing: ata(4) MFC

2008-10-16 Thread Goran Lowkrantz
It seems the attachment got eaten somewhere. Trying again with correct content type. /glz --On October 16, 2008 12:20:27 +0200 Goran Lowkrantz [EMAIL PROTECTED] wrote: Ok, I was not sure if this was a No news is good news request. Attached are the lspci:s of the MB and devices I have had

Re: Request for testing: ata(4) MFC

2008-10-16 Thread Jeremy Chadwick
On Thu, Oct 16, 2008 at 12:17:00AM -0700, Jeremy Chadwick wrote: On Fri, Oct 10, 2008 at 04:58:55AM -0700, Jeremy Chadwick wrote: On Sun, Oct 05, 2008 at 10:12:11PM -0700, Jeremy Chadwick wrote: On Mon, Oct 06, 2008 at 09:03:20AM +0400, Andrey V. Elsukov wrote: Jeremy Chadwick wrote:

Re: Request for testing: ata(4) MFC

2008-10-16 Thread Jeremy Chadwick
On Thu, Oct 16, 2008 at 10:45:25AM -0700, Jeremy Chadwick wrote: On Thu, Oct 16, 2008 at 12:17:00AM -0700, Jeremy Chadwick wrote: On Fri, Oct 10, 2008 at 04:58:55AM -0700, Jeremy Chadwick wrote: On Sun, Oct 05, 2008 at 10:12:11PM -0700, Jeremy Chadwick wrote: On Mon, Oct 06, 2008 at

Re: Request for testing: ata(4) MFC

2008-10-16 Thread Antony Mawer
Jeremy Chadwick wrote: On Fri, Oct 10, 2008 at 04:58:55AM -0700, Jeremy Chadwick wrote: On Sun, Oct 05, 2008 at 10:12:11PM -0700, Jeremy Chadwick wrote: On Mon, Oct 06, 2008 at 09:03:20AM +0400, Andrey V. Elsukov wrote: Jeremy Chadwick wrote: Also, does your patch include any fixes

Re: Request for testing: ata(4) MFC

2008-10-16 Thread Steven Hartland
You must be very careful with using less than 48bit addressing when both the drive and the controller supports it as some disks report errors at none standard crossover points. I saw this first hand with the highpoint driver where it totally trashed the RAID volumes. The solution was to always

Re: Request for testing: ata(4) MFC

2008-10-12 Thread Søren Schmidt
On 10Oct, 2008, at 13:58 , Jeremy Chadwick wrote: On Sun, Oct 05, 2008 at 10:12:11PM -0700, Jeremy Chadwick wrote: On Mon, Oct 06, 2008 at 09:03:20AM +0400, Andrey V. Elsukov wrote: Jeremy Chadwick wrote: Also, does your patch include any fixes (intentional or inadvertent) for Intel

Re: Request for testing: ata(4) MFC

2008-10-12 Thread Andrey V. Elsukov
Søren Schmidt wrote: I'm not sure how far this has gone into 7 yet, but it would be a real cool thing(tm) to have the latest ATA module work back into 7.1 as well. Its a no brainer actually with no functional changes other than the possibility to load chipset specific code as modules. I know

Re: Request for testing: ata(4) MFC

2008-10-10 Thread Jeremy Chadwick
On Sun, Oct 05, 2008 at 10:12:11PM -0700, Jeremy Chadwick wrote: On Mon, Oct 06, 2008 at 09:03:20AM +0400, Andrey V. Elsukov wrote: Jeremy Chadwick wrote: Also, does your patch include any fixes (intentional or inadvertent) for Intel MatrixRAID? This has been a sore spot for FreeBSD for

Re: Request for testing: ata(4) MFC

2008-10-05 Thread Goran Lowkrantz
--On Saturday, October 04, 2008 19:38 +0400 Andrey V. Elsukov [EMAIL PROTECTED] wrote: Hi, All. I prepared patch to make MFC of ata(4) driver into RELENG_7 before 7.1-RELEASE. Depending on results of the testing patch will be commited or not (if some regressions will be detected). So if you

Re: Request for testing: ata(4) MFC

2008-10-05 Thread Søren Schmidt
On 5Oct, 2008, at 13:04 , Marius Strobl wrote: On Sat, Oct 04, 2008 at 07:38:09PM +0400, Andrey V. Elsukov wrote: Hi, All. I prepared patch to make MFC of ata(4) driver into RELENG_7 before 7.1-RELEASE. Depending on results of the testing patch will be commited or not (if some regressions

Re: Request for testing: ata(4) MFC

2008-10-05 Thread Marius Strobl
On Sat, Oct 04, 2008 at 07:38:09PM +0400, Andrey V. Elsukov wrote: Hi, All. I prepared patch to make MFC of ata(4) driver into RELENG_7 before 7.1-RELEASE. Depending on results of the testing patch will be commited or not (if some regressions will be detected). So if you want or just can

Re: Request for testing: ata(4) MFC

2008-10-05 Thread Andrey V. Elsukov
Jeremy Chadwick wrote: Also, does your patch include any fixes (intentional or inadvertent) for Intel MatrixRAID? This has been a sore spot for FreeBSD for quite some time, and I'm curious to know if that has been fixed. There is only one fix for Intel Matrix RAID:

Re: Request for testing: ata(4) MFC

2008-10-05 Thread Jeremy Chadwick
On Mon, Oct 06, 2008 at 09:03:20AM +0400, Andrey V. Elsukov wrote: Jeremy Chadwick wrote: Also, does your patch include any fixes (intentional or inadvertent) for Intel MatrixRAID? This has been a sore spot for FreeBSD for quite some time, and I'm curious to know if that has been fixed.

Re: Request for testing: ata(4) MFC

2008-10-04 Thread Jeremy Chadwick
On Sat, Oct 04, 2008 at 07:38:09PM +0400, Andrey V. Elsukov wrote: I prepared patch to make MFC of ata(4) driver into RELENG_7 before 7.1-RELEASE. Depending on results of the testing patch will be commited or not (if some regressions will be detected). So if you want or just can test it,