PR/59130 CVS commit: [netbsd-9] src/sys/dev/ata

["Martin Husemann" <martin%netbsd.org@localhost>]

The following reply was made to PR kern/59130; it has been noted by GNATS.

From: "Martin Husemann" <martin%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/59130 CVS commit: [netbsd-9] src/sys/dev/ata
Date: Wed, 7 May 2025 17:39:03 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Wed May  7 17:39:03 UTC 2025
 
 Modified Files:
 	src/sys/dev/ata [netbsd-9]: ata_raid_via.c
 
 Log Message:
 Pull up following revision(s) (requested by andvar in ticket #1942):
 
 	sys/dev/ata/ata_raid_via.c: revision 1.11
 
 ataraid(4): obtain VIA V-RAID drive index from configuration blocks.
 
 The original logic resolved disk index based on the drive channel it was
 attached to. However, VX800 and some other VIA controllers have two SATA ports
 per channel, which causes unintended overrides when two disks share the same
 channel.
 
 VIA V-RAID configuration blocks store the disk index as a value incrementing by
 0x04 (0x00, 0x04, 0x08, 0x0C). Therefore, shift the value left by 2 obtains the
 disk number.
 
 Additionally, limit disk number to 4, which is the maximum supported by any
 VIA RAID configuration.
 
 Fixes PR kern/59130.
 
 Tested on VX800/VT6421A/VT8251 RAID controllers.
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.9 -r1.9.4.1 src/sys/dev/ata/ata_raid_via.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.