On 17.10.2022 03:20, Henry Wang wrote: >> -----Original Message----- >> From: Marek Marczykowski-Górecki <marma...@invisiblethingslab.com> >> >> On Thu, Sep 29, 2022 at 03:33:12PM +0200, Marek Marczykowski-Górecki >> wrote: >>> Re-use rmrr= parameter handling code to handle common device reserved >>> memory. >>> >>> Move MAX_USER_RMRR_PAGES limit enforcement to apply only to >>> user-configured ranges, but not those from internal callers. >>> >>> Signed-off-by: Marek Marczykowski-Górecki >> <marma...@invisiblethingslab.com> >>> Reviewed-by: Kevin Tian <kevin.t...@intel.com> >> >> Henry, can this be included in 4.17? The AMD counterpart went in >> earlier, but due to late review on Intel part, this one didn't. > > Thanks for the information. I agree this is a valid reason, but to be > safe I would like to hear opinions from the x86 maintainers (added > in CC). > > Andrew/Jan/Roger: May I have your feedback about this? Thanks!
Hmm, not sure what to say here. Yes, it would be nice for things to end up consistent across vendors. And yes, the change here is largely mechanical (afaics) and to code most of which shouldn't typically be in use on systems anyway, and so should not pose an undue risk. But still it is quite a bit of code churn ... Jan
