From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=209.132.183.28; helo=mx1.redhat.com; envelope-from=lersek@redhat.com; receiver=edk2-devel@lists.01.org Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 43C6621A07A82 for ; Tue, 18 Sep 2018 03:18:02 -0700 (PDT) Received: from smtp.corp.redhat.com (int-mx07.intmail.prod.int.phx2.redhat.com [10.5.11.22]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id 17FFD308FF16; Tue, 18 Sep 2018 10:18:02 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-120-209.rdu2.redhat.com [10.10.120.209]) by smtp.corp.redhat.com (Postfix) with ESMTP id 4492C1001F56; Tue, 18 Sep 2018 10:18:00 +0000 (UTC) To: "Subramanian, Sriram" , Wang Fan , "edk2-devel@lists.01.org" Cc: Ye Ting , Fu Siyuan , Wu Jiaxin , "Subramanian@mx0b-002e3701.pphosted.com" References: <20180914082415.800-1-fan.wang@intel.com> From: Laszlo Ersek Message-ID: Date: Tue, 18 Sep 2018 12:17:59 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.84 on 10.5.11.22 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.49]); Tue, 18 Sep 2018 10:18:02 +0000 (UTC) Subject: Re: [Patch v2] NetworkPkg: UefiPxeBcDxe: Add EXCLUSIVE attribute when opening SNP protocol installed by PXE. X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 18 Sep 2018 10:18:03 -0000 Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 09/14/18 10:50, Subramanian, Sriram wrote: > Reviewed-by: Sriram Subramanian > > -----Original Message----- > From: Wang Fan [mailto:fan.wang@intel.com] > Sent: Friday, September 14, 2018 1:54 PM > To: edk2-devel@lists.01.org > Cc: Subramanian@mx0b-002e3701.pphosted.com; Subramanian, Sriram ; Ye Ting ; Fu Siyuan ; Wu Jiaxin > Subject: [Patch v2] NetworkPkg: UefiPxeBcDxe: Add EXCLUSIVE attribute when opening SNP protocol installed by PXE. > > Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1152 > > v2: Sync the same logic to Ipv6 and update code comments. > > The PXE driver installs a SNP and open this SNP with attribute BY_DRIVER > to avoid it being opened by MNP driver, this SNP is also expected not to > be opened by other drivers with EXCLUSIVE attribute. In some cases, other > drivers may happen to do this by error, and thus cause a system crash. > This patch adds EXCLUSIVE attribute when opening SNP in PXE driver, and > will reject all OpenProtocol requests by EXCLUSIVE. > > Cc: Subramanian, Sriram > Cc: Ye Ting > Cc: Fu Siyuan > Cc: Wu Jiaxin > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Wang Fan > --- > NetworkPkg/UefiPxeBcDxe/PxeBcDriver.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) Either my edk2-devel subscription is breaking down, or our discipline regarding commit pushing is down the drain, recently. This patch has been pushed as commit cde5a72d365e. Problems with that commit: (1) The git authorship is marked as "edk2-devel-bounces@lists.01.org". That's *wrong*. This patch was authored by Wang Fan . (2) The commit lists Siyuan and Jiaxin as having given their reviews. I don't see their *v2* reviews anywhere on the list. I see their *v1* reviews, but the patch was changed in v2. Wang Fan was correct not to carry forward the v1 reviews into the v2 posting. If the v2 posting was fine, then Siyuan and Jiaxin should have confirmed that (with R-b's) on the list, before pushing the commit. Another process failure that I'm witnessing is that people forget to close BZs once the corresponding fixes are upstream. Sorry but that just makes a joke out of bug tracking. Come on, guys. We can do better than this. Show some discipline and dedication to the process, please. The process is not self-serving; it's in place to promote mutual understanding. Laszlo