[PATCH V2] arm64: Fix early pointer print plus improve comment

Guilherme G. Piccoli gpiccoli at igalia.com
Tue Dec 21 07:35:13 PST 2021


On 21/12/2021 12:27, Robin Murphy wrote:
> [...]
> Strictly it *is* the right modifier, since users who want to see 
> unhashed pointers should pass "no_hash_pointer" on the command line.
> 
> However, in this particular instance, the information leakage concern 
> does not apply since we're facing such a catastrophic failure that the 
> kernel can't even run - there's nothing for an attacker to attack! This 
> is effectively a last-gasp panic message to help debug bootloader issues 
> beyond the kernel's control, so it seems reasonable not to hamper it 
> with kernel-debugging machinery.
> 
> It might be worth spelling out the rationale clearly, at least in the 
> commit message, so it's there for easy future reference if someone comes 
> along with a "%px is bad, change it back" patch. "Hashed address stuff" 
> on its own isn't really a reason.
> 

OK Robin, thanks for the review again, I agree with you and will resubmit.


>> [...]
> 
> Nit: I think we prefer normal-style comments (i.e. "/*" on its own line 
> to start) in arch code.
> 
> Otherwise, it all seems reasonable - thanks for clearing it up.
> 
> Robin.

Cool, will fix that also in the V3 =)
Cheers,


Guilherme



More information about the linux-arm-kernel mailing list