From owner-freebsd-xen@freebsd.org Thu Feb 16 18:54:00 2017 Return-Path: Delivered-To: freebsd-xen@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id D2DD6CE02D0 for ; Thu, 16 Feb 2017 18:54:00 +0000 (UTC) (envelope-from liuyingdong@huawei.com) Received: from szxga03-in.huawei.com (szxga03-in.huawei.com [119.145.14.66]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (Client CN "myname.my.domain", Issuer "www.mirapoint.com" (not verified)) by mx1.freebsd.org (Postfix) with ESMTPS id BAF501B49 for ; Thu, 16 Feb 2017 18:53:59 +0000 (UTC) (envelope-from liuyingdong@huawei.com) Received: from 172.24.1.136 (EHLO szxeml426-hub.china.huawei.com) ([172.24.1.136]) by szxrg03-dlp.huawei.com (MOS 4.4.3-GA FastPath queued) with ESMTP id CPH00799; Fri, 17 Feb 2017 02:47:17 +0800 (CST) Received: from [127.0.0.1] (10.52.13.201) by szxeml426-hub.china.huawei.com (10.82.67.181) with Microsoft SMTP Server id 14.3.235.1; Fri, 17 Feb 2017 02:47:11 +0800 Subject: Re: Multiple patch review References: To: roger freebsd xen maintainer , From: liuyingdong Message-ID: Date: Thu, 16 Feb 2017 20:47:05 +0200 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit X-Originating-IP: [10.52.13.201] X-CFilter-Loop: Reflected X-BeenThere: freebsd-xen@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: Discussion of the freebsd port to xen - implementation and usage List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 16 Feb 2017 18:54:00 -0000 > 发件人: Roger Pau Monné [mailto:roger.pau@citrix.com] > 发送时间: 2017年2月15日 14:32 > 收件人: Liuyingdong > 抄送: freebsd-xen@freebsd.org; > 主题: Re: Multiple patch review (was: Re:[PATCH]netfront: need release all resources) after adding and removing NICs time and again > > On Wed, Feb 15, 2017 at 02:06:36PM +0200, liuyingdong wrote: >> On 2017/2/13 12:05, liuyingdong wrote: >>> 发件人: roger.pau@citrix.com [mailto:roger.pau@citrix.com] >>> 发送时间: 2017年2月10日 17:12 >>> 收件人: Liuyingdong >>> 抄送: freebsd-xen@freebsd.org; Suoben ; Zhaojun >>> (Euler) ; Wanglinkai >>> ; chuzhaosong ; >>> Wangchunfeng (Ivan) ; Gaoxiaodong (Leo) >>> >>> 主题: Multiple patch review (was: Re:[PATCH]netfront: need >>> release all resources) after adding and removing NICs time and again >>> >>> On Tue, Feb 07, 2017 at 04:55:34PM +0000, Liuyingdong wrote: >>>> Hi Roger, >>>> I am so sorry and please review tne below URL: >>>> https://lists.freebsd.org/pipermail/freebsd-xen/2017-February/002957.html >>> >>> Hello, >>> >>> Thanks for the patches, and sorry for the delay. It seems like you have not applied some of my comments, so I will have to re-post them here. If some of the comments don't apply for whatever reason, please reply back and explain why, or else this is not going to progress in an useful way for any of us. >>> >>> I would also request you to look into using `git send-email`, reviewing your patches as attachments is not very comfortable. Or else, you could create an account to https://reviews.freebsd.org/ and upload the patches there assigning me as a reviewer. >>> >> I have created an account to https://reviews.freebsd.org/ and uploaded the patches but I cann't assign you as a reviewer so I assign visible to All Users. >> The three modified patches are as follows: >> 1.https://reviews.freebsd.org/differential/diff/25207/ >> 2.https://reviews.freebsd.org/differential/diff/25208/ >> 3.https://reviews.freebsd.org/differential/diff/25209/ > > You need to finish creating the revision, click on "Continue", and add a title/description and reviewers, see https://wiki.freebsd.org/Phabricator#Create_a_Revision_via_Web_Interface for more information. > Thank you for the time and patience you devoted to guide me how to create a revision. I have created three revisions and please review again. https://reviews.freebsd.org/D9635 https://reviews.freebsd.org/D9638 https://reviews.freebsd.org/D9639 > Roger. > Thanks, Terry.