All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tejun Heo <htejun@gmail.com>
To: Jeff Garzik <jeff@garzik.org>
Cc: Alan Cox <alan@lxorguk.ukuu.org.uk>,
	akpm@osdl.org, linux-ide@vger.kernel.org
Subject: Re: [PATCH] libata: Add a drivers/ide style  DMA disable
Date: Sat, 08 Sep 2007 15:01:00 +0900	[thread overview]
Message-ID: <46E23A9C.2090003@gmail.com> (raw)
In-Reply-To: <46E1D4FB.8030409@garzik.org>

Jeff Garzik wrote:
> FWIW -- as I noted to Alan personally at KS, I would rather drop the
> "pata_" and have it apply to all, PATA or SATA.

Having the distinction between sata and pata is also nice tho
considering many modern machines mix the two and use the P part for
ATAPI, CF kind of stuff.  It's all just masks anyway, we can easily do both.

> As an aside, it would be nice to get the EH in shape where it can pause
> all ports, so we can finally allow userspace to submit SET FEATURES -
> XFER MODE and have it handled cleanly and properly.

Aye aye sir, bumping up in the to-do list.  BTW, I'm currently on
airplane and won't be back home in about ten days.  PMP patchset is
regenerated but it has some problem integrating with the AN support and
I need my gears to iron it out, so PMP patches will have to wait a bit
more.  FWIW, openSUSE 10.3 defaults to libata drivers and contains the
PMP patches.  It already received quite a bit of testing going through
two betas, so I don't think the PMP patches would introduce too many
serious regressions for non-PMP users.

Thanks.

-- 
tejun


  reply	other threads:[~2007-09-08 17:06 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-22 22:37 [PATCH] libata: Add a drivers/ide style DMA disable Alan Cox
2007-09-07  3:58 ` Tejun Heo
2007-09-07 22:47   ` Jeff Garzik
2007-09-08  6:01     ` Tejun Heo [this message]
2007-09-08 18:42     ` Alan Cox
2007-09-14 17:17     ` Chuck Ebbert
2007-09-14 19:54       ` Bartlomiej Zolnierkiewicz
2007-09-19 15:53 ` Chuck Ebbert
2007-09-19 16:40   ` Alan Cox
2007-09-20  1:18   ` Tejun Heo
2007-10-02 16:38 ` Jeff Garzik

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=46E23A9C.2090003@gmail.com \
    --to=htejun@gmail.com \
    --cc=akpm@osdl.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=jeff@garzik.org \
    --cc=linux-ide@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.