From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932154AbWFDIUp (ORCPT ); Sun, 4 Jun 2006 04:20:45 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S932189AbWFDIUp (ORCPT ); Sun, 4 Jun 2006 04:20:45 -0400 Received: from pentafluge.infradead.org ([213.146.154.40]:58517 "EHLO pentafluge.infradead.org") by vger.kernel.org with ESMTP id S932180AbWFDIUo (ORCPT ); Sun, 4 Jun 2006 04:20:44 -0400 Date: Sun, 4 Jun 2006 09:20:35 +0100 From: Christoph Hellwig To: Tejun Heo Cc: Jens Axboe , James Bottomley , Dave Miller , bzolnier@gmail.com, james.steward@dynamicratings.com, jgarzik@pobox.com, mattjreimer@gmail.com, Guennadi Liakhovetski , rmk@arm.linux.org.uk, lkml , linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org Subject: Re: [PATCH 4/5] SCSI: add cpu cache flushes after kmapping and modifying a page Message-ID: <20060604082035.GB29696@infradead.org> Mail-Followup-To: Christoph Hellwig , Tejun Heo , Jens Axboe , James Bottomley , Dave Miller , bzolnier@gmail.com, james.steward@dynamicratings.com, jgarzik@pobox.com, mattjreimer@gmail.com, Guennadi Liakhovetski , rmk@arm.linux.org.uk, lkml , linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org References: <1149392479501-git-send-email-htejun@gmail.com> <11493924803460-git-send-email-htejun@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <11493924803460-git-send-email-htejun@gmail.com> User-Agent: Mutt/1.4.2.1i X-SRS-Rewrite: SMTP reverse-path rewritten from by pentafluge.infradead.org See http://www.infradead.org/rpr.html Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Sun, Jun 04, 2006 at 12:41:20PM +0900, Tejun Heo wrote: > local_irq_save(flags); > buf = kmap_atomic(sg->page, KM_IRQ0) + sg->offset; > memcpy(buf, tw_dev->generic_buffer_virt[request_id], sg->length); > + flush_kernel_dcache_page(kmap_atomic_to_page(buf - sg->offset)); > kunmap_atomic(buf - sg->offset, KM_IRQ0); > local_irq_restore(flags); all these should switch to scsi_kmap_atomic_sg which should do the flush_kernel_dcache_page call for you.