From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S262530AbULOXQP (ORCPT ); Wed, 15 Dec 2004 18:16:15 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S262529AbULOXQO (ORCPT ); Wed, 15 Dec 2004 18:16:14 -0500 Received: from omx2-ext.sgi.com ([192.48.171.19]:56282 "EHLO omx2.sgi.com") by vger.kernel.org with ESMTP id S262527AbULOXQJ (ORCPT ); Wed, 15 Dec 2004 18:16:09 -0500 From: Jesse Barnes To: Andreas Dilger Subject: Re: [PATCH] add legacy I/O and memory access routines to /proc/bus/pci API Date: Wed, 15 Dec 2004 15:15:51 -0800 User-Agent: KMail/1.7.1 Cc: linux-pci@atrey.karlin.mff.cu, linux-ia64@vger.kernel.org, linux-kernel@vger.kernel.org, benh@kernel.crashing.org, bjorn.helgaas@hp.com References: <200412140941.56116.jbarnes@engr.sgi.com> <200412150927.51733.jbarnes@engr.sgi.com> <20041215210346.GK9923@schnapps.adilger.int> In-Reply-To: <20041215210346.GK9923@schnapps.adilger.int> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200412151515.51450.jbarnes@engr.sgi.com> Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Wednesday, December 15, 2004 1:03 pm, Andreas Dilger wrote: > On Dec 15, 2004 09:27 -0800, Jesse Barnes wrote: > > +write > > + Legacy I/O port space reads and writes must also be to a file > > + position >64k--the kernel will route them to the target device. > > Shouldn't that be < 64k based on the description of lseek? Err.. yes > > +lseek > > + Can be used to set the current file position. Note that the file > > + size is limited to 64k as that's how big legacy I/O space is. > > > > +ioctl > > + Note that not all architectures support the *_MMAP_* or *_RW_* ioctl > > + commands. If they're not supported, ioctl will return -EINVAL. > > Shouldn't they return -ENOTTY? That indicates to the caller that the > ioctl isn't handled, vs -EINVAL which indicates bad value being passed > (e.g. bad write size). Maybe, but that's not how /proc/bus/pci behaves right now... Jesse