From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Bottomley Subject: [RFC] block: fix barrier error transmission Date: Wed, 02 Apr 2008 13:02:28 -0500 Message-ID: <1207159348.3082.45.camel@localhost.localdomain> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: Received: from accolon.hansenpartnership.com ([76.243.235.52]:44642 "EHLO accolon.hansenpartnership.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757868AbYDBSCd (ORCPT ); Wed, 2 Apr 2008 14:02:33 -0400 Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: Jens Axboe Cc: linux-scsi , mtosatti@redhat.com There's a problem with barriers in SCSI. They're prepared as a SYNCHRONIZE_CACHE REQ_TYPE_BLOCK_PC command. However, our processing model dictates that we always return no block error but set req->errors to the SCSI result. The upshot is that if we get an error on a flush barrier, it's never seen by the block layer. This patch attempts to get the block layer to see such errors. Note, it's still not quite right. There are possible sense errors that could indicate success of the command (like deferred errors) that this will treat as failures. I suppose we could process the sense in scsi and try to return a first pass error even for REQ_TYPE_BLOCK_PC ... I'm just not sure what that will do to the other users. James --- diff --git a/block/blk-barrier.c b/block/blk-barrier.c index 55c5f1f..3a3947c 100644 --- a/block/blk-barrier.c +++ b/block/blk-barrier.c @@ -114,18 +114,24 @@ void blk_ordered_complete_seq(struct request_queue *q, unsigned seq, int error) static void pre_flush_end_io(struct request *rq, int error) { + error = rq->errors ? -EIO : error; + elv_completed_request(rq->q, rq); blk_ordered_complete_seq(rq->q, QUEUE_ORDSEQ_PREFLUSH, error); } static void bar_end_io(struct request *rq, int error) { + error = rq->errors ? -EIO : error; + elv_completed_request(rq->q, rq); blk_ordered_complete_seq(rq->q, QUEUE_ORDSEQ_BAR, error); } static void post_flush_end_io(struct request *rq, int error) { + error = rq->errors ? -EIO : error; + elv_completed_request(rq->q, rq); blk_ordered_complete_seq(rq->q, QUEUE_ORDSEQ_POSTFLUSH, error); }