Re: [PATCH v2 1/2] iommu/vt-d: Ratelimit fault handler
From: Joe Perches
Date: Thu Mar 17 2016 - 16:33:39 EST
On Thu, 2016-03-17 at 14:12 -0600, Alex Williamson wrote:
> Fault rates can easily overwhelm the console and make the system
> unresponsive. Ratelimit to allow an opportunity for maintenance.
[]
> diff --git a/drivers/iommu/dmar.c b/drivers/iommu/dmar.c
[]
> @@ -1602,10 +1602,17 @@ irqreturn_t dmar_fault(int irq, void *dev_id)
> int reg, fault_index;
> u32 fault_status;
> unsigned long flag;
> + bool ratelimited;
> + static DEFINE_RATELIMIT_STATE(rs,
> + DEFAULT_RATELIMIT_INTERVAL,
> + DEFAULT_RATELIMIT_BURST);
Are these the appropriate limits for dmar?
include/linux/ratelimit.h:#define DEFAULT_RATELIMIT_INTERVAL (5 * HZ)
include/linux/ratelimit.h:#define DEFAULT_RATELIMIT_BURST 10