From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Martin K. Petersen" Subject: Re: [PATCH 3/3] target/iblock: Add WRITE_SAME w/ UNMAP=0 emulation support Date: Thu, 15 Nov 2012 10:25:32 -0500 Message-ID: References: <1352405238-23267-1-git-send-email-nab@linux-iscsi.org> <1352405238-23267-4-git-send-email-nab@linux-iscsi.org> <20121115110432.GD28956@infradead.org> <50A5043E.1030301@interlog.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: <50A5043E.1030301@interlog.com> (Douglas Gilbert's message of "Thu, 15 Nov 2012 10:03:26 -0500") Sender: linux-kernel-owner@vger.kernel.org To: dgilbert@interlog.com Cc: Christoph Hellwig , "Nicholas A. Bellinger" , target-devel , linux-scsi , linux-kernel , Christoph Hellwig , "Martin K. Petersen" List-Id: linux-scsi@vger.kernel.org >>>>> "Doug" == Douglas Gilbert writes: Doug> On 12-11-15 06:04 AM, Christoph Hellwig wrote: >>> + /* >>> + * Enable WRITE_SAME emulation for IBLOCK, use scsi_debug.c default >>> + */ >> >> Why would we care what scsi_debug.c uses? Doug> Would you prefer no hint of where the magic number came from? At Doug> least somebody who cares when they see that comment might contact Doug> Martin Petersen and ask why he chose that value. And the answer is that WRITE SAME(10) takes a two-byte block count. -- Martin K. Petersen Oracle Linux Engineering