From owner-freebsd-current@FreeBSD.ORG Thu Nov 8 10:40:31 2007 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id C4EC916A468 for ; Thu, 8 Nov 2007 10:40:31 +0000 (UTC) (envelope-from screwdriver@lxnt.info) Received: from mail.lxnt.info (mail.lxnt.info [217.23.143.142]) by mx1.freebsd.org (Postfix) with ESMTP id 72B2A13C4C1 for ; Thu, 8 Nov 2007 10:40:31 +0000 (UTC) (envelope-from screwdriver@lxnt.info) Received: from [217.23.131.8] (helo=lxnt.inside.caravan.ru) by mail.lxnt.info with esmtpsa (TLSv1:DHE-RSA-AES256-SHA:256) (Exim 4.68 (FreeBSD)) (envelope-from ) id 1Iq4oC-000BpP-5H; Thu, 08 Nov 2007 13:40:24 +0300 Message-ID: <4732E7C2.3030403@lxnt.info> Date: Thu, 08 Nov 2007 13:41:06 +0300 From: Alexander Sabourenkov User-Agent: Thunderbird 2.0.0.6 (X11/20071024) MIME-Version: 1.0 To: =?UTF-8?B?U8O4cmVuIFNjaG1pZHQ=?= References: <47326FB8.50602@fusiongol.com> <4732CEE3.3070003@lxnt.info> <4732DA32.3090601@deepcore.dk> <4732E18A.6040802@lxnt.info> <4732E37F.9020707@deepcore.dk> In-Reply-To: <4732E37F.9020707@deepcore.dk> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Cc: Nathan Butcher , freebsd-current@freebsd.org Subject: Re: Remaining SATA (and other) issues X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 08 Nov 2007 10:40:31 -0000 Søren Schmidt wrote: > As I said ATA (as in my driver) uses this promise general purpose > register to store interrupts into, it is very much needed. > The reason is that on some promise chips the interrupts are reset on > read, so I can only read the status *once* but I need it several times. If I understand correctly you use it as a controller-local variable. > Oh yes it does look for the "stat_reg" in ata_promise_mio_status() which > does the interrupt status getting etc... It seems like I now need to understand how does ata_promise_mio_status() gets called on each interrupt. I'll leave this until I have time to read the code and test your patch too. > Good question, their docs says nothing about it actually, I suppose this > is more a programming style question than anything else, anyhow it will > only change evt behavior until the first interrupt, then I'll write the > entire reg anyways :) Well, so be it. -- ./lxnt