From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Bottomley Subject: Re: [PATCH scsi-misc-2.6 09/13] scsi: in scsi_prep_fn(), remove bogus comments & clean up Date: Thu, 31 Mar 2005 12:02:20 -0600 Message-ID: <1112292140.5619.26.camel@mulgrave> References: <20050331090647.FEDC3964@htj.dyndns.org> <20050331090647.B562915C@htj.dyndns.org> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: Received: from stat16.steeleye.com ([209.192.50.48]:6559 "EHLO hancock.sc.steeleye.com") by vger.kernel.org with ESMTP id S261599AbVCaSCe (ORCPT ); Thu, 31 Mar 2005 13:02:34 -0500 In-Reply-To: <20050331090647.B562915C@htj.dyndns.org> Sender: linux-scsi-owner@vger.kernel.org List-Id: linux-scsi@vger.kernel.org To: Tejun Heo Cc: Jens Axboe , SCSI Mailing List , Linux Kernel On Thu, 2005-03-31 at 18:08 +0900, Tejun Heo wrote: > - * come up when there is a medium error. We have to treat > - * these two cases differently. We differentiate by looking > - * at request->cmd, as this tells us the real story. > + * come up when there is a medium error. This comment isn't wrong. That's exactly what this piece of code: if (sreq->sr_magic == SCSI_REQ_MAGIC) { is all about ... that's how it distinguishes between the two cases. The comment is misleading --- what it actually should say is that req- >special has different contents depending upon the two cases, so rephrasing it to be more accurate would be helpful. James