linux-kernel - RE: [PATCH -tip] IA64: fix the compile error on IA64_DIG

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20090128100040J.fujita.tomonori@lab.ntt.co.jp>
Date:	Wed, 28 Jan 2009 10:00:11 +0900
From:	FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
To:	tony.luck@...el.com
Cc:	mingo@...e.hu, fujita.tomonori@....ntt.co.jp,
	linux-kernel@...r.kernel.org, linux-ia64@...r.kernel.org
Subject: RE: [PATCH -tip] IA64: fix the compile error on IA64_DIG_VTD

On Tue, 27 Jan 2009 09:17:58 -0800
"Luck, Tony" <tony.luck@...el.com> wrote:

> > > > ---
> > > >  arch/ia64/kernel/dma-mapping.c |    3 +++
> > > >  arch/ia64/kernel/pci-swiotlb.c |    3 ---
> > > >  2 files changed, 3 insertions(+), 3 deletions(-)
> > >
> > > Applied to tip/core/iommu, thanks!
> >
> > Tony, is this fix fine with you too?
> 
> It works ... and dma-mapping.c is OK as a location for the
> definition.  But there seemes to be some small scope for a
> related cleanup:
> 
> There are "extern" declarations for "iommu_detected" in both
> <asm/iommu.h> (x86 and ia64) and also in <linux/dmar.h>

We can remove them in dmar.h. X86 exported iommu_detected and no_iommu
in several IOMMU code and we merged them into dmar.h. Seems that we
forgot to clean up dmar.h.


> Does having both of these avoid some #include hell?

I think we could. The root problem is that X86 sets up IOMMU in a very
hacky way. When we added VT-d support to IA64, we needed to add the
hacky code to IA64 even though IA64 doesn't need it.

We need to clean up the code to set up IOMMUs. I'll try later.

=
From: FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
Subject: [PATCH -tip] intel-iommu: no need to export iommu_detected and no_iommu in dmar.h

The users of intel-iommu (x86 and IA64) export iommu_detected and
no_iommu in their own iommu.h.

Signed-off-by: FUJITA Tomonori <fujita.tomonori@....ntt.co.jp>
---
 include/linux/dmar.h |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)

diff --git a/include/linux/dmar.h b/include/linux/dmar.h
index f284407..3e82a13 100644
--- a/include/linux/dmar.h
+++ b/include/linux/dmar.h
@@ -129,7 +129,6 @@ extern void dmar_msi_write(int irq, struct msi_msg *msg);
 extern int dmar_set_interrupt(struct intel_iommu *iommu);
 extern int arch_setup_dmar_msi(unsigned int irq);
 
-extern int iommu_detected, no_iommu;
 extern struct list_head dmar_rmrr_units;
 struct dmar_rmrr_unit {
 	struct list_head list;		/* list of rmrr units	*/
-- 
1.6.0.6

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/