Re: [PATCH v5 4/6] MdeModulePkg/PciHostBridgeDxe: Add support for address translation

[Guo Heyi <heyi.guo@linaro.org>]

Hi Ray,

If the major concern is about backward compatibility, we can think about ways to
get around of this, e.g. add a feature PCD to turn on translation and set the
default value to false.

Thanks,
Heyi

On Tue, Mar 13, 2018 at 11:00:12AM +0800, Ni, Ruiyu wrote:
> On 3/13/2018 1:18 AM, Ard Biesheuvel wrote:
> >On 7 March 2018 at 04:30, Ni, Ruiyu <ruiyu.ni@intel.com> wrote:
> >>On 3/6/2018 10:44 AM, Guo Heyi wrote:
> >>>
> >>>Hi Ray,
> >>>
> >>>Any comments for v5?
> >>
> >>
> >>Heyi,
> >>Some backward compatibility concerns were received from internal production
> >>teams. Current change will cause silent failure on old platforms because
> >>TranslationOffset might be random if uninitialized.
> >>I will solve the concern and then send out updates to you, hopefully by end
> >>of next week.
> >>
> >
> >Hello Ray,
> >
> >Do you have any updates on these issues? I'd like to proceed with
> >getting this code merged if we can
> >
> >Thanks,
> >Ard.
> >
> Ard，
> I will try to give updates tomorrow.
> If I cannot, I will let you know.
> Then I will give you updates by the end of this week.
> 
> -- 
> Thanks,
> Ray