From owner-freebsd-stable@FreeBSD.ORG Wed Nov 24 21:20:44 2010 Return-Path: Delivered-To: stable@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 849801065672; Wed, 24 Nov 2010 21:20:44 +0000 (UTC) (envelope-from uqs@spoerlein.net) Received: from acme.spoerlein.net (acme.spoerlein.net [IPv6:2a01:4f8:131:23c2::1]) by mx1.freebsd.org (Postfix) with ESMTP id 15B428FC0A; Wed, 24 Nov 2010 21:20:43 +0000 (UTC) Received: from localhost (acme.spoerlein.net [IPv6:2a01:4f8:131:23c2::1]) by acme.spoerlein.net (8.14.4/8.14.4) with ESMTP id oAOLKdf9096710 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 24 Nov 2010 22:20:39 +0100 (CET) (envelope-from uqs@spoerlein.net) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=spoerlein.net; s=dkim200908; t=1290633639; bh=dM/UKjvnwDU30mnnmQo9m0TKsHzllPHJEnz4w5iRtXM=; h=Date:From:To:Cc:Subject:Message-ID:Reply-To:References: MIME-Version:Content-Type:In-Reply-To; b=q46rUxWezYoUX0gz9MRcOR+SsIslRZe3nnJykohdlQfEk3yA3HhwUX3Z+L2W3jyy7 XuKEhvyLWQKiJqaSHPeXWfACbeLzPagCSRmyGUsb7TNJO0EkW4k5ufhfSnxKB6r6Tk A8qcrZJNT5O8sHR2z4N0psJR9JlB2D6ovcQL7Doo= Date: Wed, 24 Nov 2010 22:20:39 +0100 From: Ulrich =?utf-8?B?U3DDtnJsZWlu?= To: Peter Jeremy Message-ID: <20101124212039.GL3120@acme.spoerlein.net> Mail-Followup-To: Peter Jeremy , freebsd-arch@freebsd.org, stable@FreeBSD.org References: <20101118222100.GA47116@server.vk2pj.dyndns.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20101118222100.GA47116@server.vk2pj.dyndns.org> User-Agent: Mutt/1.5.21 (2010-09-15) Cc: stable@freebsd.org, freebsd-arch@freebsd.org Subject: Are there digi(4) users running -STABLE? (was Re: Migrating ISA/PCI drivers) X-BeenThere: freebsd-stable@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: stable@freebsd.org List-Id: Production branch of FreeBSD source code List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 24 Nov 2010 21:20:44 -0000 [cross-posting to stable@, where some of those folk might hang out] On Fri, 19.11.2010 at 09:21:00 +1100, Peter Jeremy wrote: > I'm (belatedly) looking at porting digi(4) to the MPSAFE TTY system > and have some architectural questions. > > The digi(4) driver appears to support 5 different Digi card variants, > at least two of which exist in both ISA and PCI variants. Looking at > the Digi website, it appears that both PCI and ISA cards are still > available (as well as a PCIe card which is unlikely to work with the > current driver). I only have access to PCI/Xem cards and so can't > test my changes on any other card types. I presume Digi cards are not > that popular because noone else has shown any interest in the driver > since the MPSAFE TTY changes were announced about 2.5 years ago. > > How much effort should I invest in adapting code for other card types? > In particular, the ISA cards use windowed memory accesses and IO ports > where the PCI cards have a single flat memory aperture. Removing > support for ISA cards would simplify the code (and remove the need to > make decisions about whether I need to do window switches in new > code), as well as potentially allowing finer grained locks. > > My options would seem to be: > 1) Rip out the ISA support - this is the cleanest for me but maximises > effort for a future person wanting to support ISA Digi cards. > 2) Carry forward the ISA code as best I can and ensure new code includes > appropriate window switches etc. This maximises my effort but > hopefully makes it easier for someone to get ISA cards working. > 3) Ignore the ISA code. This is fairly easy but probably requires > similar effort to (1) to get it going on ISA since all the code > would need to be reviewed to add necessary ISA-specific locking/ > window switching. > > My preference is 1 since it leaves the least cruft (from my point of > view) in the code and doesn't give users the false impression that > ISA cards work. > > I would appreciate some advice on the best way forward. In the > absence of any input, I will probably stick with option 3 for now but > may move to option 1 if the ISA code starts getting in the way. Keep > in mind that digi(4) does not currently compile on FreeBSD-8 or later, > so any of the above options are an improvement over the status quo, > though all are a regression from FreeBSD-7. > > Of course, if someone has access to other Digi card types and wants > to assist with porting and testing, I would be happy to work with them. > > -- > Peter Jeremy