On Thu, Dec 06, 2018 at 11:55:02AM +0800, Nicolas Boichat wrote:
>On Thu, Dec 6, 2018 at 11:32 AM Wei Yang wrote:
>>
>> On Thu, Dec 06, 2018 at 08:41:36AM +0800, Nicolas Boichat wrote:
>> >On Wed, Dec 5, 2018 at 8:18 PM Wei Yang wrote:
>> >>
>> &
On Thu, Dec 06, 2018 at 08:41:36AM +0800, Nicolas Boichat wrote:
>On Wed, Dec 5, 2018 at 8:18 PM Wei Yang wrote:
>>
>> On Wed, Dec 05, 2018 at 03:39:51PM +0800, Nicolas Boichat wrote:
>> >On Wed, Dec 5, 2018 at 3:25 PM Wei Yang wrote:
>> >>
>> >> O
On Wed, Dec 05, 2018 at 03:39:51PM +0800, Nicolas Boichat wrote:
>On Wed, Dec 5, 2018 at 3:25 PM Wei Yang wrote:
>>
>> On Wed, Dec 05, 2018 at 01:48:27PM +0800, Nicolas Boichat wrote:
>> >In some cases (e.g. IOMMU ARMv7s page allocator), we need to allocate
>> >da
On Wed, Dec 05, 2018 at 03:39:51PM +0800, Nicolas Boichat wrote:
>On Wed, Dec 5, 2018 at 3:25 PM Wei Yang wrote:
>>
>> On Wed, Dec 05, 2018 at 01:48:27PM +0800, Nicolas Boichat wrote:
>> >In some cases (e.g. IOMMU ARMv7s page allocator), we need to allocate
>> >da
buf)
> {
> return sprintf(buf, "%u\n", s->usersize);
>@@ -5435,6 +5446,9 @@ static struct attribute *slab_attrs[] = {
> #ifdef CONFIG_ZONE_DMA
> &cache_dma_attr.attr,
> #endif
>+#ifdef CONFIG_ZONE_DMA32
>+ &cache_dma32_attr.attr,
>+#endif
>
Ping~ Hope you like it :-)
On Thu, Aug 18, 2016 at 11:00:26PM +, Wei Yang wrote:
>Hi, Alex & Bjorn,
>
>I tried to measure the effect of the improvement and looks good.
>
>+---+---+---+---+
>|Platform Info |Before(ns) |A
c %lu nsec\n",
+ __func__, ts2.tv_sec - ts1.tv_sec, ts2.tv_nsec -
ts1.tv_nsec);
if (group)
return group;
On Fri, Aug 05, 2016 at 02:03:17PM +, Wei Yang wrote:
>On Thu, Jul 28, 2016 at 09:01:32AM -0600, Alex Williamson wrote:
>
On Thu, Jul 28, 2016 at 09:01:32AM -0600, Alex Williamson wrote:
>On Thu, 28 Jul 2016 14:52:59 +0000
>Wei Yang wrote:
>
>> According to the comments and code, get_pci_function_alias_group() and
>> get_pci_alias_group() do the search on the same pci bus. This means we can
&
Add a function to just walk on the local bus.
Signed-off-by: Wei Yang
---
drivers/pci/bus.c | 26 +-
include/linux/pci.h | 2 ++
2 files changed, 23 insertions(+), 5 deletions(-)
diff --git a/drivers/pci/bus.c b/drivers/pci/bus.c
index dd7cdbe..ccf4290 100644
--- a
comment.
Signed-off-by: Wei Yang
---
drivers/iommu/iommu.c | 100 +++---
1 file changed, 63 insertions(+), 37 deletions(-)
diff --git a/drivers/iommu/iommu.c b/drivers/iommu/iommu.c
index 351..2fae2c6 100644
--- a/drivers/iommu/iommu.c
+++ b/drivers
comment.
Signed-off-by: Wei Yang
---
drivers/iommu/iommu.c | 22 --
1 file changed, 8 insertions(+), 14 deletions(-)
diff --git a/drivers/iommu/iommu.c b/drivers/iommu/iommu.c
index 351..0338a81 100644
--- a/drivers/iommu/iommu.c
+++ b/drivers/iommu/iommu.c
@@ -643,17 +643,15
In 'commit <55d940430ab9> ("iommu/vt-d: Get rid of domain->iommu_lock")',
the error handling path is changed a little, which makes the function
always return 0.
This path fixes this.
Signed-off-by: Wei Yang
---
drivers/iommu/intel-iommu.c |2 +-
1 file change
On Wed, Jun 15, 2016 at 01:39:21PM +0200, Joerg Roedel wrote:
>On Tue, May 24, 2016 at 11:06:55PM +0000, Wei Yang wrote:
>> Hi, Joerg
>>
>> Not sure whether you think this calculation is correct.
>>
>> If I missed something for this " + 1" in your for
In the origin commit, the size of the second level is hard coded to an
integer, 256.
This patch defines two marco for the second level size and shift.
Signed-off-by: Wei Yang
---
drivers/iommu/intel-iommu.c | 12 ++--
include/linux/intel-iommu.h | 2 ++
2 files changed, 8 insertions
* adds patch 2 for comment
Wei Yang (2):
iommu/vt-d: reduce extra first level entry in iommu->domains
iommu/vt-d: use marco for dmar_domain's second level size and shift
drivers/iommu/intel-iommu.c | 12 ++--
include/linux/intel-iommu.h | 2 ++
2 files changed, 8 insertio
current
implementation.
This patch refines this calculation to reduce the extra first level entry
when ndomains is exact a multiple of 256.
Signed-off-by: Wei Yang
---
drivers/iommu/intel-iommu.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/iommu/intel-iommu.c
On Thu, May 26, 2016 at 11:11:51AM +0100, Robin Murphy wrote:
>On 25/05/16 22:43, Wei Yang wrote:
>>On Wed, May 25, 2016 at 11:17:49AM +0100, Robin Murphy wrote:
>>>On 25/05/16 00:06, Wei Yang wrote:
>>>>Hi, Joerg
>>>>
>>>>Not sure whethe
On Wed, May 25, 2016 at 11:17:49AM +0100, Robin Murphy wrote:
>On 25/05/16 00:06, Wei Yang wrote:
>>Hi, Joerg
>>
>>Not sure whether you think this calculation is correct.
>>
>>If I missed something for this " + 1" in your formula, I am glad to hea
Hi, Joerg
Not sure whether you think this calculation is correct.
If I missed something for this " + 1" in your formula, I am glad to hear your
explanation. So that I could learn something from you :-)
Have a good day~
On Sat, May 21, 2016 at 02:41:51AM +, Wei Yang wrote:
current
implementation.
This patch refines this calculation to reduce the extra first level entry.
Signed-off-by: Wei Yang
---
drivers/iommu/intel-iommu.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/iommu/intel-iommu.c b/drivers/iommu/intel-iommu.c
index e
On Mon, May 09, 2016 at 01:24:02PM +0200, Joerg Roedel wrote:
>On Sun, May 08, 2016 at 01:22:53PM +0000, Wei Yang wrote:
>> >Wei Yang (4):
>> > iommu/vt-d: replace *hdr with {drhd/atsr}[0] in struct
>> >dmar_{drhd/atsr}_unit
>> > iommu/vt-d: us
Ping~
On Thu, Apr 14, 2016 at 02:55:06PM +, Wei Yang wrote:
>These four patches try to refine the Intel IO
dmar_acpi_dev_scope_init() iterates on the remapping structure and just do
proper job for ANDD structure. This is the what dmar_walk_dmar_table()
does.
This patch improves the code with dmar_walk_dmar_table().
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c | 56
(), so
that it notices this is an invalid dmar and avoids related setup jobs.
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c | 34 +-
1 file changed, 17 insertions(+), 17 deletions(-)
diff --git a/drivers/iommu/dmar.c b/drivers/iommu/dmar.c
index 6bff602
audience friendly.
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c| 22 ++
drivers/iommu/intel-iommu.c |8
include/acpi/actbl2.h | 33 +++--
3 files changed, 33 insertions(+), 30 deletions(-)
diff --git a/drivers/iommu
n dmar_table_detect()
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c| 17 +
drivers/iommu/intel-iommu.c | 18 +-
drivers/iommu/intel_irq_remapping.c | 10 --
include/linux/dmar.h|3 ++-
4 files changed, 2
:
* add similar change for struct dmar_atsr_unit in patch 1
* add similar change for rmrr and atsr in patch 2
V3:
* change hdr to drhd from type acpi_dmar_header to acpi_dmar_hardware_unit
* add reason in changelog for the change in Patch 1
V2:
* add patch 3 and 4
Wei Yang (4):
iommu/vt
Hi, Alex
I think this one is correct. It applies on v4.5 and works fine.
Or I missed something else?
On Wed, Mar 23, 2016 at 10:25:10PM +, Wei Yang wrote:
>According to the code path, iommu_callback_data is passed in
>iommu_bus_init() and just used in {add/remove}_iommu_group, by wh
On Wed, Mar 23, 2016 at 05:04:15PM -0600, Alex Williamson wrote:
>On Wed, 23 Mar 2016 22:25:11 +0000
>Wei Yang wrote:
>
>> The original code forgets to remove the sysfs_link to a device in
>> iommu_group/devices directory, when the creation fails or conflicts on the
>&
Obsolete this one, V2 is sent.
On Sun, Mar 20, 2016 at 01:57:52AM +, Wei Yang wrote:
>According to the code path, iommu_callback_data is passed in
>iommu_bus_init() and just used in {add/remove}_iommu_group, by when the
>bus->iommu_ops is already set up properly.
>
>This
This two patches tries to do some cleanup in iommu.
V2:
* add patch 2
Wei Yang (2):
iommu: remove the iommu_callback_data
iommu: remove sysfs_link to device in iommu_group/devices when failed
drivers/iommu/iommu.c | 22 +++---
1 file changed, 7 insertions(+), 15
The original code forgets to remove the sysfs_link to a device in
iommu_group/devices directory, when the creation fails or conflicts on the
name.
This patch tries to remove the sysfs_link on the failure.
Signed-off-by: Wei Yang
---
drivers/iommu/iommu.c |1 +
1 file changed, 1 insertion
According to the code path, iommu_callback_data is passed in
iommu_bus_init() and just used in {add/remove}_iommu_group, by when the
bus->iommu_ops is already set up properly.
This patch removes the iommu_callback_data by retrieving iommu_ops from
bus->iommu_ops directly.
Signed-off-b
(), so
that it notices this is an invalid dmar and avoids related setup jobs.
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c | 34 +-
1 file changed, 17 insertions(+), 17 deletions(-)
diff --git a/drivers/iommu/dmar.c b/drivers/iommu/dmar.c
index 6bff602
dmar_acpi_dev_scope_init() iterates on the remapping structure and just do
proper job for ANDD structure. This is the what dmar_walk_dmar_table()
does.
This patch improves the code with dmar_walk_dmar_table().
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c | 56
:
* change hdr to drhd from type acpi_dmar_header to acpi_dmar_hardware_unit
* add reason in changelog for the change in Patch 1
V2:
* add patch 3 and 4
Wei Yang (4):
iommu/vt-d: replace *hdr with drhd[0] in struct dmar_drhd_unit
iommu/vt-d: use zero-sized array in DMAR related ACPI
-sized array is
usually used for this purpose.
This patch adds zero-sized array for variable elements in DMAR ACPI
structures, which tries to make the code more audience friendly.
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c | 22 ++
include/acpi/actbl2.h | 31
t
reflects the real data type contained in dmar_drhd_unit and avoid some type
cast between them.
Besides this, this patch includes another change:
* remove redundant type cast to the same type in dmar_table_detect()
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c| 17 +--
On Mon, Mar 21, 2016 at 05:53:15PM +0100, Thomas Gleixner wrote:
>On Mon, 21 Mar 2016, Wei Yang wrote:
>> On Sun, Mar 20, 2016 at 04:42:29PM +0100, Thomas Gleixner wrote:
>> >On Sun, 20 Mar 2016, Wei Yang wrote:
>> >
>> >> hdr in struct dmar_drhd_unit
Hi, Thomas
Glad to receive your comment. Please see my reply below.
On Sun, Mar 20, 2016 at 04:42:29PM +0100, Thomas Gleixner wrote:
>On Sun, 20 Mar 2016, Wei Yang wrote:
>
>> hdr in struct dmar_drhd_unit is used to point the DMAR hardware unit copied
>> at the end of struct d
Obsolete this one. V2 is sent.
On Sat, Mar 12, 2016 at 09:57:38PM +, Wei Yang wrote:
>hdr in struct dmar_drhd_unit is used to point the DMAR hardware unit copied
>at the end of struct dmar_drhd_unit. One zero-sized array may be more
>elegant for this purpose.
>
>This patch re
-sized array is
usually used for this purpose.
This patch adds zero-sized array for variable elements in DMAR ACPI
structures, which tries to make the code more audience friendly.
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c | 22 ++
include/acpi/actbl2.h | 31
dmar_acpi_dev_scope_init() iterates on the remapping structure and just do
proper job for ANDD structure. This is the what dmar_walk_dmar_table()
does.
This patch improves the code with dmar_walk_dmar_table().
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c | 56
unnecessary type cast in dmar_table_detect()
2. type cast from acpi_dmar_header to acpi_dmar_hardware_unit directly
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c | 15 ---
include/linux/dmar.h |2 +-
2 files changed, 5 insertions(+), 12 deletions(-)
diff --git a/drivers
(), so
that it notices this is an invalid dmar and avoids related setup jobs.
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c | 34 +-
1 file changed, 17 insertions(+), 17 deletions(-)
diff --git a/drivers/iommu/dmar.c b/drivers/iommu/dmar.c
index 04f199c
easy to understand.
V2:
* add patch 3 and 4
Wei Yang (4):
iommu/vt-d: replace *hdr with hdr[0] in struct dmar_drhd_unit
iommu/vt-d: use zero-sized array in DMAR related ACPI structures
iommu/vt-d: check Register Base Address at the beginning of
dmar_parse_one_drhd()
iommu/vt-d: refine
According to the code path, iommu_callback_data is passed in
iommu_bus_init() and just used in {add/remove}_iommu_group, by when the
bus->iommu_ops is already set up properly.
This patch removes the iommu_callback_data by retrieving iommu_ops from
bus->iommu_ops directly.
Signed-off-b
unnecessary type cast in dmar_table_detect()
2. type cast from acpi_dmar_header to acpi_dmar_hardware_unit directly
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c | 15 ---
include/linux/dmar.h | 2 +-
2 files changed, 5 insertions(+), 12 deletions(-)
diff --git a/drivers/iommu
-sized array is
usually used for this purpose.
This patch adds zero-sized array for variable elements in DMAR ACPI
structures, which tries to make the code more audience friendly.
Signed-off-by: Wei Yang
---
drivers/iommu/dmar.c | 22 ++
include/acpi/actbl2.h | 31
Any more comments? Or this one is not proper?
On Thu, Aug 22, 2013 at 09:33:27PM -0600, Alex Williamson wrote:
>[+cc iommu]
>
>On Fri, 2013-08-23 at 09:55 +0800, Wei Yang wrote:
>> When removing a device from the system, iommu_group driver will try to
>> disconnect it fro
50 matches
Mail list logo