From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: Fix me in libata-scsi.c Date: Tue, 2 Dec 2014 13:07:43 -0500 Message-ID: <20141202180743.GI10918@htj.dyndns.org> References: <546A9EAC.4080509@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-qa0-f49.google.com ([209.85.216.49]:55684 "EHLO mail-qa0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932365AbaLBSHq (ORCPT ); Tue, 2 Dec 2014 13:07:46 -0500 Content-Disposition: inline In-Reply-To: <546A9EAC.4080509@gmail.com> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: nick Cc: linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org On Mon, Nov 17, 2014 at 08:19:40PM -0500, nick wrote: > I am wondering about the fix me in following code in the file listed > in my message. Seems this is necessary from reading the code and the > FIX ME is no longer needed as the memset is correct and needed for > the function. Yeah, I think the sense buffer clearing is the right thing regardless of it being not strictly necessary for all devices. There's no point in not clearing it. Please feel free to send a patch to remove the comment. Thanks. -- tejun