From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:53222) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ao8pk-0004OR-Iz for qemu-devel@nongnu.org; Thu, 07 Apr 2016 08:14:49 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ao8pj-0000fC-RB for qemu-devel@nongnu.org; Thu, 07 Apr 2016 08:14:48 -0400 Date: Thu, 7 Apr 2016 14:14:40 +0200 From: Kevin Wolf Message-ID: <20160407121440.GE4509@noname.redhat.com> References: <1459965434-21531-1-git-send-email-mreitz@redhat.com> <1459965434-21531-5-git-send-email-mreitz@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1459965434-21531-5-git-send-email-mreitz@redhat.com> Subject: Re: [Qemu-devel] [PATCH v2 for-2.7 4/8] block: Drop blk_new_with_bs() List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Max Reitz Cc: Alberto Garcia , qemu-devel@nongnu.org, qemu-block@nongnu.org Am 06.04.2016 um 19:57 hat Max Reitz geschrieben: > Its only caller is blk_new_open(), so we can just inline it there. > > Signed-off-by: Max Reitz Maybe mention that this isn't pure code motion, but that you switch to a bdrv_open() call with a NULL BDS. Either way: Reviewed-by: Kevin Wolf