Re: [PATCH 01/17] kallsyms: support big kernel symbols (2-byte lengths)

From: Linus Torvalds
Date: Sun Jul 04 2021 - 17:28:48 EST


On Sun, Jul 4, 2021 at 2:15 PM Miguel Ojeda
<miguel.ojeda.sandonis@xxxxxxxxx> wrote:
>
> No particular reason. It makes sense to use LE -- I will change it.

Matthew's suggestion is denser, which is nice.

At that point, it would be neither BE nor LE. But the "LE-like" version would be

len = data[0];
if (len & 128)
len += data[1] << 7;

which ends up having a tiny bit more range (it goes to 11^H32895).

Of course, if the range is expected to be just 0-300, I guess that
matters not one whit.

Linus