On 05/13/15 at 09:47am, Li, ZhenHua wrote: > On 05/12/2015 04:37 PM, Dave Young wrote: > >Seems the subject was truncated? Maybe "re" means root entry? Then please fix it > > > >On 05/11/15 at 05:52pm, Li, Zhen-Hua wrote: > >>Add functions to load root entry table from old kernel, and to save updated > >>root entry table. > >>Add two member in struct intel_iommu, to store the RTA in old kernel, and > >>the mapped virt address of it. > > > >Please explain a bit what is "RTA" in patch log, it is unclear to most of people > >who do not know iommu details. > > > >> > > If people want to understand this patchset, I assume they have read the vt-d > specs. RTA is defined clearly in this spec. > I think explain a bit is better, it will be easier for review, RTA is defined in spec, right, but if you refer to it in kernel source code, describe the meaning will help people to understand your code. I would not stick on this 'RTA', but a descriptive subject and patch log is still important, please check the logs, not only for this patch. Thanks Dave -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
