Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1198453 > unrolled thread
| Started by | Joe Perches <joe@perches.com> |
|---|---|
| First post | 2015-08-02 22:30 +0200 |
| Last post | 2015-08-04 06:10 +0200 |
| Articles | 8 — 3 participants |
Back to article view | Back to linux.kernel
[TRIVIAL PATCH] block: Correct misuses of 0x%<decimal> Joe Perches <joe@perches.com> - 2015-08-02 22:30 +0200
Re: [TRIVIAL PATCH] block: Correct misuses of 0x%<decimal> Johannes Thumshirn <jthumshirn@suse.de> - 2015-08-03 09:30 +0200
Re: [TRIVIAL PATCH] block: Correct misuses of 0x%<decimal> yalin wang <yalin.wang2010@gmail.com> - 2015-08-03 09:30 +0200
Re: [TRIVIAL PATCH] block: Correct misuses of 0x%<decimal> Joe Perches <joe@perches.com> - 2015-08-03 10:10 +0200
Re: [TRIVIAL PATCH] block: Correct misuses of 0x%<decimal> yalin wang <yalin.wang2010@gmail.com> - 2015-08-03 11:00 +0200
Re: [TRIVIAL PATCH] block: Correct misuses of 0x%<decimal> yalin wang <yalin.wang2010@gmail.com> - 2015-08-04 04:20 +0200
Re: [TRIVIAL PATCH] block: Correct misuses of 0x%<decimal> Joe Perches <joe@perches.com> - 2015-08-04 04:30 +0200
Re: [TRIVIAL PATCH] block: Correct misuses of 0x%<decimal> yalin wang <yalin.wang2010@gmail.com> - 2015-08-04 06:10 +0200
| From | Joe Perches <joe@perches.com> |
|---|---|
| Date | 2015-08-02 22:30 +0200 |
| Subject | [TRIVIAL PATCH] block: Correct misuses of 0x%<decimal> |
| Message-ID | <pT83o-6i0-9@gated-at.bofh.it> |
Correct misuse of 0x%d in logging messages.
Signed-off-by: Joe Perches <joe@perches.com>
---
drivers/block/DAC960.c | 4 ++--
drivers/block/cciss.c | 2 +-
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/block/DAC960.c b/drivers/block/DAC960.c
index 811e11c..d9b32f2 100644
--- a/drivers/block/DAC960.c
+++ b/drivers/block/DAC960.c
@@ -2954,7 +2954,7 @@ DAC960_DetectController(struct pci_dev *PCI_Device,
case DAC960_PD_Controller:
if (!request_region(Controller->IO_Address, 0x80,
Controller->FullModelName)) {
- DAC960_Error("IO port 0x%d busy for Controller at\n",
+ DAC960_Error("IO port 0x%x busy for Controller at\n",
Controller, Controller->IO_Address);
goto Failure;
}
@@ -2990,7 +2990,7 @@ DAC960_DetectController(struct pci_dev *PCI_Device,
case DAC960_P_Controller:
if (!request_region(Controller->IO_Address, 0x80,
Controller->FullModelName)){
- DAC960_Error("IO port 0x%d busy for Controller at\n",
+ DAC960_Error("IO port 0x%x busy for Controller at\n",
Controller, Controller->IO_Address);
goto Failure;
}
diff --git a/drivers/block/cciss.c b/drivers/block/cciss.c
index 0422c47..2758982 100644
--- a/drivers/block/cciss.c
+++ b/drivers/block/cciss.c
@@ -3854,7 +3854,7 @@ static void print_cfg_table(ctlr_info_t *h)
readl(&(tb->HostWrite.CoalIntDelay)));
dev_dbg(&h->pdev->dev, " Coalesce Interrupt Count = 0x%x\n",
readl(&(tb->HostWrite.CoalIntCount)));
- dev_dbg(&h->pdev->dev, " Max outstanding commands = 0x%d\n",
+ dev_dbg(&h->pdev->dev, " Max outstanding commands = 0x%x\n",
readl(&(tb->CmdsOutMax)));
dev_dbg(&h->pdev->dev, " Bus Types = 0x%x\n",
readl(&(tb->BusTypes)));
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [next] | [standalone]
| From | Johannes Thumshirn <jthumshirn@suse.de> |
|---|---|
| Date | 2015-08-03 09:30 +0200 |
| Message-ID | <pTim5-4Ds-3@gated-at.bofh.it> |
| In reply to | #1198453 |
Joe Perches <joe@perches.com> writes:
> Correct misuse of 0x%d in logging messages.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/block/DAC960.c | 4 ++--
> drivers/block/cciss.c | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/block/DAC960.c b/drivers/block/DAC960.c
> index 811e11c..d9b32f2 100644
> --- a/drivers/block/DAC960.c
> +++ b/drivers/block/DAC960.c
> @@ -2954,7 +2954,7 @@ DAC960_DetectController(struct pci_dev *PCI_Device,
> case DAC960_PD_Controller:
> if (!request_region(Controller->IO_Address, 0x80,
> Controller->FullModelName)) {
> - DAC960_Error("IO port 0x%d busy for Controller at\n",
> + DAC960_Error("IO port 0x%x busy for Controller at\n",
> Controller, Controller->IO_Address);
> goto Failure;
> }
> @@ -2990,7 +2990,7 @@ DAC960_DetectController(struct pci_dev *PCI_Device,
> case DAC960_P_Controller:
> if (!request_region(Controller->IO_Address, 0x80,
> Controller->FullModelName)){
> - DAC960_Error("IO port 0x%d busy for Controller at\n",
> + DAC960_Error("IO port 0x%x busy for Controller at\n",
> Controller, Controller->IO_Address);
> goto Failure;
> }
> diff --git a/drivers/block/cciss.c b/drivers/block/cciss.c
> index 0422c47..2758982 100644
> --- a/drivers/block/cciss.c
> +++ b/drivers/block/cciss.c
> @@ -3854,7 +3854,7 @@ static void print_cfg_table(ctlr_info_t *h)
> readl(&(tb->HostWrite.CoalIntDelay)));
> dev_dbg(&h->pdev->dev, " Coalesce Interrupt Count = 0x%x\n",
> readl(&(tb->HostWrite.CoalIntCount)));
> - dev_dbg(&h->pdev->dev, " Max outstanding commands = 0x%d\n",
> + dev_dbg(&h->pdev->dev, " Max outstanding commands = 0x%x\n",
> readl(&(tb->CmdsOutMax)));
> dev_dbg(&h->pdev->dev, " Bus Types = 0x%x\n",
> readl(&(tb->BusTypes)));
> --
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
Reviewed-by: Johannes Thumshirn <jthumshirn@suse.de>
--
Johannes Thumshirn Storage
jthumshirn@suse.de +49 911 74053 689
SUSE LINUX GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: F. Imendörffer, J. Smithard, J. Guild, D. Upmanyu, G. Norton
HRB 21284 (AG Nürnberg)
Key fingerprint = EC38 9CAB C2C4 F25D 8600 D0D0 0393 969D 2D76 0850
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | yalin wang <yalin.wang2010@gmail.com> |
|---|---|
| Date | 2015-08-03 09:30 +0200 |
| Message-ID | <pTim5-4Ds-5@gated-at.bofh.it> |
| In reply to | #1198453 |
> On Aug 3, 2015, at 04:25, Joe Perches <joe@perches.com> wrote:
>
> Correct misuse of 0x%d in logging messages.
>
> Signed-off-by: Joe Perches <joe@perches.com>
> ---
> drivers/block/DAC960.c | 4 ++--
> drivers/block/cciss.c | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/block/DAC960.c b/drivers/block/DAC960.c
> index 811e11c..d9b32f2 100644
> --- a/drivers/block/DAC960.c
> +++ b/drivers/block/DAC960.c
> @@ -2954,7 +2954,7 @@ DAC960_DetectController(struct pci_dev *PCI_Device,
> case DAC960_PD_Controller:
> if (!request_region(Controller->IO_Address, 0x80,
> Controller->FullModelName)) {
> - DAC960_Error("IO port 0x%d busy for Controller at\n",
> + DAC960_Error("IO port 0x%x busy for Controller at\n",
> Controller, Controller->IO_Address);
> goto Failure;
> }
> @@ -2990,7 +2990,7 @@ DAC960_DetectController(struct pci_dev *PCI_Device,
> case DAC960_P_Controller:
> if (!request_region(Controller->IO_Address, 0x80,
> Controller->FullModelName)){
> - DAC960_Error("IO port 0x%d busy for Controller at\n",
> + DAC960_Error("IO port 0x%x busy for Controller at\n",
> Controller, Controller->IO_Address);
> goto Failure;
> }
> diff --git a/drivers/block/cciss.c b/drivers/block/cciss.c
> index 0422c47..2758982 100644
> --- a/drivers/block/cciss.c
> +++ b/drivers/block/cciss.c
> @@ -3854,7 +3854,7 @@ static void print_cfg_table(ctlr_info_t *h)
> readl(&(tb->HostWrite.CoalIntDelay)));
> dev_dbg(&h->pdev->dev, " Coalesce Interrupt Count = 0x%x\n",
> readl(&(tb->HostWrite.CoalIntCount)));
> - dev_dbg(&h->pdev->dev, " Max outstanding commands = 0x%d\n",
> + dev_dbg(&h->pdev->dev, " Max outstanding commands = 0x%x\n",
> readl(&(tb->CmdsOutMax)));
> dev_dbg(&h->pdev->dev, " Bus Types = 0x%x\n",
> readl(&(tb->BusTypes)));
> —
why not use like this : dev_dbg(&h->pdev->dev, " Max outstanding commands = %#x\n” ?
%#x will add 0x prefix automatically .
Thanks
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Joe Perches <joe@perches.com> |
|---|---|
| Date | 2015-08-03 10:10 +0200 |
| Message-ID | <pTiYO-5CT-21@gated-at.bofh.it> |
| In reply to | #1198593 |
On Mon, 2015-08-03 at 15:25 +0800, yalin wang wrote: > > On Aug 3, 2015, at 04:25, Joe Perches <joe@perches.com> wrote: > > > > Correct misuse of 0x%d in logging messages. > > [] > why not use like this : dev_dbg(&h->pdev->dev, " Max outstanding > commands = %#x\n” ? > %#x will add 0x prefix automatically . It's generally a consistency thing. A 0 value would be emitted as 0 and not 0x0. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | yalin wang <yalin.wang2010@gmail.com> |
|---|---|
| Date | 2015-08-03 11:00 +0200 |
| Message-ID | <pTjLd-6xC-15@gated-at.bofh.it> |
| In reply to | #1198618 |
> On Aug 3, 2015, at 16:03, Joe Perches <joe@perches.com> wrote:
>
> On Mon, 2015-08-03 at 15:25 +0800, yalin wang wrote:
>>> On Aug 3, 2015, at 04:25, Joe Perches <joe@perches.com> wrote:
>>>
>>> Correct misuse of 0x%d in logging messages.
>>>
> []
>> why not use like this : dev_dbg(&h->pdev->dev, " Max outstanding
>> commands = %#x\n” ?
>> %#x will add 0x prefix automatically .
>
> It's generally a consistency thing.
> A 0 value would be emitted as 0 and not 0x0.
>
i try on my ubuntu ,
static int __init throtl_init(void)
{
printk("module init test: %#x %p\n", 0, (void *)0x123);
return 0;
}
module_init(throtl_init);
#uname -a
Linux ubuntu 3.16.0-38-generic #52~14.04.1-Ubuntu SMP Fri May 8 09:43:57 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
#dmesg
[259356.375586] module init test: 0x0 0000000000000123
it seems don’t need 0x%x for 0, just need %#x for all numbers.
there are lots of use like this, i can change them if needed:
# egrep -r -i '0x%\d*x' . | wc -l
11776
Thanks
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | yalin wang <yalin.wang2010@gmail.com> |
|---|---|
| Date | 2015-08-04 04:20 +0200 |
| Message-ID | <pTzZE-57V-17@gated-at.bofh.it> |
| In reply to | #1198646 |
Ping ?
> On Aug 3, 2015, at 16:56, yalin wang <yalin.wang2010@gmail.com> wrote:
>
>
>> On Aug 3, 2015, at 16:03, Joe Perches <joe@perches.com> wrote:
>>
>> On Mon, 2015-08-03 at 15:25 +0800, yalin wang wrote:
>>>> On Aug 3, 2015, at 04:25, Joe Perches <joe@perches.com> wrote:
>>>>
>>>> Correct misuse of 0x%d in logging messages.
>>>>
>> []
>>> why not use like this : dev_dbg(&h->pdev->dev, " Max outstanding
>>> commands = %#x\n” ?
>>> %#x will add 0x prefix automatically .
>>
>> It's generally a consistency thing.
>> A 0 value would be emitted as 0 and not 0x0.
>>
> i try on my ubuntu ,
>
> static int __init throtl_init(void)
> {
> printk("module init test: %#x %p\n", 0, (void *)0x123);
> return 0;
>
> }
>
> module_init(throtl_init);
>
> #uname -a
> Linux ubuntu 3.16.0-38-generic #52~14.04.1-Ubuntu SMP Fri May 8 09:43:57 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
>
> #dmesg
> [259356.375586] module init test: 0x0 0000000000000123
>
> it seems don’t need 0x%x for 0, just need %#x for all numbers.
> there are lots of use like this, i can change them if needed:
>
> # egrep -r -i '0x%\d*x' . | wc -l
> 11776
>
>
> Thanks
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | Joe Perches <joe@perches.com> |
|---|---|
| Date | 2015-08-04 04:30 +0200 |
| Message-ID | <pTA9j-5jk-7@gated-at.bofh.it> |
| In reply to | #1199377 |
On Tue, 2015-08-04 at 10:19 +0800, yalin wang wrote:
> Ping ?
> > On Aug 3, 2015, at 16:56, yalin wang <yalin.wang2010@gmail.com> wrote:
> >
> >
> >> On Aug 3, 2015, at 16:03, Joe Perches <joe@perches.com> wrote:
> >>
> >> On Mon, 2015-08-03 at 15:25 +0800, yalin wang wrote:
> >>>> On Aug 3, 2015, at 04:25, Joe Perches <joe@perches.com> wrote:
> >>>>
> >>>> Correct misuse of 0x%d in logging messages.
> >>>>
> >> []
> >>> why not use like this : dev_dbg(&h->pdev->dev, " Max outstanding
> >>> commands = %#x\n” ?
> >>> %#x will add 0x prefix automatically .
> >>
> >> It's generally a consistency thing.
> >> A 0 value would be emitted as 0 and not 0x0.
> >>
> > i try on my ubuntu ,
> >
> > static int __init throtl_init(void)
> > {
> > printk("module init test: %#x %p\n", 0, (void *)0x123);
> > return 0;
> >
> > }
> >
> > module_init(throtl_init);
> >
> > #uname -a
> > Linux ubuntu 3.16.0-38-generic #52~14.04.1-Ubuntu SMP Fri May 8 09:43:57 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
> >
> > #dmesg
> > [259356.375586] module init test: 0x0 0000000000000123
> >
> > it seems don’t need 0x%x for 0, just need %#x for all numbers.
> > there are lots of use like this, i can change them if needed:
> >
> > # egrep -r -i '0x%\d*x' . | wc -l
> > 11776
I suggest not, it's not a standard usage and the 0
may be unexpected.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [next] | [standalone]
| From | yalin wang <yalin.wang2010@gmail.com> |
|---|---|
| Date | 2015-08-04 06:10 +0200 |
| Message-ID | <pTBI6-7D3-3@gated-at.bofh.it> |
| In reply to | #1199387 |
> On Aug 4, 2015, at 10:27, Joe Perches <joe@perches.com> wrote:
>
> On Tue, 2015-08-04 at 10:19 +0800, yalin wang wrote:
>> Ping ?
>>> On Aug 3, 2015, at 16:56, yalin wang <yalin.wang2010@gmail.com> wrote:
>>>
>>>
>>>> On Aug 3, 2015, at 16:03, Joe Perches <joe@perches.com> wrote:
>>>>
>>>> On Mon, 2015-08-03 at 15:25 +0800, yalin wang wrote:
>>>>>> On Aug 3, 2015, at 04:25, Joe Perches <joe@perches.com> wrote:
>>>>>>
>>>>>> Correct misuse of 0x%d in logging messages.
>>>>>>
>>>> []
>>>>> why not use like this : dev_dbg(&h->pdev->dev, " Max outstanding
>>>>> commands = %#x\n” ?
>>>>> %#x will add 0x prefix automatically .
>>>>
>>>> It's generally a consistency thing.
>>>> A 0 value would be emitted as 0 and not 0x0.
>>>>
>>> i try on my ubuntu ,
>>>
>>> static int __init throtl_init(void)
>>> {
>>> printk("module init test: %#x %p\n", 0, (void *)0x123);
>>> return 0;
>>>
>>> }
>>>
>>> module_init(throtl_init);
>>>
>>> #uname -a
>>> Linux ubuntu 3.16.0-38-generic #52~14.04.1-Ubuntu SMP Fri May 8 09:43:57 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux
>>>
>>> #dmesg
>>> [259356.375586] module init test: 0x0 0000000000000123
>>>
>>> it seems don’t need 0x%x for 0, just need %#x for all numbers.
>>> there are lots of use like this, i can change them if needed:
>>>
>>> # egrep -r -i '0x%\d*x' . | wc -l
>>> 11776
>
> I suggest not, it's not a standard usage and the 0
> may be unexpected.
>
ok, printk for %x is really not compatible with glibc printf behavior,
another is %p, printk print %p as hex but not with 0x prefix, while
printf print %p as 0x…. with prefix, is this need change ?
so we don’t need lots of 0x%p printk .
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web