Re: [PATCH] irqchip/gic-v3: change gic_data into gic_v3_data

From: zc
Date: Thu Sep 03 2020 - 06:05:22 EST



  thank for you reply.


   of course it won't,   i just think it is better to splite the global data between irq-gic and irq-gic-v3.

I wrote some hook module for gic,  kallsyms_lookup_name can't support for mutlipte symbol,

i need to call kallsyms scan to get the addr.


  If you think it isn't necessary, well for me.


Regards,

zou cao

在 2020/9/3 下午5:55, Marc Zyngier 写道:
On 2020-09-03 10:08, Zou Cao wrote:
There are multiple gic_data symbol between irq_gic.c and irq_gic_v3.c,
we can see the same symbol name:

    cat /proc/kallsyms | grep gic_data
------>
    ffff80001138f1d0 d gic_data
    ffff80001138f940 d gic_data

Normally CONFIG_ARM_GIC and CONFIG_ARM_GIC_V3 are all enabled, move
the gic_data symbol into different name, it will be friend for
kallsyms_lookup_name to get addr easily.

Signed-off-by: Zou Cao <zoucao@xxxxxxxxxxxxxxxxx>

I'm sorry, but this kind of change is not acceptable.

It is pure churn, for zero benefit. As an example, give the following
script a go:

awk '{ print $3 }' /proc/kallsyms | sort | uniq -c| awk '$1 > 1 {print}'| sort -n

Are you going to repaint all these symbols?

        M.