On Fri, Jan 15, 2021 at 3:34 PM Nick Desaulniers <ndesaulni...@google.com> wrote: > > On Fri, Jan 15, 2021 at 3:24 PM Yonghong Song <y...@fb.com> wrote: > > > > > > > > On 1/15/21 1:53 PM, Sedat Dilek wrote: > > > En plus, I encountered breakage with GCC v10.2.1 and LLVM=1 and > > > CONFIG_DEBUG_INFO_DWARF4. > > > So might be good to add a "depends on !DEBUG_INFO_BTF" in this > > > combination. > > Can you privately send me your configs that repro? Maybe I can isolate > it to a set of configs?
Why privately? To reproduce and fix the issue we (BPF and pahole community) would need the config as well. > > > > > I suggested not to add !DEBUG_INFO_BTF to CONFIG_DEBUG_INFO_DWARF4. > > It is not there before and adding this may suddenly break some users. > > > > If certain combination of gcc/llvm does not work for > > CONFIG_DEBUG_INFO_DWARF4 with pahole, this is a bug bpf community > > should fix. > > Is there a place I should report bugs? b...@vger.kernel.org (BPF in general) and dwar...@vger.kernel.org (pahole, which seems to be emitting these warnings and having problems with DWARF5). > > > > > > > > > I had some other small nits commented in the single patches. > > > > > > As requested in your previous patch-series, feel free to add my: > > > > > > Tested-by: Sedat Dilek <sedat.di...@gmail.com> > > Yeah, I'll keep it if v6 is just commit message changes. > > -- > Thanks, > ~Nick Desaulniers
