[PATCH] Documentation/Intel-IOMMU.txt: Modify definition of DRHD
Hi David & Joerg, According to "Intel Virtualization Technology for Directed I/O" specification, DRHD stands for "DMA Remapping Hardware Unit Definition" , not "DMA Engine Reporting Structure". Signed-off-by: Nan Xiao --- Documentation/Intel-IOMMU.txt | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Documentation/Intel-IOMMU.txt b/Documentation/Intel-IOMMU.txt index cf9431d..7b57fc0 100644 --- a/Documentation/Intel-IOMMU.txt +++ b/Documentation/Intel-IOMMU.txt @@ -10,7 +10,7 @@ This guide gives a quick cheat sheet for some basic understanding. Some Keywords DMAR - DMA remapping -DRHD - DMA Engine Reporting Structure +DRHD - DMA Remapping Hardware Unit Definition RMRR - Reserved memory Region Reporting Structure ZLR - Zero length reads from PCI devices IOVA - IO Virtual address. -- 2.4.3 Best Regards Nan Xiao ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
Re: [PATCH] Documentation/Intel-IOMMU.txt: Modify definition of DRHD
Hi Nan, I applied this patch with some formatting fixes, thanks. Details below: From: "Xiao, Nan (Nan@HPS Performance, Beijing)" git-am made this author-line out of your patch: "(Nan@HPS <(Nan@HPS>" Which doesn't even look like a valid email address. I fixed it, but please include a From: line in your future patches like this: From: Xiao, Nan This will guarantee that your patches will be accounted correctly. Next is the patch subject. For iommu patches please use a prefix like this: / In case of Intel VT-d this would look like "iommu/vt-d:", and my updated subject line looks like: Subject: x86/vt-d: Fix documentation of DRHD On Mon, Aug 24, 2015 at 06:22:42AM +, Xiao, Nan (Nan@HPS Performance, Beijing) wrote: > According to "Intel Virtualization Technology for Directed I/O" specification, > DRHD stands for "DMA Remapping Hardware Unit Definition" , not "DMA Engine > Reporting Structure". These lines are too long. Please try to limit the line-length in patches to 60 characters for iommu patches. Thanks, Joerg ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu
RE: [PATCH] Documentation/Intel-IOMMU.txt: Modify definition of DRHD
Hi Joerg, Firstly, thanks very much for your detailed comments! I first send this patch to linux-ker...@vger.kernel.org only (the result of get_maintainer.pl ), but after about 3 days, there is no any responses. I thought no one would notice this bug, so I resend the patch to you. But this morning, Jonathan confirmed he has merged the patch (https://lkml.org/lkml/2015/8/24/637) , so I think maybe there is no need for you to merge it again. I am very sorry for not updating the information immediately! Thanks for your understanding! Best Regards Nan Xiao -Original Message- From: j...@8bytes.org [mailto:j...@8bytes.org] Sent: Tuesday, August 25, 2015 4:52 PM To: Xiao, Nan (Nan@HPservers-Core-OE-PSC) Cc: dw...@infradead.org; iommu@lists.linux-foundation.org Subject: Re: [PATCH] Documentation/Intel-IOMMU.txt: Modify definition of DRHD Hi Nan, I applied this patch with some formatting fixes, thanks. Details below: From: "Xiao, Nan (Nan@HPS Performance, Beijing)" git-am made this author-line out of your patch: "(Nan@HPS <(Nan@HPS>" Which doesn't even look like a valid email address. I fixed it, but please include a From: line in your future patches like this: From: Xiao, Nan This will guarantee that your patches will be accounted correctly. Next is the patch subject. For iommu patches please use a prefix like this: / In case of Intel VT-d this would look like "iommu/vt-d:", and my updated subject line looks like: Subject: x86/vt-d: Fix documentation of DRHD On Mon, Aug 24, 2015 at 06:22:42AM +, Xiao, Nan (Nan@HPS Performance, Beijing) wrote: > According to "Intel Virtualization Technology for Directed I/O" > specification, DRHD stands for "DMA Remapping Hardware Unit Definition" , not > "DMA Engine Reporting Structure". These lines are too long. Please try to limit the line-length in patches to 60 characters for iommu patches. Thanks, Joerg ___ iommu mailing list iommu@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/iommu