From mboxrd@z Thu Jan 1 00:00:00 1970 From: James Bottomley Subject: Re: [dm-devel] [PATCH 2/6] block: no need to initialize rq->cmd with blk_get_request Date: Fri, 25 Apr 2008 11:45:01 -0500 Message-ID: <1209141901.3087.17.camel@localhost.localdomain> References: <1209140607-28654-1-git-send-email-fujita.tomonori@lab.ntt.co.jp> <1209140607-28654-2-git-send-email-fujita.tomonori@lab.ntt.co.jp> <1209140607-28654-3-git-send-email-fujita.tomonori@lab.ntt.co.jp> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1209140607-28654-3-git-send-email-fujita.tomonori@lab.ntt.co.jp> Sender: linux-scsi-owner@vger.kernel.org To: device-mapper development Cc: linux-scsi@vger.kernel.org, bzolnier@gmail.com, Jens Axboe , FUJITA Tomonori , linux-ide@vger.kernel.org, bharrosh@panasas.com, Alasdair G Kergon List-Id: linux-ide@vger.kernel.org On Sat, 2008-04-26 at 01:23 +0900, FUJITA Tomonori wrote: > blk_get_request initializes rq->cmd (rq_init does) so the users don't > need to do that. > > The purpose of this patch is to remove sizeof(rq->cmd) and &rq->cmd, > as a preparation for large command support, which changes rq->cmd from > the static array to a pointer. sizeof(rq->cmd) will not make sense and > &rq->cmd won't work. > > Signed-off-by: FUJITA Tomonori > Cc: James Bottomley > Cc: Alasdair G Kergon > Cc: Jens Axboe > --- > block/scsi_ioctl.c | 3 --- > drivers/block/pktcdvd.c | 2 -- > drivers/cdrom/cdrom.c | 1 - These: > drivers/md/dm-emc.c | 2 -- > drivers/md/dm-mpath-hp-sw.c | 1 - > drivers/md/dm-mpath-rdac.c | 1 - Conflict with this set of patches: http://git.kernel.org/?p=linux/kernel/git/jejb/scsi-pending-2.6.git;a=summary Which move the dm pieces to being request based. The patch series hasn't had an ACK yet from Alasdair, but it would be nice if you could look it over and see if it needs the memsets pulling out. Thanks, James