Re: [PATCH] irqchip: Use %px to print pointer value

2018-02-12 Thread Jonas Gorski 
On 9 February 2018 at 17:04, Marc Zyngier  wrote:
> On 09/02/18 15:54, Florian Fainelli wrote:
>> On February 9, 2018 12:51:33 AM PST, Marc Zyngier  
>> wrote:
>>> On 09/02/18 02:10, Jaedon Shin wrote:
 Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
 pointers printed with %p are hashed. Use %px instead of %p to print
 pointer value.

 Signed-off-by: Jaedon Shin 
 ---
  drivers/irqchip/irq-bcm7038-l1.c | 2 +-
  drivers/irqchip/irq-bcm7120-l2.c | 2 +-
  drivers/irqchip/irq-brcmstb-l2.c | 2 +-
  3 files changed, 3 insertions(+), 3 deletions(-)

 diff --git a/drivers/irqchip/irq-bcm7038-l1.c
>>> b/drivers/irqchip/irq-bcm7038-l1.c
 index 55cfb986225b..f604c1d89b3b 100644
 --- a/drivers/irqchip/irq-bcm7038-l1.c
 +++ b/drivers/irqchip/irq-bcm7038-l1.c
 @@ -339,7 +339,7 @@ int __init bcm7038_l1_of_init(struct device_node
>>> *dn,
 goto out_unmap;
 }

 -   pr_info("registered BCM7038 L1 intc (mem: 0x%p, IRQs: %d)\n",
 +   pr_info("registered BCM7038 L1 intc (mem: 0x%px, IRQs: %d)\n",
 intc->cpus[0]->map_base, IRQS_PER_WORD * intc->n_words);

 return 0;
 diff --git a/drivers/irqchip/irq-bcm7120-l2.c
>>> b/drivers/irqchip/irq-bcm7120-l2.c
 index 983640eba418..1cc4dd1d584a 100644
 --- a/drivers/irqchip/irq-bcm7120-l2.c
 +++ b/drivers/irqchip/irq-bcm7120-l2.c
 @@ -318,7 +318,7 @@ static int __init bcm7120_l2_intc_probe(struct
>>> device_node *dn,
 }
 }

 -   pr_info("registered %s intc (mem: 0x%p, parent IRQ(s): %d)\n",
 +   pr_info("registered %s intc (mem: 0x%px, parent IRQ(s): %d)\n",
 intc_name, data->map_base[0], data->num_parent_irqs);

 return 0;
 diff --git a/drivers/irqchip/irq-brcmstb-l2.c
>>> b/drivers/irqchip/irq-brcmstb-l2.c
 index 691d20eb0bec..6760edeeb666 100644
 --- a/drivers/irqchip/irq-brcmstb-l2.c
 +++ b/drivers/irqchip/irq-brcmstb-l2.c
 @@ -262,7 +262,7 @@ static int __init brcmstb_l2_intc_of_init(struct
>>> device_node *np,
 ct->chip.irq_set_wake = irq_gc_set_wake;
 }

 -   pr_info("registered L2 intc (mem: 0x%p, parent irq: %d)\n",
 +   pr_info("registered L2 intc (mem: 0x%px, parent irq: %d)\n",
 base, parent_irq);

 return 0;

>>>
>>> Why is that something useful to do? This just tells you where the
>>> device
>>> is mapped in the VA space, and I doubt that's a useful information,
>>> hashed pointers or not. Am I missing something obvious?
>>
>> No you are right there is not much value in printing the register
>> virtual address (sometimes there is e.g: on MIPS) either we fix the
>> prints to show the physical address of the base register or we could
>> possibly drop the prints entirely.
>
> Displaying the PA can be useful if you have several identical blocks in
> your system and you want to be able to identify them. Given that there
> is probably only one of these controllers per system, the address is
> pretty pointless.

Multiple instances are actually quite common in the STB SoCs, e.g.
bcm7362 has one instance of bcm7038-l1, two instances of bcm7120-l2
and four instances of brcmstb-l2.


Regards
Jonas

P.S: Also What about bcm6345-l1? It also prints it's mapped VAs, not the PAs.

Re: [PATCH] irqchip: Use %px to print pointer value

2018-02-12 Thread Jonas Gorski 
On 9 February 2018 at 17:04, Marc Zyngier  wrote:
> On 09/02/18 15:54, Florian Fainelli wrote:
>> On February 9, 2018 12:51:33 AM PST, Marc Zyngier  
>> wrote:
>>> On 09/02/18 02:10, Jaedon Shin wrote:
 Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
 pointers printed with %p are hashed. Use %px instead of %p to print
 pointer value.

 Signed-off-by: Jaedon Shin 
 ---
  drivers/irqchip/irq-bcm7038-l1.c | 2 +-
  drivers/irqchip/irq-bcm7120-l2.c | 2 +-
  drivers/irqchip/irq-brcmstb-l2.c | 2 +-
  3 files changed, 3 insertions(+), 3 deletions(-)

 diff --git a/drivers/irqchip/irq-bcm7038-l1.c
>>> b/drivers/irqchip/irq-bcm7038-l1.c
 index 55cfb986225b..f604c1d89b3b 100644
 --- a/drivers/irqchip/irq-bcm7038-l1.c
 +++ b/drivers/irqchip/irq-bcm7038-l1.c
 @@ -339,7 +339,7 @@ int __init bcm7038_l1_of_init(struct device_node
>>> *dn,
 goto out_unmap;
 }

 -   pr_info("registered BCM7038 L1 intc (mem: 0x%p, IRQs: %d)\n",
 +   pr_info("registered BCM7038 L1 intc (mem: 0x%px, IRQs: %d)\n",
 intc->cpus[0]->map_base, IRQS_PER_WORD * intc->n_words);

 return 0;
 diff --git a/drivers/irqchip/irq-bcm7120-l2.c
>>> b/drivers/irqchip/irq-bcm7120-l2.c
 index 983640eba418..1cc4dd1d584a 100644
 --- a/drivers/irqchip/irq-bcm7120-l2.c
 +++ b/drivers/irqchip/irq-bcm7120-l2.c
 @@ -318,7 +318,7 @@ static int __init bcm7120_l2_intc_probe(struct
>>> device_node *dn,
 }
 }

 -   pr_info("registered %s intc (mem: 0x%p, parent IRQ(s): %d)\n",
 +   pr_info("registered %s intc (mem: 0x%px, parent IRQ(s): %d)\n",
 intc_name, data->map_base[0], data->num_parent_irqs);

 return 0;
 diff --git a/drivers/irqchip/irq-brcmstb-l2.c
>>> b/drivers/irqchip/irq-brcmstb-l2.c
 index 691d20eb0bec..6760edeeb666 100644
 --- a/drivers/irqchip/irq-brcmstb-l2.c
 +++ b/drivers/irqchip/irq-brcmstb-l2.c
 @@ -262,7 +262,7 @@ static int __init brcmstb_l2_intc_of_init(struct
>>> device_node *np,
 ct->chip.irq_set_wake = irq_gc_set_wake;
 }

 -   pr_info("registered L2 intc (mem: 0x%p, parent irq: %d)\n",
 +   pr_info("registered L2 intc (mem: 0x%px, parent irq: %d)\n",
 base, parent_irq);

 return 0;

>>>
>>> Why is that something useful to do? This just tells you where the
>>> device
>>> is mapped in the VA space, and I doubt that's a useful information,
>>> hashed pointers or not. Am I missing something obvious?
>>
>> No you are right there is not much value in printing the register
>> virtual address (sometimes there is e.g: on MIPS) either we fix the
>> prints to show the physical address of the base register or we could
>> possibly drop the prints entirely.
>
> Displaying the PA can be useful if you have several identical blocks in
> your system and you want to be able to identify them. Given that there
> is probably only one of these controllers per system, the address is
> pretty pointless.

Multiple instances are actually quite common in the STB SoCs, e.g.
bcm7362 has one instance of bcm7038-l1, two instances of bcm7120-l2
and four instances of brcmstb-l2.


Regards
Jonas

P.S: Also What about bcm6345-l1? It also prints it's mapped VAs, not the PAs.

Re: [PATCH] irqchip: Use %px to print pointer value

2018-02-09 Thread Jaedon Shin 


> On 10 Feb 2018, at 1:04 AM, Marc Zyngier  wrote:
> 
> On 09/02/18 15:54, Florian Fainelli wrote:
>> On February 9, 2018 12:51:33 AM PST, Marc Zyngier  
>> wrote:
>>> On 09/02/18 02:10, Jaedon Shin wrote:
 Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
 pointers printed with %p are hashed. Use %px instead of %p to print
 pointer value.
 
 Signed-off-by: Jaedon Shin 
 ---
 drivers/irqchip/irq-bcm7038-l1.c | 2 +-
 drivers/irqchip/irq-bcm7120-l2.c | 2 +-
 drivers/irqchip/irq-brcmstb-l2.c | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)
 
 diff --git a/drivers/irqchip/irq-bcm7038-l1.c
>>> b/drivers/irqchip/irq-bcm7038-l1.c
 index 55cfb986225b..f604c1d89b3b 100644
 --- a/drivers/irqchip/irq-bcm7038-l1.c
 +++ b/drivers/irqchip/irq-bcm7038-l1.c
 @@ -339,7 +339,7 @@ int __init bcm7038_l1_of_init(struct device_node
>>> *dn,
goto out_unmap;
}
 
 -  pr_info("registered BCM7038 L1 intc (mem: 0x%p, IRQs: %d)\n",
 +  pr_info("registered BCM7038 L1 intc (mem: 0x%px, IRQs: %d)\n",
intc->cpus[0]->map_base, IRQS_PER_WORD * intc->n_words);
 
return 0;
 diff --git a/drivers/irqchip/irq-bcm7120-l2.c
>>> b/drivers/irqchip/irq-bcm7120-l2.c
 index 983640eba418..1cc4dd1d584a 100644
 --- a/drivers/irqchip/irq-bcm7120-l2.c
 +++ b/drivers/irqchip/irq-bcm7120-l2.c
 @@ -318,7 +318,7 @@ static int __init bcm7120_l2_intc_probe(struct
>>> device_node *dn,
}
}
 
 -  pr_info("registered %s intc (mem: 0x%p, parent IRQ(s): %d)\n",
 +  pr_info("registered %s intc (mem: 0x%px, parent IRQ(s): %d)\n",
intc_name, data->map_base[0], data->num_parent_irqs);
 
return 0;
 diff --git a/drivers/irqchip/irq-brcmstb-l2.c
>>> b/drivers/irqchip/irq-brcmstb-l2.c
 index 691d20eb0bec..6760edeeb666 100644
 --- a/drivers/irqchip/irq-brcmstb-l2.c
 +++ b/drivers/irqchip/irq-brcmstb-l2.c
 @@ -262,7 +262,7 @@ static int __init brcmstb_l2_intc_of_init(struct
>>> device_node *np,
ct->chip.irq_set_wake = irq_gc_set_wake;
}
 
 -  pr_info("registered L2 intc (mem: 0x%p, parent irq: %d)\n",
 +  pr_info("registered L2 intc (mem: 0x%px, parent irq: %d)\n",
base, parent_irq);
 
return 0;
 
>>> 
>>> Why is that something useful to do? This just tells you where the
>>> device
>>> is mapped in the VA space, and I doubt that's a useful information,
>>> hashed pointers or not. Am I missing something obvious?
>> 
>> No you are right there is not much value in printing the register
>> virtual address (sometimes there is e.g: on MIPS) either we fix the
>> prints to show the physical address of the base register or we could
>> possibly drop the prints entirely.
> 
> Displaying the PA can be useful if you have several identical blocks in
> your system and you want to be able to identify them. Given that there
> is probably only one of these controllers per system, the address is
> pretty pointless.
> 
> If you send me a patch removing the prints, I'll queue it.
> 
> Thanks,
> 
>   M.
> -- 
> Jazz is not dead. It just smells funny...

I'll submit a new one to drop the entire print.

Thanks,
Jaedon

Re: [PATCH] irqchip: Use %px to print pointer value

2018-02-09 Thread Jaedon Shin 


> On 10 Feb 2018, at 1:04 AM, Marc Zyngier  wrote:
> 
> On 09/02/18 15:54, Florian Fainelli wrote:
>> On February 9, 2018 12:51:33 AM PST, Marc Zyngier  
>> wrote:
>>> On 09/02/18 02:10, Jaedon Shin wrote:
 Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
 pointers printed with %p are hashed. Use %px instead of %p to print
 pointer value.
 
 Signed-off-by: Jaedon Shin 
 ---
 drivers/irqchip/irq-bcm7038-l1.c | 2 +-
 drivers/irqchip/irq-bcm7120-l2.c | 2 +-
 drivers/irqchip/irq-brcmstb-l2.c | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)
 
 diff --git a/drivers/irqchip/irq-bcm7038-l1.c
>>> b/drivers/irqchip/irq-bcm7038-l1.c
 index 55cfb986225b..f604c1d89b3b 100644
 --- a/drivers/irqchip/irq-bcm7038-l1.c
 +++ b/drivers/irqchip/irq-bcm7038-l1.c
 @@ -339,7 +339,7 @@ int __init bcm7038_l1_of_init(struct device_node
>>> *dn,
goto out_unmap;
}
 
 -  pr_info("registered BCM7038 L1 intc (mem: 0x%p, IRQs: %d)\n",
 +  pr_info("registered BCM7038 L1 intc (mem: 0x%px, IRQs: %d)\n",
intc->cpus[0]->map_base, IRQS_PER_WORD * intc->n_words);
 
return 0;
 diff --git a/drivers/irqchip/irq-bcm7120-l2.c
>>> b/drivers/irqchip/irq-bcm7120-l2.c
 index 983640eba418..1cc4dd1d584a 100644
 --- a/drivers/irqchip/irq-bcm7120-l2.c
 +++ b/drivers/irqchip/irq-bcm7120-l2.c
 @@ -318,7 +318,7 @@ static int __init bcm7120_l2_intc_probe(struct
>>> device_node *dn,
}
}
 
 -  pr_info("registered %s intc (mem: 0x%p, parent IRQ(s): %d)\n",
 +  pr_info("registered %s intc (mem: 0x%px, parent IRQ(s): %d)\n",
intc_name, data->map_base[0], data->num_parent_irqs);
 
return 0;
 diff --git a/drivers/irqchip/irq-brcmstb-l2.c
>>> b/drivers/irqchip/irq-brcmstb-l2.c
 index 691d20eb0bec..6760edeeb666 100644
 --- a/drivers/irqchip/irq-brcmstb-l2.c
 +++ b/drivers/irqchip/irq-brcmstb-l2.c
 @@ -262,7 +262,7 @@ static int __init brcmstb_l2_intc_of_init(struct
>>> device_node *np,
ct->chip.irq_set_wake = irq_gc_set_wake;
}
 
 -  pr_info("registered L2 intc (mem: 0x%p, parent irq: %d)\n",
 +  pr_info("registered L2 intc (mem: 0x%px, parent irq: %d)\n",
base, parent_irq);
 
return 0;
 
>>> 
>>> Why is that something useful to do? This just tells you where the
>>> device
>>> is mapped in the VA space, and I doubt that's a useful information,
>>> hashed pointers or not. Am I missing something obvious?
>> 
>> No you are right there is not much value in printing the register
>> virtual address (sometimes there is e.g: on MIPS) either we fix the
>> prints to show the physical address of the base register or we could
>> possibly drop the prints entirely.
> 
> Displaying the PA can be useful if you have several identical blocks in
> your system and you want to be able to identify them. Given that there
> is probably only one of these controllers per system, the address is
> pretty pointless.
> 
> If you send me a patch removing the prints, I'll queue it.
> 
> Thanks,
> 
>   M.
> -- 
> Jazz is not dead. It just smells funny...

I'll submit a new one to drop the entire print.

Thanks,
Jaedon

Re: [PATCH] irqchip: Use %px to print pointer value

2018-02-09 Thread Marc Zyngier 
On 09/02/18 15:54, Florian Fainelli wrote:
> On February 9, 2018 12:51:33 AM PST, Marc Zyngier  
> wrote:
>> On 09/02/18 02:10, Jaedon Shin wrote:
>>> Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
>>> pointers printed with %p are hashed. Use %px instead of %p to print
>>> pointer value.
>>>
>>> Signed-off-by: Jaedon Shin 
>>> ---
>>>  drivers/irqchip/irq-bcm7038-l1.c | 2 +-
>>>  drivers/irqchip/irq-bcm7120-l2.c | 2 +-
>>>  drivers/irqchip/irq-brcmstb-l2.c | 2 +-
>>>  3 files changed, 3 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/drivers/irqchip/irq-bcm7038-l1.c
>> b/drivers/irqchip/irq-bcm7038-l1.c
>>> index 55cfb986225b..f604c1d89b3b 100644
>>> --- a/drivers/irqchip/irq-bcm7038-l1.c
>>> +++ b/drivers/irqchip/irq-bcm7038-l1.c
>>> @@ -339,7 +339,7 @@ int __init bcm7038_l1_of_init(struct device_node
>> *dn,
>>> goto out_unmap;
>>> }
>>>  
>>> -   pr_info("registered BCM7038 L1 intc (mem: 0x%p, IRQs: %d)\n",
>>> +   pr_info("registered BCM7038 L1 intc (mem: 0x%px, IRQs: %d)\n",
>>> intc->cpus[0]->map_base, IRQS_PER_WORD * intc->n_words);
>>>  
>>> return 0;
>>> diff --git a/drivers/irqchip/irq-bcm7120-l2.c
>> b/drivers/irqchip/irq-bcm7120-l2.c
>>> index 983640eba418..1cc4dd1d584a 100644
>>> --- a/drivers/irqchip/irq-bcm7120-l2.c
>>> +++ b/drivers/irqchip/irq-bcm7120-l2.c
>>> @@ -318,7 +318,7 @@ static int __init bcm7120_l2_intc_probe(struct
>> device_node *dn,
>>> }
>>> }
>>>  
>>> -   pr_info("registered %s intc (mem: 0x%p, parent IRQ(s): %d)\n",
>>> +   pr_info("registered %s intc (mem: 0x%px, parent IRQ(s): %d)\n",
>>> intc_name, data->map_base[0], data->num_parent_irqs);
>>>  
>>> return 0;
>>> diff --git a/drivers/irqchip/irq-brcmstb-l2.c
>> b/drivers/irqchip/irq-brcmstb-l2.c
>>> index 691d20eb0bec..6760edeeb666 100644
>>> --- a/drivers/irqchip/irq-brcmstb-l2.c
>>> +++ b/drivers/irqchip/irq-brcmstb-l2.c
>>> @@ -262,7 +262,7 @@ static int __init brcmstb_l2_intc_of_init(struct
>> device_node *np,
>>> ct->chip.irq_set_wake = irq_gc_set_wake;
>>> }
>>>  
>>> -   pr_info("registered L2 intc (mem: 0x%p, parent irq: %d)\n",
>>> +   pr_info("registered L2 intc (mem: 0x%px, parent irq: %d)\n",
>>> base, parent_irq);
>>>  
>>> return 0;
>>>
>>
>> Why is that something useful to do? This just tells you where the
>> device
>> is mapped in the VA space, and I doubt that's a useful information,
>> hashed pointers or not. Am I missing something obvious?
> 
> No you are right there is not much value in printing the register
> virtual address (sometimes there is e.g: on MIPS) either we fix the
> prints to show the physical address of the base register or we could
> possibly drop the prints entirely.

Displaying the PA can be useful if you have several identical blocks in
your system and you want to be able to identify them. Given that there
is probably only one of these controllers per system, the address is
pretty pointless.

If you send me a patch removing the prints, I'll queue it.

Thanks,

M.
-- 
Jazz is not dead. It just smells funny...

Re: [PATCH] irqchip: Use %px to print pointer value

2018-02-09 Thread Marc Zyngier 
On 09/02/18 15:54, Florian Fainelli wrote:
> On February 9, 2018 12:51:33 AM PST, Marc Zyngier  
> wrote:
>> On 09/02/18 02:10, Jaedon Shin wrote:
>>> Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
>>> pointers printed with %p are hashed. Use %px instead of %p to print
>>> pointer value.
>>>
>>> Signed-off-by: Jaedon Shin 
>>> ---
>>>  drivers/irqchip/irq-bcm7038-l1.c | 2 +-
>>>  drivers/irqchip/irq-bcm7120-l2.c | 2 +-
>>>  drivers/irqchip/irq-brcmstb-l2.c | 2 +-
>>>  3 files changed, 3 insertions(+), 3 deletions(-)
>>>
>>> diff --git a/drivers/irqchip/irq-bcm7038-l1.c
>> b/drivers/irqchip/irq-bcm7038-l1.c
>>> index 55cfb986225b..f604c1d89b3b 100644
>>> --- a/drivers/irqchip/irq-bcm7038-l1.c
>>> +++ b/drivers/irqchip/irq-bcm7038-l1.c
>>> @@ -339,7 +339,7 @@ int __init bcm7038_l1_of_init(struct device_node
>> *dn,
>>> goto out_unmap;
>>> }
>>>  
>>> -   pr_info("registered BCM7038 L1 intc (mem: 0x%p, IRQs: %d)\n",
>>> +   pr_info("registered BCM7038 L1 intc (mem: 0x%px, IRQs: %d)\n",
>>> intc->cpus[0]->map_base, IRQS_PER_WORD * intc->n_words);
>>>  
>>> return 0;
>>> diff --git a/drivers/irqchip/irq-bcm7120-l2.c
>> b/drivers/irqchip/irq-bcm7120-l2.c
>>> index 983640eba418..1cc4dd1d584a 100644
>>> --- a/drivers/irqchip/irq-bcm7120-l2.c
>>> +++ b/drivers/irqchip/irq-bcm7120-l2.c
>>> @@ -318,7 +318,7 @@ static int __init bcm7120_l2_intc_probe(struct
>> device_node *dn,
>>> }
>>> }
>>>  
>>> -   pr_info("registered %s intc (mem: 0x%p, parent IRQ(s): %d)\n",
>>> +   pr_info("registered %s intc (mem: 0x%px, parent IRQ(s): %d)\n",
>>> intc_name, data->map_base[0], data->num_parent_irqs);
>>>  
>>> return 0;
>>> diff --git a/drivers/irqchip/irq-brcmstb-l2.c
>> b/drivers/irqchip/irq-brcmstb-l2.c
>>> index 691d20eb0bec..6760edeeb666 100644
>>> --- a/drivers/irqchip/irq-brcmstb-l2.c
>>> +++ b/drivers/irqchip/irq-brcmstb-l2.c
>>> @@ -262,7 +262,7 @@ static int __init brcmstb_l2_intc_of_init(struct
>> device_node *np,
>>> ct->chip.irq_set_wake = irq_gc_set_wake;
>>> }
>>>  
>>> -   pr_info("registered L2 intc (mem: 0x%p, parent irq: %d)\n",
>>> +   pr_info("registered L2 intc (mem: 0x%px, parent irq: %d)\n",
>>> base, parent_irq);
>>>  
>>> return 0;
>>>
>>
>> Why is that something useful to do? This just tells you where the
>> device
>> is mapped in the VA space, and I doubt that's a useful information,
>> hashed pointers or not. Am I missing something obvious?
> 
> No you are right there is not much value in printing the register
> virtual address (sometimes there is e.g: on MIPS) either we fix the
> prints to show the physical address of the base register or we could
> possibly drop the prints entirely.

Displaying the PA can be useful if you have several identical blocks in
your system and you want to be able to identify them. Given that there
is probably only one of these controllers per system, the address is
pretty pointless.

If you send me a patch removing the prints, I'll queue it.

Thanks,

M.
-- 
Jazz is not dead. It just smells funny...

Re: [PATCH] irqchip: Use %px to print pointer value

2018-02-09 Thread Florian Fainelli 
On February 9, 2018 12:51:33 AM PST, Marc Zyngier  wrote:
>On 09/02/18 02:10, Jaedon Shin wrote:
>> Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
>> pointers printed with %p are hashed. Use %px instead of %p to print
>> pointer value.
>> 
>> Signed-off-by: Jaedon Shin 
>> ---
>>  drivers/irqchip/irq-bcm7038-l1.c | 2 +-
>>  drivers/irqchip/irq-bcm7120-l2.c | 2 +-
>>  drivers/irqchip/irq-brcmstb-l2.c | 2 +-
>>  3 files changed, 3 insertions(+), 3 deletions(-)
>> 
>> diff --git a/drivers/irqchip/irq-bcm7038-l1.c
>b/drivers/irqchip/irq-bcm7038-l1.c
>> index 55cfb986225b..f604c1d89b3b 100644
>> --- a/drivers/irqchip/irq-bcm7038-l1.c
>> +++ b/drivers/irqchip/irq-bcm7038-l1.c
>> @@ -339,7 +339,7 @@ int __init bcm7038_l1_of_init(struct device_node
>*dn,
>>  goto out_unmap;
>>  }
>>  
>> -pr_info("registered BCM7038 L1 intc (mem: 0x%p, IRQs: %d)\n",
>> +pr_info("registered BCM7038 L1 intc (mem: 0x%px, IRQs: %d)\n",
>>  intc->cpus[0]->map_base, IRQS_PER_WORD * intc->n_words);
>>  
>>  return 0;
>> diff --git a/drivers/irqchip/irq-bcm7120-l2.c
>b/drivers/irqchip/irq-bcm7120-l2.c
>> index 983640eba418..1cc4dd1d584a 100644
>> --- a/drivers/irqchip/irq-bcm7120-l2.c
>> +++ b/drivers/irqchip/irq-bcm7120-l2.c
>> @@ -318,7 +318,7 @@ static int __init bcm7120_l2_intc_probe(struct
>device_node *dn,
>>  }
>>  }
>>  
>> -pr_info("registered %s intc (mem: 0x%p, parent IRQ(s): %d)\n",
>> +pr_info("registered %s intc (mem: 0x%px, parent IRQ(s): %d)\n",
>>  intc_name, data->map_base[0], data->num_parent_irqs);
>>  
>>  return 0;
>> diff --git a/drivers/irqchip/irq-brcmstb-l2.c
>b/drivers/irqchip/irq-brcmstb-l2.c
>> index 691d20eb0bec..6760edeeb666 100644
>> --- a/drivers/irqchip/irq-brcmstb-l2.c
>> +++ b/drivers/irqchip/irq-brcmstb-l2.c
>> @@ -262,7 +262,7 @@ static int __init brcmstb_l2_intc_of_init(struct
>device_node *np,
>>  ct->chip.irq_set_wake = irq_gc_set_wake;
>>  }
>>  
>> -pr_info("registered L2 intc (mem: 0x%p, parent irq: %d)\n",
>> +pr_info("registered L2 intc (mem: 0x%px, parent irq: %d)\n",
>>  base, parent_irq);
>>  
>>  return 0;
>> 
>
>Why is that something useful to do? This just tells you where the
>device
>is mapped in the VA space, and I doubt that's a useful information,
>hashed pointers or not. Am I missing something obvious?

No you are right there is not much value in printing the register virtual 
address (sometimes there is e.g: on MIPS) either we fix the prints to show the 
physical address of the base register or we could possibly drop the prints 
entirely.

-- 
Florian

Re: [PATCH] irqchip: Use %px to print pointer value

2018-02-09 Thread Florian Fainelli 
On February 9, 2018 12:51:33 AM PST, Marc Zyngier  wrote:
>On 09/02/18 02:10, Jaedon Shin wrote:
>> Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
>> pointers printed with %p are hashed. Use %px instead of %p to print
>> pointer value.
>> 
>> Signed-off-by: Jaedon Shin 
>> ---
>>  drivers/irqchip/irq-bcm7038-l1.c | 2 +-
>>  drivers/irqchip/irq-bcm7120-l2.c | 2 +-
>>  drivers/irqchip/irq-brcmstb-l2.c | 2 +-
>>  3 files changed, 3 insertions(+), 3 deletions(-)
>> 
>> diff --git a/drivers/irqchip/irq-bcm7038-l1.c
>b/drivers/irqchip/irq-bcm7038-l1.c
>> index 55cfb986225b..f604c1d89b3b 100644
>> --- a/drivers/irqchip/irq-bcm7038-l1.c
>> +++ b/drivers/irqchip/irq-bcm7038-l1.c
>> @@ -339,7 +339,7 @@ int __init bcm7038_l1_of_init(struct device_node
>*dn,
>>  goto out_unmap;
>>  }
>>  
>> -pr_info("registered BCM7038 L1 intc (mem: 0x%p, IRQs: %d)\n",
>> +pr_info("registered BCM7038 L1 intc (mem: 0x%px, IRQs: %d)\n",
>>  intc->cpus[0]->map_base, IRQS_PER_WORD * intc->n_words);
>>  
>>  return 0;
>> diff --git a/drivers/irqchip/irq-bcm7120-l2.c
>b/drivers/irqchip/irq-bcm7120-l2.c
>> index 983640eba418..1cc4dd1d584a 100644
>> --- a/drivers/irqchip/irq-bcm7120-l2.c
>> +++ b/drivers/irqchip/irq-bcm7120-l2.c
>> @@ -318,7 +318,7 @@ static int __init bcm7120_l2_intc_probe(struct
>device_node *dn,
>>  }
>>  }
>>  
>> -pr_info("registered %s intc (mem: 0x%p, parent IRQ(s): %d)\n",
>> +pr_info("registered %s intc (mem: 0x%px, parent IRQ(s): %d)\n",
>>  intc_name, data->map_base[0], data->num_parent_irqs);
>>  
>>  return 0;
>> diff --git a/drivers/irqchip/irq-brcmstb-l2.c
>b/drivers/irqchip/irq-brcmstb-l2.c
>> index 691d20eb0bec..6760edeeb666 100644
>> --- a/drivers/irqchip/irq-brcmstb-l2.c
>> +++ b/drivers/irqchip/irq-brcmstb-l2.c
>> @@ -262,7 +262,7 @@ static int __init brcmstb_l2_intc_of_init(struct
>device_node *np,
>>  ct->chip.irq_set_wake = irq_gc_set_wake;
>>  }
>>  
>> -pr_info("registered L2 intc (mem: 0x%p, parent irq: %d)\n",
>> +pr_info("registered L2 intc (mem: 0x%px, parent irq: %d)\n",
>>  base, parent_irq);
>>  
>>  return 0;
>> 
>
>Why is that something useful to do? This just tells you where the
>device
>is mapped in the VA space, and I doubt that's a useful information,
>hashed pointers or not. Am I missing something obvious?

No you are right there is not much value in printing the register virtual 
address (sometimes there is e.g: on MIPS) either we fix the prints to show the 
physical address of the base register or we could possibly drop the prints 
entirely.

-- 
Florian

Re: [PATCH] irqchip: Use %px to print pointer value

2018-02-09 Thread Marc Zyngier 
On 09/02/18 02:10, Jaedon Shin wrote:
> Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
> pointers printed with %p are hashed. Use %px instead of %p to print
> pointer value.
> 
> Signed-off-by: Jaedon Shin 
> ---
>  drivers/irqchip/irq-bcm7038-l1.c | 2 +-
>  drivers/irqchip/irq-bcm7120-l2.c | 2 +-
>  drivers/irqchip/irq-brcmstb-l2.c | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/irqchip/irq-bcm7038-l1.c 
> b/drivers/irqchip/irq-bcm7038-l1.c
> index 55cfb986225b..f604c1d89b3b 100644
> --- a/drivers/irqchip/irq-bcm7038-l1.c
> +++ b/drivers/irqchip/irq-bcm7038-l1.c
> @@ -339,7 +339,7 @@ int __init bcm7038_l1_of_init(struct device_node *dn,
>   goto out_unmap;
>   }
>  
> - pr_info("registered BCM7038 L1 intc (mem: 0x%p, IRQs: %d)\n",
> + pr_info("registered BCM7038 L1 intc (mem: 0x%px, IRQs: %d)\n",
>   intc->cpus[0]->map_base, IRQS_PER_WORD * intc->n_words);
>  
>   return 0;
> diff --git a/drivers/irqchip/irq-bcm7120-l2.c 
> b/drivers/irqchip/irq-bcm7120-l2.c
> index 983640eba418..1cc4dd1d584a 100644
> --- a/drivers/irqchip/irq-bcm7120-l2.c
> +++ b/drivers/irqchip/irq-bcm7120-l2.c
> @@ -318,7 +318,7 @@ static int __init bcm7120_l2_intc_probe(struct 
> device_node *dn,
>   }
>   }
>  
> - pr_info("registered %s intc (mem: 0x%p, parent IRQ(s): %d)\n",
> + pr_info("registered %s intc (mem: 0x%px, parent IRQ(s): %d)\n",
>   intc_name, data->map_base[0], data->num_parent_irqs);
>  
>   return 0;
> diff --git a/drivers/irqchip/irq-brcmstb-l2.c 
> b/drivers/irqchip/irq-brcmstb-l2.c
> index 691d20eb0bec..6760edeeb666 100644
> --- a/drivers/irqchip/irq-brcmstb-l2.c
> +++ b/drivers/irqchip/irq-brcmstb-l2.c
> @@ -262,7 +262,7 @@ static int __init brcmstb_l2_intc_of_init(struct 
> device_node *np,
>   ct->chip.irq_set_wake = irq_gc_set_wake;
>   }
>  
> - pr_info("registered L2 intc (mem: 0x%p, parent irq: %d)\n",
> + pr_info("registered L2 intc (mem: 0x%px, parent irq: %d)\n",
>   base, parent_irq);
>  
>   return 0;
> 

Why is that something useful to do? This just tells you where the device
is mapped in the VA space, and I doubt that's a useful information,
hashed pointers or not. Am I missing something obvious?

Thanks,

M.
-- 
Jazz is not dead. It just smells funny...

Re: [PATCH] irqchip: Use %px to print pointer value

2018-02-09 Thread Marc Zyngier 
On 09/02/18 02:10, Jaedon Shin wrote:
> Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
> pointers printed with %p are hashed. Use %px instead of %p to print
> pointer value.
> 
> Signed-off-by: Jaedon Shin 
> ---
>  drivers/irqchip/irq-bcm7038-l1.c | 2 +-
>  drivers/irqchip/irq-bcm7120-l2.c | 2 +-
>  drivers/irqchip/irq-brcmstb-l2.c | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/drivers/irqchip/irq-bcm7038-l1.c 
> b/drivers/irqchip/irq-bcm7038-l1.c
> index 55cfb986225b..f604c1d89b3b 100644
> --- a/drivers/irqchip/irq-bcm7038-l1.c
> +++ b/drivers/irqchip/irq-bcm7038-l1.c
> @@ -339,7 +339,7 @@ int __init bcm7038_l1_of_init(struct device_node *dn,
>   goto out_unmap;
>   }
>  
> - pr_info("registered BCM7038 L1 intc (mem: 0x%p, IRQs: %d)\n",
> + pr_info("registered BCM7038 L1 intc (mem: 0x%px, IRQs: %d)\n",
>   intc->cpus[0]->map_base, IRQS_PER_WORD * intc->n_words);
>  
>   return 0;
> diff --git a/drivers/irqchip/irq-bcm7120-l2.c 
> b/drivers/irqchip/irq-bcm7120-l2.c
> index 983640eba418..1cc4dd1d584a 100644
> --- a/drivers/irqchip/irq-bcm7120-l2.c
> +++ b/drivers/irqchip/irq-bcm7120-l2.c
> @@ -318,7 +318,7 @@ static int __init bcm7120_l2_intc_probe(struct 
> device_node *dn,
>   }
>   }
>  
> - pr_info("registered %s intc (mem: 0x%p, parent IRQ(s): %d)\n",
> + pr_info("registered %s intc (mem: 0x%px, parent IRQ(s): %d)\n",
>   intc_name, data->map_base[0], data->num_parent_irqs);
>  
>   return 0;
> diff --git a/drivers/irqchip/irq-brcmstb-l2.c 
> b/drivers/irqchip/irq-brcmstb-l2.c
> index 691d20eb0bec..6760edeeb666 100644
> --- a/drivers/irqchip/irq-brcmstb-l2.c
> +++ b/drivers/irqchip/irq-brcmstb-l2.c
> @@ -262,7 +262,7 @@ static int __init brcmstb_l2_intc_of_init(struct 
> device_node *np,
>   ct->chip.irq_set_wake = irq_gc_set_wake;
>   }
>  
> - pr_info("registered L2 intc (mem: 0x%p, parent irq: %d)\n",
> + pr_info("registered L2 intc (mem: 0x%px, parent irq: %d)\n",
>   base, parent_irq);
>  
>   return 0;
> 

Why is that something useful to do? This just tells you where the device
is mapped in the VA space, and I doubt that's a useful information,
hashed pointers or not. Am I missing something obvious?

Thanks,

M.
-- 
Jazz is not dead. It just smells funny...

Re: [PATCH] irqchip: Use %px to print pointer value

2018-02-09 Thread Geert Uytterhoeven 
Hi Jaedon,

On Fri, Feb 9, 2018 at 3:10 AM, Jaedon Shin  wrote:
> Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
> pointers printed with %p are hashed. Use %px instead of %p to print
> pointer value.
>
> Signed-off-by: Jaedon Shin 

Thanks for your patch!

> --- a/drivers/irqchip/irq-bcm7038-l1.c
> +++ b/drivers/irqchip/irq-bcm7038-l1.c
> @@ -339,7 +339,7 @@ int __init bcm7038_l1_of_init(struct device_node *dn,
> goto out_unmap;
> }
>
> -   pr_info("registered BCM7038 L1 intc (mem: 0x%p, IRQs: %d)\n",
> +   pr_info("registered BCM7038 L1 intc (mem: 0x%px, IRQs: %d)\n",
> intc->cpus[0]->map_base, IRQS_PER_WORD * intc->n_words);

I believe the printing of this value should just be removed.
Printing virtual addresses of mapped registers is usually not helpful, except
for an attacker.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

Re: [PATCH] irqchip: Use %px to print pointer value

2018-02-09 Thread Geert Uytterhoeven 
Hi Jaedon,

On Fri, Feb 9, 2018 at 3:10 AM, Jaedon Shin  wrote:
> Since commit ad67b74d2469 ("printk: hash addresses printed with %p")
> pointers printed with %p are hashed. Use %px instead of %p to print
> pointer value.
>
> Signed-off-by: Jaedon Shin 

Thanks for your patch!

> --- a/drivers/irqchip/irq-bcm7038-l1.c
> +++ b/drivers/irqchip/irq-bcm7038-l1.c
> @@ -339,7 +339,7 @@ int __init bcm7038_l1_of_init(struct device_node *dn,
> goto out_unmap;
> }
>
> -   pr_info("registered BCM7038 L1 intc (mem: 0x%p, IRQs: %d)\n",
> +   pr_info("registered BCM7038 L1 intc (mem: 0x%px, IRQs: %d)\n",
> intc->cpus[0]->map_base, IRQS_PER_WORD * intc->n_words);

I believe the printing of this value should just be removed.
Printing virtual addresses of mapped registers is usually not helpful, except
for an attacker.

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds