From owner-freebsd-arch@FreeBSD.ORG Sun Sep 28 20:00:55 2008 Return-Path: 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 63CB5106569D for ; Sun, 28 Sep 2008 20:00:54 +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 40D788FC17 for ; Sun, 28 Sep 2008 20:00:54 +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 m8SJw3e1089581; Sun, 28 Sep 2008 13:58:06 -0600 (MDT) (envelope-from imp@bsdimp.com) Date: Sun, 28 Sep 2008 13:58:55 -0600 (MDT) Message-Id: <20080928.135855.1708680935.imp@bsdimp.com> To: rizzo@iet.unipi.it From: "M. Warner Losh" In-Reply-To: <20080928100731.GA49323@onelab2.iet.unipi.it> References: <20080928100731.GA49323@onelab2.iet.unipi.it> 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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 28 Sep 2008 20:00:55 -0000 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. Warner