From owner-freebsd-arch@FreeBSD.ORG Mon Jun 16 10:06:21 2003 Return-Path: Delivered-To: freebsd-arch@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id A634037B401 for ; Mon, 16 Jun 2003 10:06:21 -0700 (PDT) Received: from mail.cyberonic.com (mail.cyberonic.com [4.17.179.4]) by mx1.FreeBSD.org (Postfix) with ESMTP id 7CC0B43F85 for ; Mon, 16 Jun 2003 10:06:20 -0700 (PDT) (envelope-from jmg@hydrogen.funkthat.com) Received: from hydrogen.funkthat.com (node-40244c0a.sfo.onnet.us.uu.net [64.36.76.10]) by mail.cyberonic.com (8.12.8/8.12.5) with ESMTP id h5GHVpMo031881; Mon, 16 Jun 2003 13:31:52 -0400 Received: (from jmg@localhost) by hydrogen.funkthat.com (8.12.9/8.11.6) id h5GH6jM1009680; Mon, 16 Jun 2003 10:06:45 -0700 (PDT) (envelope-from jmg) Date: Mon, 16 Jun 2003 10:06:45 -0700 From: John-Mark Gurney To: Bruce Evans Message-ID: <20030616170645.GI73854@funkthat.com> Mail-Followup-To: Bruce Evans , arch@freebsd.org References: <20030616074122.GF73854@funkthat.com> <20030616193932.X27844@gamplex.bde.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20030616193932.X27844@gamplex.bde.org> User-Agent: Mutt/1.4.1i X-Operating-System: FreeBSD 4.2-RELEASE i386 X-PGP-Fingerprint: B7 EC EF F8 AE ED A7 31 96 7A 22 B3 D8 56 36 F4 X-Files: The truth is out there X-URL: http://resnet.uoregon.edu/~gurney_j/ X-Resume: http://resnet.uoregon.edu/~gurney_j/resume.html cc: arch@freebsd.org Subject: Re: make /dev/pci really readable X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list Reply-To: John-Mark Gurney List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 16 Jun 2003 17:06:21 -0000 Bruce Evans wrote this message on Mon, Jun 16, 2003 at 19:42 +1000: > On Mon, 16 Jun 2003, John-Mark Gurney wrote: > > > Does anyone have an objection to making /dev/pci really honor the > > permissions, and giving normal users (or just group wheel) premission > > to run pciconf -l. Right now the code requires the write bit set for > > any operation. > > IIRC, it is like it is because reading it may have side effects (and > thus isn't really just reading). If it honored the permissions then > it should have mode 600 so that normal users can't run pciconf -l :-]. Now if we were reading the pci registers with -r, then yes, but -l just copys the data from pci_devinfo. If we wanted to make -r readable, we'd have to clamp the registers passed in, and make sure that all platforms didn't trap on PCI register reads (a patch for sparc should be going in soon). -- John-Mark Gurney Voice: +1 415 225 5579 "All that I will do, has been done, All that I have, has not."