[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH 1/5] iommu: trivial re-organisation to avoid unnecessary test

To: "Paul Durrant" <paul.durrant@xxxxxxxxxx>
From: "Jan Beulich" <JBeulich@xxxxxxxx>
Date: Mon, 13 May 2019 09:22:48 -0600
Cc: xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxxx>
Delivery-date: Mon, 13 May 2019 15:23:01 +0000
List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

>>> On 08.05.19 at 15:23, <paul.durrant@xxxxxxxxxx> wrote:
> An 'if ( !iommu_enabled )' followed by an 'if ( iommu_enabled )' with
> only a printk() in between seems a little silly. Move the printk() and
> use 'else' instead.
> 
> Signed-off-by: Paul Durrant <paul.durrant@xxxxxxxxxx>

Acked-by: Jan Beulich <jbeulich@xxxxxxxx>



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

References:
- [Xen-devel] [PATCH 0/5] iommu groups + cleanup
  - From: Paul Durrant
- [Xen-devel] [PATCH 1/5] iommu: trivial re-organisation to avoid unnecessary test
  - From: Paul Durrant

Prev by Date: Re: [Xen-devel] [PATCH 1/4] xen/watchdog: Fold exit paths to have a single unlock
Next by Date: Re: [Xen-devel] how to disable build of pv-shim?
Previous by thread: [Xen-devel] [PATCH 1/5] iommu: trivial re-organisation to avoid unnecessary test
Next by thread: [Xen-devel] [PATCH 2/5] iommu / x86: move call to scan_pci_devices() out of vendor code
Index(es):
- Date
- Thread

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.