From: Jiang Liu <jiang.liu-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
To: Bjorn Helgaas <bhelgaas-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org>
Cc: Tony Luck <tony.luck-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
Vinod Koul <vinod.koul-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
David Woodhouse <dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>,
"Rafael J . Wysocki"
<rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
linux hotplug mailing
<linux-hotplug-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
dmaengine-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
"open list:INTEL IOMMU (VT-d)"
<iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org>,
"linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
<linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
Dan Williams
<dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>,
Yinghai Lu <yinghai-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Subject: Re: [Patch Part3 V3 00/21] Enable support of Intel DMAR device hotplug
Date: Thu, 26 Jun 2014 21:59:58 +0800 [thread overview]
Message-ID: <53AC275E.4090602@linux.intel.com> (raw)
In-Reply-To: <CAErSpo5z9OH_u3f9L5qKoJsJ+mgRU5rba9mnQG0Y5XXn0zcEvg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
On 2014/6/26 2:00, Bjorn Helgaas wrote:
>> iommu/vt-d: match segment number when searching for dev_iotlb capable
>> devices
>> iommu/vt-d: use correct domain id to flush virtual machine domains
>> iommu/vt-d: introduce helper functions to improve code readability
>> iommu/vt-d: introduce helper functions to make code symmetric for
>> readability
>> iommu/vt-d: only dynamically allocate domain id for virtual domains
>> iommu/vt-d: fix possible invalid memory access caused by
>> free_dmar_iommu()
>> iommu/vt-d: avoid freeing virtual machine domain in free_dmar_iommu()
>> iommu/VT-d: simplify include/linux/dmar.h
>> iommu/vt-d: change iommu_enable/disable_translation to return void
>> iommu/vt-d: simplify intel_unmap_sg() and kill duplicated code
>> iommu/vt-d: introduce helper domain_pfn_within_range() to simplify
>> code
>> iommu/vt-d: introduce helper function iova_size() to improve code
>> readability
>> iommu/vt-d: fix bug in computing domain's iommu_snooping flag
>> IOMMU/vt-d: introduce helper function dmar_walk_resources()
>> iommu/vt-d: dynamically allocate and free seq_id for DMAR units
>> iommu/vt-d: implement DMAR unit hotplug framework
>> iommu/vt-d: search _DSM method for DMAR hotplug
>> iommu/vt-d: enhance intel_irq_remapping driver to support DMAR unit
>> hotplug
>> iommu/vt-d: enhance error recovery in function
>> intel_enable_irq_remapping()
>> iommu/vt-d: enhance intel-iommu driver to support DMAR unit hotplug
>> pci, ACPI, iommu: enhance pci_root to support DMAR device hotplug
>
> This looks a little sloppy; you have three different ways of
> capitalizing the area:
>
> iommu/vt-d:
> iommu/VT-d:
> IOMMU/vt-d:
>
> Also, "git log --oneline drivers/iommu" says that the convention for
> drivers/iommu is to capitalize the first word of the rest of the
> subject line.
Hi Bjorn,
Really sorry about my careless. I just rebased the patch set
onto v3.16-rc1, I should do more careful check about the commit
messages. I will wait for more comments and resend a new version
to address the commit message issues.
Thanks for your reminder again.
Regards!
Gerry
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
prev parent reply other threads:[~2014-06-26 13:59 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-25 9:02 [Patch Part3 V3 00/21] Enable support of Intel DMAR device hotplug Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 01/21] iommu/vt-d: match segment number when searching for dev_iotlb capable devices Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 02/21] iommu/vt-d: use correct domain id to flush virtual machine domains Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 04/21] iommu/vt-d: introduce helper functions to make code symmetric for readability Jiang Liu
[not found] ` <1403686967-10887-1-git-send-email-jiang.liu-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>
2014-06-25 9:02 ` [Patch Part3 V3 03/21] iommu/vt-d: introduce helper functions to improve code readability Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 05/21] iommu/vt-d: only dynamically allocate domain id for virtual domains Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 06/21] iommu/vt-d: fix possible invalid memory access caused by free_dmar_iommu() Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 07/21] iommu/vt-d: avoid freeing virtual machine domain in free_dmar_iommu() Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 08/21] iommu/VT-d: simplify include/linux/dmar.h Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 09/21] iommu/vt-d: change iommu_enable/disable_translation to return void Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 10/21] iommu/vt-d: simplify intel_unmap_sg() and kill duplicated code Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 11/21] iommu/vt-d: introduce helper domain_pfn_within_range() to simplify code Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 12/21] iommu/vt-d: introduce helper function iova_size() to improve code readability Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 13/21] iommu/vt-d: fix bug in computing domain's iommu_snooping flag Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 14/21] IOMMU/vt-d: introduce helper function dmar_walk_resources() Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 15/21] iommu/vt-d: dynamically allocate and free seq_id for DMAR units Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 16/21] iommu/vt-d: implement DMAR unit hotplug framework Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 17/21] iommu/vt-d: search _DSM method for DMAR hotplug Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 19/21] iommu/vt-d: enhance error recovery in function intel_enable_irq_remapping() Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 20/21] iommu/vt-d: enhance intel-iommu driver to support DMAR unit hotplug Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 21/21] pci, ACPI, iommu: enhance pci_root to support DMAR device hotplug Jiang Liu
2014-06-25 9:02 ` [Patch Part3 V3 18/21] iommu/vt-d: enhance intel_irq_remapping driver to support DMAR unit hotplug Jiang Liu
2014-06-25 18:00 ` [Patch Part3 V3 00/21] Enable support of Intel DMAR device hotplug Bjorn Helgaas
[not found] ` <CAErSpo5z9OH_u3f9L5qKoJsJ+mgRU5rba9mnQG0Y5XXn0zcEvg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-06-26 13:59 ` Jiang Liu [this message]
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=53AC275E.4090602@linux.intel.com \
--to=jiang.liu-vuqaysv1563yd54fqh9/ca@public.gmane.org \
--cc=bhelgaas-hpIqsD4AKlfQT0dZR+AlfA@public.gmane.org \
--cc=dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
--cc=dmaengine-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=dwmw2-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org \
--cc=iommu-cunTk1MwBs9QetFLy7KEm3xJsTq8ys+cHZ5vskTnxNA@public.gmane.org \
--cc=linux-hotplug-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=linux-pci-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=rafael.j.wysocki-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
--cc=tony.luck-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
--cc=vinod.koul-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org \
--cc=yinghai-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).