From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 27 Sep 2016 19:06:10 -0700 From: Christoph Hellwig To: Adam Manzanares Cc: axboe@kernel.dk, tj@kernel.org, linux-block@vger.kernel.org, linux-ide@vger.kernel.org Subject: Re: [PATCH 0/3] Enabling ATA Command Priorities Message-ID: <20160928020610.GA28998@infradead.org> References: <1475000096-6148-1-git-send-email-adam.manzanares@hgst.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <1475000096-6148-1-git-send-email-adam.manzanares@hgst.com> List-ID: The series looks fine to me: Reviewed-by: Christoph Hellwig The only question is if we need to bother with the last patch to make the feature conditional at all, given that we both need hardware support and applications opting into using I/O priorities to even use it. But if you feel it's safer that way the unable certainly doesn't hurt.