All of lore.kernel.org
 help / color / mirror / Atom feed
From: Keqian Zhu <zhukeqian1@huawei.com>
To: Auger Eric <eric.auger@redhat.com>, <eric.auger.pro@gmail.com>,
	<iommu@lists.linux-foundation.org>,
	<linux-kernel@vger.kernel.org>, <kvm@vger.kernel.org>,
	<kvmarm@lists.cs.columbia.edu>, <will@kernel.org>,
	<joro@8bytes.org>, <maz@kernel.org>, <robin.murphy@arm.com>,
	<alex.williamson@redhat.com>
Cc: jean-philippe@linaro.org, jacob.jun.pan@linux.intel.com,
	nicoleotsuka@gmail.com, vivek.gautam@arm.com, yi.l.liu@intel.com,
	zhangfei.gao@linaro.org
Subject: Re: [PATCH v13 03/15] iommu/arm-smmu-v3: Maintain a SID->device structure
Date: Tue, 2 Feb 2021 15:20:31 +0800	[thread overview]
Message-ID: <04fb111d-304e-2707-e5bb-e77b5ae93ed5@huawei.com> (raw)
In-Reply-To: <c457b450-8755-308e-7c7a-abe23b33d0d6@redhat.com>

Hi Eric,

On 2021/2/2 1:19, Auger Eric wrote:
> Hi Keqian,
> 
> On 2/1/21 1:26 PM, Keqian Zhu wrote:
>> Hi Eric,
>>
>> On 2020/11/18 19:21, Eric Auger wrote:
>>> From: Jean-Philippe Brucker <jean-philippe@linaro.org>
>>>
>>> When handling faults from the event or PRI queue, we need to find the
>>> struct device associated to a SID. Add a rb_tree to keep track of SIDs.
>>>
>>> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
>> [...]
>>
>>>  }
>>>  
>>> +static int arm_smmu_insert_master(struct arm_smmu_device *smmu,
>>> +				  struct arm_smmu_master *master)
[...]

>>>  	kfree(master);
>>
>> Thanks,
>> Keqian
>>
> Thank you for the review. Jean will address this issues in his own
> series and on my end I will rebase on this latter.
> 
> Best Regards
> 
> Eric
>

Yeah, and hope this series can be accepted earlier ;-)

Thanks,
Keqian
_______________________________________________
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm

WARNING: multiple messages have this Message-ID (diff)
From: Keqian Zhu <zhukeqian1@huawei.com>
To: Auger Eric <eric.auger@redhat.com>, <eric.auger.pro@gmail.com>,
	<iommu@lists.linux-foundation.org>,
	<linux-kernel@vger.kernel.org>, <kvm@vger.kernel.org>,
	<kvmarm@lists.cs.columbia.edu>, <will@kernel.org>,
	<joro@8bytes.org>, <maz@kernel.org>, <robin.murphy@arm.com>,
	<alex.williamson@redhat.com>
Cc: jean-philippe@linaro.org, vivek.gautam@arm.com, zhangfei.gao@linaro.org
Subject: Re: [PATCH v13 03/15] iommu/arm-smmu-v3: Maintain a SID->device structure
Date: Tue, 2 Feb 2021 15:20:31 +0800	[thread overview]
Message-ID: <04fb111d-304e-2707-e5bb-e77b5ae93ed5@huawei.com> (raw)
In-Reply-To: <c457b450-8755-308e-7c7a-abe23b33d0d6@redhat.com>

Hi Eric,

On 2021/2/2 1:19, Auger Eric wrote:
> Hi Keqian,
> 
> On 2/1/21 1:26 PM, Keqian Zhu wrote:
>> Hi Eric,
>>
>> On 2020/11/18 19:21, Eric Auger wrote:
>>> From: Jean-Philippe Brucker <jean-philippe@linaro.org>
>>>
>>> When handling faults from the event or PRI queue, we need to find the
>>> struct device associated to a SID. Add a rb_tree to keep track of SIDs.
>>>
>>> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
>> [...]
>>
>>>  }
>>>  
>>> +static int arm_smmu_insert_master(struct arm_smmu_device *smmu,
>>> +				  struct arm_smmu_master *master)
[...]

>>>  	kfree(master);
>>
>> Thanks,
>> Keqian
>>
> Thank you for the review. Jean will address this issues in his own
> series and on my end I will rebase on this latter.
> 
> Best Regards
> 
> Eric
>

Yeah, and hope this series can be accepted earlier ;-)

Thanks,
Keqian
_______________________________________________
iommu mailing list
iommu@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/iommu

WARNING: multiple messages have this Message-ID (diff)
From: Keqian Zhu <zhukeqian1@huawei.com>
To: Auger Eric <eric.auger@redhat.com>, <eric.auger.pro@gmail.com>,
	<iommu@lists.linux-foundation.org>,
	<linux-kernel@vger.kernel.org>, <kvm@vger.kernel.org>,
	<kvmarm@lists.cs.columbia.edu>, <will@kernel.org>,
	<joro@8bytes.org>, <maz@kernel.org>, <robin.murphy@arm.com>,
	<alex.williamson@redhat.com>
Cc: <jean-philippe@linaro.org>, <jacob.jun.pan@linux.intel.com>,
	<nicoleotsuka@gmail.com>, <vivek.gautam@arm.com>,
	<yi.l.liu@intel.com>, <zhangfei.gao@linaro.org>
Subject: Re: [PATCH v13 03/15] iommu/arm-smmu-v3: Maintain a SID->device structure
Date: Tue, 2 Feb 2021 15:20:31 +0800	[thread overview]
Message-ID: <04fb111d-304e-2707-e5bb-e77b5ae93ed5@huawei.com> (raw)
In-Reply-To: <c457b450-8755-308e-7c7a-abe23b33d0d6@redhat.com>

Hi Eric,

On 2021/2/2 1:19, Auger Eric wrote:
> Hi Keqian,
> 
> On 2/1/21 1:26 PM, Keqian Zhu wrote:
>> Hi Eric,
>>
>> On 2020/11/18 19:21, Eric Auger wrote:
>>> From: Jean-Philippe Brucker <jean-philippe@linaro.org>
>>>
>>> When handling faults from the event or PRI queue, we need to find the
>>> struct device associated to a SID. Add a rb_tree to keep track of SIDs.
>>>
>>> Signed-off-by: Jean-Philippe Brucker <jean-philippe@linaro.org>
>> [...]
>>
>>>  }
>>>  
>>> +static int arm_smmu_insert_master(struct arm_smmu_device *smmu,
>>> +				  struct arm_smmu_master *master)
[...]

>>>  	kfree(master);
>>
>> Thanks,
>> Keqian
>>
> Thank you for the review. Jean will address this issues in his own
> series and on my end I will rebase on this latter.
> 
> Best Regards
> 
> Eric
>

Yeah, and hope this series can be accepted earlier ;-)

Thanks,
Keqian

  reply	other threads:[~2021-02-02  7:20 UTC|newest]

Thread overview: 175+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-18 11:21 [PATCH v13 00/15] SMMUv3 Nested Stage Setup (IOMMU part) Eric Auger
2020-11-18 11:21 ` Eric Auger
2020-11-18 11:21 ` Eric Auger
2020-11-18 11:21 ` [PATCH v13 01/15] iommu: Introduce attach/detach_pasid_table API Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 16:19   ` Jacob Pan
2020-11-18 16:19     ` Jacob Pan
2020-11-18 16:19     ` Jacob Pan
2020-11-19 17:02     ` Auger Eric
2020-11-19 17:02       ` Auger Eric
2020-11-19 17:02       ` Auger Eric
2021-02-01 11:27   ` Keqian Zhu
2021-02-01 11:27     ` Keqian Zhu
2021-02-01 11:27     ` Keqian Zhu
2021-02-01 17:18     ` Auger Eric
2021-02-01 17:18       ` Auger Eric
2021-02-01 17:18       ` Auger Eric
2020-11-18 11:21 ` [PATCH v13 02/15] iommu: Introduce bind/unbind_guest_msi Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2021-02-01 11:52   ` Keqian Zhu
2021-02-01 11:52     ` Keqian Zhu
2021-02-01 11:52     ` Keqian Zhu
2021-02-12  8:55     ` Auger Eric
2021-02-12  8:55       ` Auger Eric
2021-02-12  8:55       ` Auger Eric
2021-02-18  8:43       ` Keqian Zhu
2021-02-18  8:43         ` Keqian Zhu
2021-02-18  8:43         ` Keqian Zhu
2021-02-18 10:35         ` Auger Eric
2021-02-18 10:35           ` Auger Eric
2021-02-18 10:35           ` Auger Eric
2020-11-18 11:21 ` [PATCH v13 03/15] iommu/arm-smmu-v3: Maintain a SID->device structure Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2021-02-01 12:26   ` Keqian Zhu
2021-02-01 12:26     ` Keqian Zhu
2021-02-01 12:26     ` Keqian Zhu
2021-02-01 15:15     ` Jean-Philippe Brucker
2021-02-01 15:15       ` Jean-Philippe Brucker
2021-02-01 15:15       ` Jean-Philippe Brucker
2021-02-02  6:39       ` Keqian Zhu
2021-02-02  6:39         ` Keqian Zhu
2021-02-02  6:39         ` Keqian Zhu
2021-02-01 17:19     ` Auger Eric
2021-02-01 17:19       ` Auger Eric
2021-02-01 17:19       ` Auger Eric
2021-02-02  7:20       ` Keqian Zhu [this message]
2021-02-02  7:20         ` Keqian Zhu
2021-02-02  7:20         ` Keqian Zhu
2020-11-18 11:21 ` [PATCH v13 04/15] iommu/smmuv3: Allow s1 and s2 configs to coexist Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2021-02-01 12:35   ` Keqian Zhu
2021-02-01 12:35     ` Keqian Zhu
2021-02-01 12:35     ` Keqian Zhu
2020-11-18 11:21 ` [PATCH v13 05/15] iommu/smmuv3: Get prepared for nested stage support Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-19  3:59   ` kernel test robot
2020-11-19  3:59     ` kernel test robot
2020-11-19  3:59     ` kernel test robot
2020-11-19  3:59     ` kernel test robot
2020-12-03 12:32   ` Kunkun Jiang
2020-12-03 12:32     ` Kunkun Jiang
2020-12-03 13:01     ` Auger Eric
2020-12-03 13:01       ` Auger Eric
2020-12-03 13:01       ` Auger Eric
2020-12-03 13:23       ` Kunkun Jiang
2020-12-03 13:23         ` Kunkun Jiang
2020-12-03 13:23         ` Kunkun Jiang
2020-12-09 14:26   ` Shameerali Kolothum Thodi
2020-12-09 14:26     ` Shameerali Kolothum Thodi
2020-12-09 14:26     ` Shameerali Kolothum Thodi
2021-02-02  7:14   ` Keqian Zhu
2021-02-02  7:14     ` Keqian Zhu
2021-02-02  7:14     ` Keqian Zhu
2021-02-11 17:36     ` Auger Eric
2021-02-11 17:36       ` Auger Eric
2021-02-11 17:36       ` Auger Eric
2020-11-18 11:21 ` [PATCH v13 06/15] iommu/smmuv3: Implement attach/detach_pasid_table Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2021-02-02  8:03   ` Keqian Zhu
2021-02-02  8:03     ` Keqian Zhu
2021-02-02  8:03     ` Keqian Zhu
2021-02-11 17:35     ` Auger Eric
2021-02-11 17:35       ` Auger Eric
2021-02-11 17:35       ` Auger Eric
2020-11-18 11:21 ` [PATCH v13 07/15] iommu/smmuv3: Allow stage 1 invalidation with unmanaged ASIDs Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-12-01 13:33   ` Xingang Wang
2020-12-01 13:33     ` Xingang Wang
2020-12-01 13:33     ` Xingang Wang
2020-12-01 13:58     ` Auger Eric
2020-12-01 13:58       ` Auger Eric
2020-12-01 13:58       ` Auger Eric
2020-12-02 12:59       ` Wang Xingang
2020-12-02 12:59         ` Wang Xingang
2020-12-02 12:59         ` Wang Xingang
2020-12-03 18:42       ` Shameerali Kolothum Thodi
2020-12-03 18:42         ` Shameerali Kolothum Thodi
2020-12-03 18:42         ` Shameerali Kolothum Thodi
2020-12-04  9:53         ` Jean-Philippe Brucker
2020-12-04  9:53           ` Jean-Philippe Brucker
2020-12-04  9:53           ` Jean-Philippe Brucker
2020-12-04 10:20           ` Shameerali Kolothum Thodi
2020-12-04 10:20             ` Shameerali Kolothum Thodi
2020-12-04 10:20             ` Shameerali Kolothum Thodi
2020-12-04 10:23             ` Auger Eric
2020-12-04 10:23               ` Auger Eric
2020-12-04 10:23               ` Auger Eric
2021-01-14 16:58               ` Auger Eric
2021-01-14 16:58                 ` Auger Eric
2021-01-14 16:58                 ` Auger Eric
2021-01-14 17:09                 ` Shameerali Kolothum Thodi
2021-01-14 17:09                   ` Shameerali Kolothum Thodi
2021-01-14 17:09                   ` Shameerali Kolothum Thodi
2021-01-14 17:33                 ` Jean-Philippe Brucker
2021-01-14 17:33                   ` Jean-Philippe Brucker
2021-01-14 17:33                   ` Jean-Philippe Brucker
2021-01-14 18:00                   ` Auger Eric
2021-01-14 18:00                     ` Auger Eric
2021-01-14 18:00                     ` Auger Eric
2021-02-15 13:17         ` Auger Eric
2021-02-15 13:17           ` Auger Eric
2021-02-15 13:17           ` Auger Eric
2020-11-18 11:21 ` [PATCH v13 08/15] iommu/smmuv3: Implement cache_invalidate Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2021-01-16  2:24   ` chenxiang (M)
2020-11-18 11:21 ` [PATCH v13 09/15] dma-iommu: Implement NESTED_MSI cookie Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21 ` [PATCH v13 10/15] iommu/smmuv3: Nested mode single MSI doorbell per domain enforcement Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21 ` [PATCH v13 11/15] iommu/smmuv3: Enforce incompatibility between nested mode and HW MSI regions Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21 ` [PATCH v13 12/15] iommu/smmuv3: Implement bind/unbind_guest_msi Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21 ` [PATCH v13 13/15] iommu/smmuv3: Report non recoverable faults Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21 ` [PATCH v13 14/15] iommu/smmuv3: Accept configs with more than one context descriptor Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21 ` [PATCH v13 15/15] iommu/smmuv3: Add PASID cache invalidation per PASID Eric Auger
2020-11-18 11:21   ` Eric Auger
2020-11-18 11:21   ` Eric Auger
2021-01-08 17:05 ` [PATCH v13 00/15] SMMUv3 Nested Stage Setup (IOMMU part) Shameerali Kolothum Thodi
2021-01-08 17:05   ` Shameerali Kolothum Thodi
2021-01-08 17:05   ` Shameerali Kolothum Thodi
2021-01-13 15:37   ` Auger Eric
2021-01-13 15:37     ` Auger Eric
2021-01-13 15:37     ` Auger Eric
2021-02-21 18:21   ` Auger Eric
2021-02-21 18:21     ` Auger Eric
2021-02-21 18:21     ` Auger Eric
2021-02-22  8:56     ` Shameerali Kolothum Thodi
2021-02-22  8:56       ` Shameerali Kolothum Thodi
2021-02-22  8:56       ` Shameerali Kolothum Thodi
2021-03-15 18:04 ` Krishna Reddy
2021-03-15 18:04   ` Krishna Reddy
2021-03-15 18:04   ` Krishna Reddy
2021-03-16  8:22   ` Auger Eric
2021-03-16  8:22     ` Auger Eric
2021-03-16  8:22     ` Auger Eric
2021-03-16 18:10     ` Krishna Reddy
2021-03-16 18:10       ` Krishna Reddy
2021-03-16 18:10       ` Krishna Reddy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=04fb111d-304e-2707-e5bb-e77b5ae93ed5@huawei.com \
    --to=zhukeqian1@huawei.com \
    --cc=alex.williamson@redhat.com \
    --cc=eric.auger.pro@gmail.com \
    --cc=eric.auger@redhat.com \
    --cc=iommu@lists.linux-foundation.org \
    --cc=jacob.jun.pan@linux.intel.com \
    --cc=jean-philippe@linaro.org \
    --cc=joro@8bytes.org \
    --cc=kvm@vger.kernel.org \
    --cc=kvmarm@lists.cs.columbia.edu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=maz@kernel.org \
    --cc=nicoleotsuka@gmail.com \
    --cc=robin.murphy@arm.com \
    --cc=vivek.gautam@arm.com \
    --cc=will@kernel.org \
    --cc=yi.l.liu@intel.com \
    --cc=zhangfei.gao@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.