Messages in this thread | | | Date | Mon, 4 Aug 2003 14:28:10 +0200 (CEST) | From | Thomas Horsten <> | Subject | Re: [PATCH] (2.4.2x) Driver for Medley software RAID (Silicon Image 3112 SATARaid, CMD680, etc?) for testing |
| |
On 4 Aug 2003, Arjan van de Ven wrote:
> > I was under the impression that silraid.c never worked at all (it > > certainly didn't on my system), because the magic number it looks for is > > incorrect (actually, part of the drive's serial number), and thus not > > present on all systems with this RAID). > > well... that's a 1 line fix to silraid.c......
Not really, it needs to look up the PCI device ID of the card to do it (of course still just a few lines).
But the reason for rewriting the driver was that I wanted to use the specification as a guideline instead of the old implementation which was really for a different RAID type (with the cutoff handling of the differing disk sizes etc).
In hindsight, after the driver is "completed" at least for RAID0, I can see that modifying the old driver could have been just as good (although I'd still advocate renaming it), but now it is done like this and I think the new driver is certainly a better starting point for Medley (e.g. if one wanted to add support for the striped modes etc).
Keep in mind that I didn't know anyone had successfully used silraid.c since on my system it a) looked in the wrong place for the superblock and b) looked for a wrong magic number, and I didn't get any response when I provided you with a lot of details about this.
The detection of the arrays is certainly safer, and the make request routine is improved in my driver so bottom line I thik it's a better driver and will also provide a better starting point for adding stuff like RAID1 and RAID0+1 support (although one could argue that the whole ataraid method should be regarded as a temporary solution and there would be ways to integrate support for these software RAIDs better in the future).
Regards, Thomas
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |