From: Jiang Liu <jiang.liu@linux.intel.com>
To: Bjorn Helgaas <bhelgaas@google.com>
Cc: Joerg Roedel <joro@8bytes.org>,
David Woodhouse <dwmw2@infradead.org>,
Yinghai Lu <yinghai@kernel.org>,
Dan Williams <dan.j.williams@intel.com>,
Vinod Koul <vinod.koul@intel.com>,
"Rafael J . Wysocki" <rafael.j.wysocki@intel.com>,
Ashok Raj <ashok.raj@intel.com>,
Yijing Wang <wangyijing@huawei.com>,
Tony Luck <tony.luck@intel.com>,
"open list:INTEL IOMMU (VT-d)" <iommu@lists.linux-foundation.org>,
"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
linux hotplug mailing <linux-hotplug@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
dmaengine@vger.kernel.org
Subject: Re: [Patch Part3 V3 00/21] Enable support of Intel DMAR device hotplug
Date: Thu, 26 Jun 2014 13:59:58 +0000 [thread overview]
Message-ID: <53AC275E.4090602@linux.intel.com> (raw)
In-Reply-To: <CAErSpo5z9OH_u3f9L5qKoJsJ+mgRU5rba9mnQG0Y5XXn0zcEvg@mail.gmail.com>
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@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
WARNING: multiple messages have this Message-ID (diff)
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/
>
WARNING: multiple messages have this Message-ID (diff)
From: Jiang Liu <jiang.liu@linux.intel.com>
To: Bjorn Helgaas <bhelgaas@google.com>
Cc: Joerg Roedel <joro@8bytes.org>,
David Woodhouse <dwmw2@infradead.org>,
Yinghai Lu <yinghai@kernel.org>,
Dan Williams <dan.j.williams@intel.com>,
Vinod Koul <vinod.koul@intel.com>,
"Rafael J . Wysocki" <rafael.j.wysocki@intel.com>,
Ashok Raj <ashok.raj@intel.com>,
Yijing Wang <wangyijing@huawei.com>,
Tony Luck <tony.luck@intel.com>,
"open list:INTEL IOMMU (VT-d)" <iommu@lists.linux-foundation.org>,
"linux-pci@vger.kernel.org" <linux-pci@vger.kernel.org>,
linux hotplug mailing <linux-hotplug@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
dmaengine@vger.kernel.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@mail.gmail.com>
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@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
next prev parent reply other threads:[~2014-06-26 13:59 UTC|newest]
Thread overview: 66+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-25 8:59 [Patch Part3 V3 00/21] Enable support of Intel DMAR device hotplug Jiang Liu
2014-06-25 9:02 ` 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 ` 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 ` Jiang Liu
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 ` Jiang Liu
2014-06-25 9:02 ` 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
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 ` Jiang Liu
2014-06-25 9:02 ` 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 21/21] pci, ACPI, iommu: enhance pci_root to support DMAR device hotplug Jiang Liu
2014-06-25 9:02 ` Jiang Liu
2014-06-25 9:02 ` Jiang Liu
2014-06-25 18:00 ` [Patch Part3 V3 00/21] Enable support of Intel " Bjorn Helgaas
2014-06-25 18:00 ` Bjorn Helgaas
2014-06-26 13:59 ` Jiang Liu [this message]
2014-06-26 13:59 ` Jiang Liu
2014-06-26 13:59 ` Jiang Liu
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@linux.intel.com \
--cc=ashok.raj@intel.com \
--cc=bhelgaas@google.com \
--cc=dan.j.williams@intel.com \
--cc=dmaengine@vger.kernel.org \
--cc=dwmw2@infradead.org \
--cc=iommu@lists.linux-foundation.org \
--cc=joro@8bytes.org \
--cc=linux-hotplug@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=rafael.j.wysocki@intel.com \
--cc=tony.luck@intel.com \
--cc=vinod.koul@intel.com \
--cc=wangyijing@huawei.com \
--cc=yinghai@kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.