From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:54464) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UwsKf-000602-Aa for qemu-devel@nongnu.org; Wed, 10 Jul 2013 07:13:19 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UwsKZ-0000sR-1Y for qemu-devel@nongnu.org; Wed, 10 Jul 2013 07:13:13 -0400 Received: from mx1.redhat.com ([209.132.183.28]:46884) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UwsKY-0000rp-On for qemu-devel@nongnu.org; Wed, 10 Jul 2013 07:13:06 -0400 Date: Wed, 10 Jul 2013 19:13:03 +0800 From: Fam Zheng Message-ID: <20130710111303.GG18465@T430s.nay.redhat.com> References: <1373268366-14508-1-git-send-email-cngesaint@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1373268366-14508-1-git-send-email-cngesaint@gmail.com> Subject: Re: [Qemu-devel] [PATCH V2 0/5] Add infinite loop check for backing file chain Reply-To: famz@redhat.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Xu Wang Cc: kwolf@redhat.com, stefanha@gmail.com, qemu-devel@nongnu.org, xiawenc@linux.vnet.ibm.com On Mon, 07/08 03:26, Xu Wang wrote: > Updates: > 1. Changed infinite loop check in collect_image_info_list() from filename > checking to inode checking. > 2. Absolute or relative path is OK for filename path. > 3. Hard and soft link are works well. > 4. Added WIN32 platform support (shortcuts could be recognized correctly.) > 5. Create a file which contains loop in backing file will failed. > 6. Start a vm which boot block file contains loop in backing file chain > will failed instead of no response and segment fault. > > Xu Wang (5): > Refine and export infinite loop checking in collect_image_info_list() > Add WIN32 platform support for backing_file_loop_check() > Check infinite loop in bdrv_img_create() > Add backing file loop check in change_backing_file() > Add infinite loop check in drive_init() > > block.c | 211 ++++++++++++++++++++++++++++++++++++++++++++++++-- > blockdev.c | 5 ++ > include/block/block.h | 4 + > qemu-img.c | 30 +++---- > 4 files changed, 224 insertions(+), 26 deletions(-) > Thanks for the series. I made a few comments on each patch. You hashed windows full path to get the "inode", but case sensitively, would that work with the same files of different case? Please prefix your commit message with "block:" or "qemu-img:", e.g.: block: refactor collect_image_info_list() block: Add WIN32 platform support for backing_file_loop_check() etc.. and maybe also describe some of the long patches, explaining what is changed. Also please consider to write some test case on your changes, as requested in the replies to previous version. -- Fam