From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartlomiej Zolnierkiewicz Subject: Re: [RFC PATCH 0/12] ide-atapi remove pc->buf Date: Sun, 10 May 2009 23:32:24 +0200 Message-ID: <200905102332.24775.bzolnier@gmail.com> References: <1241855134-4984-1-git-send-email-petkovbb@gmail.com> Mime-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-15" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-bw0-f174.google.com ([209.85.218.174]:44286 "EHLO mail-bw0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756530AbZEJVgC (ORCPT ); Sun, 10 May 2009 17:36:02 -0400 In-Reply-To: <1241855134-4984-1-git-send-email-petkovbb@gmail.com> Content-Disposition: inline Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Borislav Petkov Cc: linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, Tejun Heo , Jens Axboe Hi, On Saturday 09 May 2009 09:45:20 Borislav Petkov wrote: > Hi Bart, > > Here's an assortment :) of ide patches touching stuff here and there. > I've been working on those for a while and initial testing shows no > problems. Among the changes is a first attempt to remove the 64 byte > buffer embedded in each packet command sent to an ide drive. Also, some > ireason handling unification is there along with other minor stuff. Generally looks good but needs some fixes (see my other mails). [ also: if you are re-posting any patches please always indicate whether (and how) they were changed ] > This is based ontop of > > git://git.kernel.org/pub/scm/linux/kernel/git/tj/misc.git block-unify-sector-and-data_len > > and I'm more than open to suggestions on the logistics of merging those > (how/when/on top of whose branch/who picks them up/do we want to hit the > .31 merge window or do more testing?/etc) after being reviewed. > > > drivers/ide/ide-atapi.c | 145 +++++++++++++++++++++++------------ > drivers/ide/ide-cd.c | 69 +++-------------- > drivers/ide/ide-floppy.c | 53 ++++++------- > drivers/ide/ide-floppy_ioctl.c | 43 ++++++----- > drivers/ide/ide-tape.c | 168 +++++++++++++++++++--------------------- > include/linux/ide.h | 18 +---- > 6 files changed, 238 insertions(+), 258 deletions(-) Good question, how's about: Once tj/block-unify-sector-and-data_len is in block/for-2.6.31 and your tree is ready I will pull it into the separate branch. This branch can be then pulled into ide/for-next or into new ide/all branch (replacing ide/for-next in linux-next). This means that I'll have to synchronize merge window merges with Jens but this is needed already and shouldn't be a problem. If somebody [ Tejun/Jens: I'm talking to you :) ] has a better idea how to handle this please stand-up. Thanks, Bart