From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from list by lists.gnu.org with archive (Exim 4.71) id 1ZroUp-00014s-Ac for mharc-grub-devel@gnu.org; Thu, 29 Oct 2015 10:48:07 -0400 Received: from eggs.gnu.org ([2001:4830:134:3::10]:33907) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZroUl-000147-U5 for grub-devel@gnu.org; Thu, 29 Oct 2015 10:48:05 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZroUg-0005ZH-1S for grub-devel@gnu.org; Thu, 29 Oct 2015 10:48:03 -0400 Received: from mail-wm0-x22d.google.com ([2a00:1450:400c:c09::22d]:38436) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZroUf-0005Z5-SH for grub-devel@gnu.org; Thu, 29 Oct 2015 10:47:57 -0400 Received: by wmeg8 with SMTP id g8so26361708wme.1 for ; Thu, 29 Oct 2015 07:47:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=subject:to:references:from:message-id:date:user-agent:mime-version :in-reply-to:content-type; bh=W1bA+0qVifMfh14qOPJ9KneMyULt+ZH1thru3cTvLXc=; b=wwvJPnaVyWwlKDejqlvs5yYmhk5ZhHu5b/AgZmxJIPYy4MTVicbgSfUoREB+rC1e4x gi7+aYtHgY3CzwxDqvHD2lQlphlf4iDD8yEcxb/J4fufJBxTbfzNYCjdEX914vP8t4Sl 86J+loUpESpCBTElYFSgFZYhSdpiCskVcjVDqeV66lWD4w6pw+wfMkos+TMkgEjLa+wM g+KWE7C7oq8K3DHEg6f59TjGTGThoFyFSU3vD7kREgBrI782ZkyiwwZ+Dgw3DQfrAopr Nw2c9+42d2uzM8QPdp2OrbOniwVxIDfFv5qXxsCOpT+VrmbKm4rp4DolcbpOLka883tN DIOQ== X-Received: by 10.28.51.70 with SMTP id z67mr6846082wmz.25.1446130077121; Thu, 29 Oct 2015 07:47:57 -0700 (PDT) Received: from ?IPv6:2620:0:105f:fd00:863a:4bff:fe50:abc4? ([2620:0:105f:fd00:863a:4bff:fe50:abc4]) by smtp.gmail.com with ESMTPSA id w66sm3876665wme.11.2015.10.29.07.47.56 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 29 Oct 2015 07:47:56 -0700 (PDT) Subject: Re: [edk2] [grub PATCH] efinet: disable MNP background polling To: grub-devel@gnu.org References: <20151001.182655.371384337.d.hatayama@jp.fujitsu.com> <560D1E07.3090902@redhat.com> <20151013214919.GA6140@router-fw-old.local.net-space.pl> <561D83E9.6050703@redhat.com> <20151014110847.GQ6226@olila.local.net-space.pl> <1BAD7DA9-D60E-4743-9143-3327989DF6B6@oracle.com> From: =?UTF-8?Q?Vladimir_'=cf=86-coder/phcoder'_Serbinenko?= Message-ID: <56323196.2000706@gmail.com> Date: Thu, 29 Oct 2015 15:47:50 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Icedove/38.3.0 MIME-Version: 1.0 In-Reply-To: <1BAD7DA9-D60E-4743-9143-3327989DF6B6@oracle.com> Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="KAw6f714GGfWkdTehmtvHg33ILWpwasxR" X-detected-operating-system: by eggs.gnu.org: Error: Malformed IPv6 address (bad octet value). X-Received-From: 2a00:1450:400c:c09::22d X-BeenThere: grub-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list Reply-To: The development of GNU GRUB List-Id: The development of GNU GRUB List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 29 Oct 2015 14:48:05 -0000 This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --KAw6f714GGfWkdTehmtvHg33ILWpwasxR Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable On 14.10.2015 17:39, Seth Goldberg wrote: >=20 >=20 >> On Oct 14, 2015, at 4:08 AM, Daniel Kiper wr= ote: >> >>> On Wed, Oct 14, 2015 at 05:19:32AM +0000, Ye, Ting wrote: >>> Hi all, >>> >>> If I understand the issue correctly, I don't quite agree that UEFI >>> spec is imprecise about SNP constraints described as following. >>> The "constraint" described here is that the grub should use attribute= >>> "EXCLUSIVE" to open SNP protocol to gain exclusive access. This usage= >>> is clearly described in page 184, chapter 6.3 EFI_BOOT_SERVICES.OpenP= rotocol(). >>> >>> EXCLUSIVE Used by applications to gain exclusive access to a p= rotocol interface. >>> If any drivers have the protocol interface opened with an = attribute of BY_DRIVER, >>> then an attempt will be made to remove them by calling the= driver's Stop() function. >>> >>> The grub code should not assume that the SNP is not occupied by other= >>> drivers, instead, it should use EXCLUSIVE to open SNP protocol, or to= >>> be more precise, use OpenProtocolInformation() to check whether SNP i= s >>> already opened by other driver, then decide whether need to use EXCLU= SIVE >>> to disconnect the other drivers. This is the typical usage for all UE= FI >>> protocol, not particular constraints to SNP protocol. >> >> Looks good! Great! However, it looks that we still have a problem if s= omebody >> opens SNP in EXCLUSIVE mode. Then GRUB2 SNP open will fail according t= o UEFI spec. >> Sadly we do not have a control on other stuff and one day our approach= may fail >> because somebody decided to open SNP in EXCLUSIVE mode in e.g. a drive= r. Does >> it mean that migration to MNP is one sensible solution for our problem= s? As I know >> this is huge overhaul, so, we should think twice before choosing that = way. >=20 >=20 > Then it is fortunate that when I wrote the MNP implementation that w= e ship with Oracle Solaris 11.2, that I tested it on many thousands of sy= stems as well as on new UEFI implementations at the UEFI Plugfest ;). >=20 Can you please point to the patch you used? I think the only sane solution judging from what I have read so far is to use MNP as far as possible and fallback to current code if MNP fails > --S >=20 >=20 >=20 >> >> Daniel >> >> _______________________________________________ >> Grub-devel mailing list >> Grub-devel@gnu.org >> https://lists.gnu.org/mailman/listinfo/grub-devel >=20 > _______________________________________________ > Grub-devel mailing list > Grub-devel@gnu.org > https://lists.gnu.org/mailman/listinfo/grub-devel > . >=20 --KAw6f714GGfWkdTehmtvHg33ILWpwasxR Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iF4EAREKAAYFAlYyMZYACgkQmBXlbbo5nOsuDAD/aB9Ev9ZepMVLtogYjPM39tcu ljDluNNIJna3m9LSEqoA/jgH+8kQ+Ab89g99r0uIwzlJGyKgvrdXq/1EJ5SHGe5X =mvS7 -----END PGP SIGNATURE----- --KAw6f714GGfWkdTehmtvHg33ILWpwasxR--