From owner-freebsd-arch@FreeBSD.ORG  Sun Sep 28 22:09:37 2008
Return-Path: <owner-freebsd-arch@FreeBSD.ORG>
Delivered-To: freebsd-arch@FreeBSD.org
Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34])
	by hub.freebsd.org (Postfix) with ESMTP id E62071065694;
	Sun, 28 Sep 2008 22:09:37 +0000 (UTC) (envelope-from imp@bsdimp.com)
Received: from harmony.bsdimp.com (bsdimp.com [199.45.160.85])
	by mx1.freebsd.org (Postfix) with ESMTP id A48128FC17;
	Sun, 28 Sep 2008 22:09:37 +0000 (UTC) (envelope-from imp@bsdimp.com)
Received: from localhost (localhost [127.0.0.1])
	by harmony.bsdimp.com (8.14.2/8.14.1) with ESMTP id m8SM9KnT090913;
	Sun, 28 Sep 2008 16:09:20 -0600 (MDT) (envelope-from imp@bsdimp.com)
Date: Sun, 28 Sep 2008 16:10:10 -0600 (MDT)
Message-Id: <20080928.161010.1649769915.imp@bsdimp.com>
To: alfred@FreeBSD.org
From: "M. Warner Losh" <imp@bsdimp.com>
In-Reply-To: <20080928201948.GE36572@elvis.mu.org>
References: <20080928100731.GA49323@onelab2.iet.unipi.it>
	<20080928.135855.1708680935.imp@bsdimp.com>
	<20080928201948.GE36572@elvis.mu.org>
X-Mailer: Mew version 5.2 on Emacs 21.3 / Mule 5.0 (SAKAKI)
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Cc: freebsd-arch@FreeBSD.org
Subject: Re: dynamic update of usb/pci/quirks tables
X-BeenThere: freebsd-arch@freebsd.org
X-Mailman-Version: 2.1.5
Precedence: list
List-Id: Discussion related to FreeBSD architecture <freebsd-arch.freebsd.org>
List-Unsubscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-arch>,
	<mailto:freebsd-arch-request@freebsd.org?subject=unsubscribe>
List-Archive: <http://lists.freebsd.org/pipermail/freebsd-arch>
List-Post: <mailto:freebsd-arch@freebsd.org>
List-Help: <mailto:freebsd-arch-request@freebsd.org?subject=help>
List-Subscribe: <http://lists.freebsd.org/mailman/listinfo/freebsd-arch>,
	<mailto:freebsd-arch-request@freebsd.org?subject=subscribe>
X-List-Received-Date: Sun, 28 Sep 2008 22:09:38 -0000

In message: <20080928201948.GE36572@elvis.mu.org>
            Alfred Perlstein <alfred@freebsd.org> writes:
: * M. Warner Losh <imp@bsdimp.com> [080928 13:01] wrote:
: > uscanner_devs and sis_devs aren't quirks.  They are device tables.
: > 
: > You've repeated ignored the mapping idea that I've posted.  You can't
: > just add stuff to tables randomly and expect that to work (says
: > someone who has actually done this to lots of drivers in the tree in
: > the pccard era).  the driver has to know what kind of device to treat
: > it as.
: > 
: > Putting a translation table into the kernel is much easier and you
: > don't have to worry about hokey kludges like what you describe with
: > 'patch'.
: > 
: > Maybe it will work out for the other tables you want to update, but it
: > won't work well for device tables.
: 
: I really like the idea of using a kmod to just add the new device
: strings.. (some form of what Hans did).

The problem is that except for the most trivial driver, that doesn't
work.  Most of the NIC drivers in the tree do special things based on
what chip they thing they are talking to.  An unknown chip may work,
but it would work a lot better if the unknown chip is compatible with
a specific chip, as opposed to being compatible with the driver.

I'm cool with having the overriden ids also override the device
description too...

Warner