Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S261644AbVB1PY7 (ORCPT ); Mon, 28 Feb 2005 10:24:59 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S261648AbVB1PYz (ORCPT ); Mon, 28 Feb 2005 10:24:55 -0500 Received: from wproxy.gmail.com ([64.233.184.196]:58445 "EHLO wproxy.gmail.com") by vger.kernel.org with ESMTP id S261644AbVB1PYr (ORCPT ); Mon, 28 Feb 2005 10:24:47 -0500 DomainKey-Signature: a=rsa-sha1; q=dns; c=nofws; s=beta; d=gmail.com; h=received:message-id:date:from:user-agent:x-accept-language:mime-version:to:cc:subject:references:in-reply-to:content-type:content-transfer-encoding; b=lUJDJQW74QafXT9zzKriMMS7Z6uehUm0PxqvBvSTDtV868QvOii0GpGeBTgm51/7Vr5RaaswBSF9qO1HiCvIlr9cDY919S2E3J9y/j+Ck0AzLsTEaMxpcfpcL+p3RiUxW0QRduSm5qLo76fl7E0hOatuW/26qKCCd+h3H2eJwQY= Message-ID: <422337A1.4060806@gmail.com> Date: Tue, 01 Mar 2005 00:24:17 +0900 From: Tejun Heo User-Agent: Debian Thunderbird 1.0 (X11/20050118) X-Accept-Language: en-us, en MIME-Version: 1.0 To: Bartlomiej Zolnierkiewicz Cc: linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [patch ide-dev 8/9] make ide_task_ioctl() use REQ_DRIVE_TASKFILE References: <20050227073608.GA30796@htj.dyndns.org> <200502271731.29448.bzolnier@elka.pw.edu.pl> In-Reply-To: <200502271731.29448.bzolnier@elka.pw.edu.pl> Content-Type: text/plain; charset=iso-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1658 Lines: 49 Hi, Bartlomiej Zolnierkiewicz wrote: > On Sunday 27 February 2005 08:36, Tejun Heo wrote: > >> Hello, Bartlomiej. >> >> This patch should be modified to use flagged taskfile if the >>task_end_request_fix patch isn't applied. As non-flagged taskfile >>won't return valid result registers, TASK ioctl users won't get the >>correct register output. > > > Nope, it works just fine because REQ_DRIVE_TASK used only > no-data protocol, please check task_no_data_intr(). > Sorry, I missed that. IDE really has a lot of ways to finish a command, doesn't it? hdio.txt is gonna look ugly. :-) > >> IMHO, this flag-to-get-result-registers thing is way too subtle. How >>about keeping old behavior by just not copying out register outputs in >>ide_taskfile_ioctl() in applicable cases instead of not reading >>registers when ending commands? That is, unless there's some >>noticeable performance impacts I'm not aware of. > > > This would miss whole point of not _reading_ these registers (IO is slow). > IMHO new flags denoting {in,out} registers should be added (to > to share them with libata) so new code can be sane and old flags would map > on new flags when needed. Please do it. Or, let me know what you have in mind (added fields, flag names, etc...); then, I'll do it. I think we also need to hear Jeff's opinion as things need to be added to ata.h. Thanks. -- tejun - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/