Hi Joerg, Firstly, thanks very much for your detailed comments!
I first send this patch to [email protected] 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: [email protected] [mailto:[email protected]] Sent: Tuesday, August 25, 2015 4:52 PM To: Xiao, Nan (Nan@HPservers-Core-OE-PSC) Cc: [email protected]; [email protected] 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)" <[email protected]> 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 <[email protected]> This will guarantee that your patches will be accounted correctly. Next is the patch subject. For iommu patches please use a prefix like this: <arch>/<iommu> 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 +0000, 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 [email protected] https://lists.linuxfoundation.org/mailman/listinfo/iommu
