From mboxrd@z Thu Jan 1 00:00:00 1970 From: Christoph Hellwig Subject: Re: [PATCH v2] Add support for SCT Write Same Date: Mon, 13 Jun 2016 01:02:43 -0700 Message-ID: <20160613080243.GB26544@infradead.org> References: <1465453967-11085-1-git-send-email-shaun@tancheff.com> <1465453967-11085-2-git-send-email-shaun@tancheff.com> <20160609092212.GA7459@infradead.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from bombadil.infradead.org ([198.137.202.9]:33252 "EHLO bombadil.infradead.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965072AbcFMICp (ORCPT ); Mon, 13 Jun 2016 04:02:45 -0400 Content-Disposition: inline In-Reply-To: Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Shaun Tancheff Cc: Christoph Hellwig , Shaun Tancheff , linux-ide@vger.kernel.org, linux-scsi@vger.kernel.org, "James E . J . Bottomley" , "Martin K . Petersen" , Tejun Heo On Thu, Jun 09, 2016 at 10:24:40PM -0500, Shaun Tancheff wrote: > > - The plain page_address above looks harmful, how do we know that > > the page is mapped into kernel memory? This might actually be broken > > already, though. > > I think it just happens to work because it's always used with a recently > allocated page. Fixing it to include the (possible) offset is just a good thing. I haven't spent much time with libata recently, but what prevents an arbitrary user page ending up here through SG_IO?