[PATCH] xen/arm: make the error log printk ratelimited

[PATCH] xen/arm: make the error log printk ratelimited

[Shannon Zhao]

From: Shannon Zhao <shannon.zhao@linaro.org>

To avoid a lot of possible error logs in function map_dev_mmio_region,
make it printk ratelimited.

Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org>
---
 xen/arch/arm/p2m.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/xen/arch/arm/p2m.c b/xen/arch/arm/p2m.c
index 0011708..be4ab9d 100644
--- a/xen/arch/arm/p2m.c
+++ b/xen/arch/arm/p2m.c
@@ -1284,8 +1284,9 @@ int map_dev_mmio_region(struct domain *d,
     res = map_mmio_regions(d, start_gfn, nr, mfn);
     if ( res < 0 )
     {
-        printk(XENLOG_ERR "Unable to map [%#lx - %#lx] in Dom%d\n",
-               start_gfn, start_gfn + nr - 1, d->domain_id);
+        if ( printk_ratelimit() )
+            printk(XENLOG_ERR "Unable to map [%#lx - %#lx] in Dom%d\n",
+                   start_gfn, start_gfn + nr - 1, d->domain_id);
         return res;
     }
 
-- 
2.0.4



_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH] xen/arm: make the error log printk ratelimited

[Jan Beulich]

>>> On 31.03.16 at 11:05, <zhaoshenglong@huawei.com> wrote:
> --- a/xen/arch/arm/p2m.c
> +++ b/xen/arch/arm/p2m.c
> @@ -1284,8 +1284,9 @@ int map_dev_mmio_region(struct domain *d,
>      res = map_mmio_regions(d, start_gfn, nr, mfn);
>      if ( res < 0 )
>      {
> -        printk(XENLOG_ERR "Unable to map [%#lx - %#lx] in Dom%d\n",
> -               start_gfn, start_gfn + nr - 1, d->domain_id);
> +        if ( printk_ratelimit() )
> +            printk(XENLOG_ERR "Unable to map [%#lx - %#lx] in Dom%d\n",
> +                   start_gfn, start_gfn + nr - 1, d->domain_id);
>          return res;
>      }

I think this would better simply use XENLOG_G_ERR.

Jan


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Re: [PATCH] xen/arm: make the error log printk ratelimited

[Shannon Zhao]

On 2016/3/31 17:12, Jan Beulich wrote:
>>>> On 31.03.16 at 11:05, <zhaoshenglong@huawei.com> wrote:
>> > --- a/xen/arch/arm/p2m.c
>> > +++ b/xen/arch/arm/p2m.c
>> > @@ -1284,8 +1284,9 @@ int map_dev_mmio_region(struct domain *d,
>> >      res = map_mmio_regions(d, start_gfn, nr, mfn);
>> >      if ( res < 0 )
>> >      {
>> > -        printk(XENLOG_ERR "Unable to map [%#lx - %#lx] in Dom%d\n",
>> > -               start_gfn, start_gfn + nr - 1, d->domain_id);
>> > +        if ( printk_ratelimit() )
>> > +            printk(XENLOG_ERR "Unable to map [%#lx - %#lx] in Dom%d\n",
>> > +                   start_gfn, start_gfn + nr - 1, d->domain_id);
>> >          return res;
>> >      }
> I think this would better simply use XENLOG_G_ERR.
Ok, will update.

Thanks,
-- 
Shannon


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel