All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jens Axboe <axboe@kernel.dk>
To: Christoph Hellwig <hch@infradead.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"Sam Bradshaw (sbradshaw)" <sbradshaw@micron.com>
Subject: Re: [GIT PULL] Block drivers for 3.3-rc
Date: Sun, 15 Jan 2012 13:22:05 +0100	[thread overview]
Message-ID: <4F12C4ED.6060301@kernel.dk> (raw)
In-Reply-To: <20120115103331.GA31956@infradead.org>

On 2012-01-15 11:33, Christoph Hellwig wrote:
> On Sun, Jan 15, 2012 at 10:52:12AM +0100, Jens Axboe wrote:
>> Hi Linus,
>>
>> Please see below for the 3.3-rc1 block driver pull request.
>>
>> Essentially two different categories here:
>>
>> - Adds the driver for the Micron RealSSD P320 pci-express flash card.
> 
> The REQ_FUA and REQ_FLUSH handling in it is still completely broken,
> with no attempt to fix it since it last came up.
> 
> So a pretty clear NAK on that driver from me.

Good point, I had forgotten about the ordered writes, I believe the rest
of the items have been handled. The driver/hw, as it stands, does not
reorder writes and it doesn't have a write back cache. So it should
actually be safe. Any acked write is stable. Sam, correct me if I'm
wrong.

-- 
Jens Axboe


  reply	other threads:[~2012-01-15 12:22 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-15  9:52 [GIT PULL] Block drivers for 3.3-rc Jens Axboe
2012-01-15 10:33 ` Christoph Hellwig
2012-01-15 12:22   ` Jens Axboe [this message]
2012-01-15 12:44     ` Christoph Hellwig
2012-01-15 12:50       ` Jens Axboe
2012-01-16 19:10         ` Sam Bradshaw (sbradshaw)
2012-01-16 19:13     ` Sam Bradshaw (sbradshaw)

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=4F12C4ED.6060301@kernel.dk \
    --to=axboe@kernel.dk \
    --cc=hch@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=sbradshaw@micron.com \
    --cc=torvalds@linux-foundation.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.