From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tejun Heo Subject: Re: [PATCH] sata_mv: fix broken DSM/TRIM support (v2) Date: Mon, 23 Aug 2010 15:59:32 +0200 Message-ID: <4C727EC4.6060903@kernel.org> References: <1282267195.8909.6.camel@corey> <1282268444.9628.4.camel@corey> <4C722FD4.2010305@kernel.org> <1282570893.30799.1.camel@corey> <4C727BFB.5070203@kernel.org> <4C727D93.4000105@teksavvy.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Return-path: Received: from hera.kernel.org ([140.211.167.34]:46754 "EHLO hera.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751163Ab0HWN6Z (ORCPT ); Mon, 23 Aug 2010 09:58:25 -0400 In-Reply-To: <4C727D93.4000105@teksavvy.com> Sender: linux-ide-owner@vger.kernel.org List-Id: linux-ide@vger.kernel.org To: Mark Lord Cc: linux-ide@vger.kernel.org, jeff@garzik.org On 08/23/2010 03:54 PM, Mark Lord wrote: > On 10-08-23 09:47 AM, Tejun Heo wrote: >> On 08/23/2010 03:41 PM, Mark Lord wrote: >>>> Don't we need this for other BMDMA commands too? If so, maybe it's >>>> better to make it two patches? >>> >>> >>> TRIM is the first/only command which actually breaks >>> without this. It is possible that other commands may >>> benefit as well on some DVD-RW drives, but I have not >>> encountered those yet. >>> >>> So for now, I've left it as a single patch, >>> with all of the (small) pieces together for TRIM. >> >> I see. Looks good to me then. >> >> Acked-by: Tejun Heo > .. > > Peachy. I do imagine that both of these patches > might also be prime candidates for -stable, > if/when they ever make it upstream. Just add Cc: stable@kernel.org below SOB's. Greg will be automatically notified when these patches hit mainline. Thanks. -- tejun