From mboxrd@z Thu Jan 1 00:00:00 1970 From: Toshi Kani Subject: Re: [PATCH v4 07/12] efi: passing kexec necessary efi data via setup_data Date: Wed, 04 Dec 2013 09:43:48 -0700 Message-ID: <1386175428.1791.225.camel@misato.fc.hp.com> References: <1385445477-9665-1-git-send-email-dyoung@redhat.com> <1385445477-9665-8-git-send-email-dyoung@redhat.com> <20131127140732.GD32267@pd.tnic> <20131129091415.GH4186@dhcp-16-126.nay.redhat.com> <1386023600.1791.201.camel@misato.fc.hp.com> <1386034289.1791.210.camel@misato.fc.hp.com> <20131203015613.GB3980@dhcp-16-126.nay.redhat.com> <1386085140.1791.218.camel@misato.fc.hp.com> <20131204024610.GA3764@dhcp-16-126.nay.redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20131204024610.GA3764-je1gSBvt1TcFLmT5oZ11vB/sF2h8X+2i0E9HWUfgJXw@public.gmane.org> Sender: linux-efi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Dave Young Cc: Borislav Petkov , linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-efi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, x86-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, mjg59-1xO5oi07KQx4cg9Nei1l7Q@public.gmane.org, hpa-YMNOUZJC4hwAvxtiuMwx3w@public.gmane.org, James.Bottomley-d9PhHud1JfjCXq6kfMZ53/egYHeGw8Jk@public.gmane.org, vgoyal-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org, ebiederm-aS9lmoZGLiVWk0Htik3J/w@public.gmane.org, horms-/R6kz+dDXgpPR4JQBCEnsQ@public.gmane.org, kexec-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, greg-U8xfFu+wG4EAvxtiuMwx3w@public.gmane.org, matt-HNK1S37rvNbeXh+fF434Mdi2O/JbrIOy@public.gmane.org List-Id: linux-efi@vger.kernel.org On Wed, 2013-12-04 at 10:46 +0800, Dave Young wrote: > Hi, Toshi > > > Oh, I think I now understand what the issue was. The z420 firmware > > updates the SMBIOS table address in the EFI system table to a virtual > > address after calling EFI SetVirtualAddressMap. So, you are passing the > > original physical address of the SMBIOS table from the 1st kernel to the > > 2nd kernel to put it back to physical. Is that right? > > Right. Hi Dave, The z420 firmware is based on some UEFI core that may be used by other vendors as well. Since this handling is totally harmless (just redundant), I'd suggest not to have a platform check on this handling. Thanks, -Toshi