On Thu, Mar 7, 2019 at 6:09 AM Arnaldo Carvalho de Melo <arnaldo.melo@xxxxxxxxx> wrote: > > Em Thu, Mar 07, 2019 at 11:02:47AM -0300, Arnaldo Carvalho de Melo escreveu: > > Em Wed, Mar 06, 2019 at 04:23:21PM -0800, Andrii Nakryiko escreveu: > > > Due to this assumption, libdwarves and other parts of pahole are using 16-bit > > > counters to iterate over entities within CU. This can cause infinite loop when > > > iterating BTF data, if there are more than 65535 types. This patch changes > > > non-public variables to use 32-bit integers, where appropriate. > > > Ok, there are some bits that are unrelated, I'm comment on it and remove > > before applying. Yeah, thanks for cleaning this up! My intent was to convert all places where we rely on uint16_t for things that potentially can overflow (so not just IDs) and are safe to change without binary compatibility issues. I didn't intend this to cause you more work, sorry about that. > > Ooops, see below, I'm removing the non-type related parts, and will look > at changing the types till btfdiff with that allyesconfig aarch64 > vmlinux image shows no diffs. > > - Arnaldo <snip>
