Re: [PATCH 4/5] pata: Update experimental tags

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 11/19/2009 04:49 PM, Bartlomiej Zolnierkiewicz wrote:
On Thursday 19 November 2009 22:38:39 Jeff Garzik wrote:
On 11/18/2009 02:56 PM, Bartlomiej Zolnierkiewicz wrote:
On Wednesday 18 November 2009 20:07:07 Bartlomiej Zolnierkiewicz wrote:
On Wednesday 18 November 2009 19:41:25 Alan Cox wrote:
Maybe they are 'stable' but when it comes to features they are behind hpt366
(i.e. they lack PCI PM), which is also much cleaner than your drivers, easier
to understand and much smaller..

37x and 3xn lack PCI PM.  Added to the TODO list.

The smaller size is a bit questionable given its mostly comments, and
three drivers versus one.

I wonder what you find 'questionable' in the numbers given..

BTW we can immediately reclaim 300 LOC by simply merging drivers back..

Merging drivers is a big pain for distributions and end users, even with
the new module_alias facility.

Could please explain in more detail 'a big pain' part?

It makes issue tracking (bug reports, feature requests, etc.) more difficult. The "where did my driver go?" issue, which is largely a communication/education not technical problem. Manually specifying drivers in /etc/modprobe.conf remains supported.


Absent stronger justification, libata remains with separate drivers,
even if that means some code duplication.

I can always hold my own tree with such changes if needed.

If it really bugs you, you could take the drivers/ata/sata_promise.h approach, or do multi-file modules.

	Jeff



--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux