Re: [PATCHv4 2/2] iommu/vt-d: Limit number of faults to clear in irq handler

Hi, On 05/03/2018 10:34 AM, Dmitry Safonov wrote: > On Thu, 2018-05-03 at 10:16 +0800, Lu Baolu wrote: >> Hi, >> >> On 05/03/2018 09:59 AM, Dmitry Safonov wrote: >>> On Thu, 2018-05-03 at 09:32 +0800, Lu Baolu wrote: Hi, On 05/03/2018 08:52 AM, Dmitry Safonov wrote: > AFAICS, we

Re: [PATCHv4 2/2] iommu/vt-d: Limit number of faults to clear in irq handler

On Thu, 2018-05-03 at 10:16 +0800, Lu Baolu wrote: > Hi, > > On 05/03/2018 09:59 AM, Dmitry Safonov wrote: > > On Thu, 2018-05-03 at 09:32 +0800, Lu Baolu wrote: > > > Hi, > > > > > > On 05/03/2018 08:52 AM, Dmitry Safonov wrote: > > > > AFAICS, we're doing fault-clearing in a loop inside irq > >

Re: [PATCHv4 2/2] iommu/vt-d: Limit number of faults to clear in irq handler

Hi, On 05/03/2018 10:16 AM, Lu Baolu wrote: > Hi, > > On 05/03/2018 09:59 AM, Dmitry Safonov wrote: >> On Thu, 2018-05-03 at 09:32 +0800, Lu Baolu wrote: >>> Hi, >>> >>> On 05/03/2018 08:52 AM, Dmitry Safonov wrote: AFAICS, we're doing fault-clearing in a loop inside irq handler. That me

Re: [PATCHv4 2/2] iommu/vt-d: Limit number of faults to clear in irq handler

Hi, On 05/03/2018 09:59 AM, Dmitry Safonov wrote: > On Thu, 2018-05-03 at 09:32 +0800, Lu Baolu wrote: >> Hi, >> >> On 05/03/2018 08:52 AM, Dmitry Safonov wrote: >>> AFAICS, we're doing fault-clearing in a loop inside irq handler. >>> That means that while we're clearing if a fault raises, it'll m

Re: [PATCHv4 2/2] iommu/vt-d: Limit number of faults to clear in irq handler

On Thu, 2018-05-03 at 09:32 +0800, Lu Baolu wrote: > Hi, > > On 05/03/2018 08:52 AM, Dmitry Safonov wrote: > > AFAICS, we're doing fault-clearing in a loop inside irq handler. > > That means that while we're clearing if a fault raises, it'll make > > an irq level triggered (or on edge) on lapic. S

Re: [PATCHv4 2/2] iommu/vt-d: Limit number of faults to clear in irq handler

Hi, On 05/03/2018 08:52 AM, Dmitry Safonov wrote: > On Thu, 2018-05-03 at 07:49 +0800, Lu Baolu wrote: >> Hi, >> >> On 05/02/2018 08:38 PM, Dmitry Safonov wrote: >>> Hi Lu, >>> >>> On Wed, 2018-05-02 at 14:34 +0800, Lu Baolu wrote: Hi, On 03/31/2018 08:33 AM, Dmitry Safonov wrote: >

Re: [PATCHv4 2/2] iommu/vt-d: Limit number of faults to clear in irq handler

On Thu, 2018-05-03 at 07:49 +0800, Lu Baolu wrote: > Hi, > > On 05/02/2018 08:38 PM, Dmitry Safonov wrote: > > Hi Lu, > > > > On Wed, 2018-05-02 at 14:34 +0800, Lu Baolu wrote: > > > Hi, > > > > > > On 03/31/2018 08:33 AM, Dmitry Safonov wrote: > > > > Theoretically, on some machines faults migh

Re: [PATCHv4 2/2] iommu/vt-d: Limit number of faults to clear in irq handler

Hi, On 05/02/2018 08:38 PM, Dmitry Safonov wrote: > Hi Lu, > > On Wed, 2018-05-02 at 14:34 +0800, Lu Baolu wrote: >> Hi, >> >> On 03/31/2018 08:33 AM, Dmitry Safonov wrote: >>> Theoretically, on some machines faults might be generated faster >>> than >>> they're cleared by CPU. >> Is this a real c

Re: [PATCHv4 2/2] iommu/vt-d: Limit number of faults to clear in irq handler

Hi Lu, On Wed, 2018-05-02 at 14:34 +0800, Lu Baolu wrote: > Hi, > > On 03/31/2018 08:33 AM, Dmitry Safonov wrote: > > Theoretically, on some machines faults might be generated faster > > than > > they're cleared by CPU. > > Is this a real case? No. 1/2 is a real case and this one was discussed

Re: [PATCHv4 2/2] iommu/vt-d: Limit number of faults to clear in irq handler

Hi, On 03/31/2018 08:33 AM, Dmitry Safonov wrote: > Theoretically, on some machines faults might be generated faster than > they're cleared by CPU. Is this a real case? > Let's limit the cleaning-loop by number of hw > fault registers. Will this cause the fault recording registers full of faul