On Tue, Mar 06, 2007 at 04:25:41PM -0800, Andrew Morton wrote:
On Wed, 28 Feb 2007 15:25:54 +0100
Joerg Roedel [EMAIL PROTECTED] wrote:
From: Joerg Roedel [EMAIL PROTECTED]
This patch simplifies the get_cycles_sync() function by removing the
#ifdefs from it. Further it introduces
:
* rebased to current linus git tree
* replaced RDTSCP usage in get_cycles_sync with the opcode to
make it compile with older binutils
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body
From: Joerg Roedel [EMAIL PROTECTED]
This patch extends the alternative instructions framework to support 2
alternative instructions.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
diff --git a/arch/i386/kernel
From: Joerg Roedel [EMAIL PROTECTED]
In this patch updates the x86_64 architecture to work with the changes
to alternative instructions in i386
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
diff --git a/arch/x86_64/lib
From: Joerg Roedel [EMAIL PROTECTED]
This patch adds the X86_FEATURE_SYNC_RDTSC to the i386 architecture.
This is very helpfull to simplify the get_cycles_sync() function and
remove the #ifdefs from it.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
--
Joerg Roedel
Operating System Research
From: Joerg Roedel [EMAIL PROTECTED]
This patch simplifies the get_cycles_sync() function by removing
the #ifdefs from it. Further it introduces an optimization for AMD
processors. There the RDTSCP instruction is used instead of CPUID;RDTSC
which is helpfull if the kernel runs as a KVM guest
On Fri, Mar 09, 2007 at 08:10:03PM +0200, Avi Kivity wrote:
Joerg Roedel wrote:
From: Joerg Roedel [EMAIL PROTECTED]
This patch simplifies the get_cycles_sync() function by removing
the #ifdefs from it. Further it introduces an optimization for AMD
processors. There the RDTSCP instruction
On Mon, Mar 12, 2007 at 02:09:18PM +0100, Andi Kleen wrote:
On Monday 12 March 2007 14:02, Joerg Roedel wrote:
On Fri, Mar 09, 2007 at 08:10:03PM +0200, Avi Kivity wrote:
Joerg Roedel wrote:
From: Joerg Roedel [EMAIL PROTECTED]
This patch simplifies the get_cycles_sync() function
On Mon, Mar 12, 2007 at 02:29:43PM +0100, Michael Matz wrote:
Hi Joerg,
On Mon, 12 Mar 2007, Joerg Roedel wrote:
+#define RDTSCP .byte 0x0f, 0x01, 0xf9
+alternative_io_two(cpuid\nrdtsc,
+ rdtsc, X86_FEATURE_SYNC_RDTSC
On Mon, Mar 12, 2007 at 06:08:11PM +0200, Avi Kivity wrote:
Joerg Roedel wrote:
On Fri, Mar 09, 2007 at 08:10:03PM +0200, Avi Kivity wrote:
Joerg Roedel wrote:
From: Joerg Roedel [EMAIL PROTECTED]
This patch simplifies the get_cycles_sync() function by removing
the #ifdefs from
From: Mark Langsdorf [EMAIL PROTECTED]
From: Joerg Roedel [EMAIL PROTECTED]
This patch fixes the reporting of cpu_mhz in /proc/cpuinfo on CPUs with
a constant TSC rate and a kernel with disabled cpufreq.
Signed-off-by: Mark Langsdorf [EMAIL PROTECTED]
Signed-off-by: Joerg Roedel [EMAIL PROTECTED
On Thu, Mar 15, 2007 at 12:01:09AM +0100, Andi Kleen wrote:
On Tuesday 13 March 2007 11:00, Joerg Roedel wrote:
From: Mark Langsdorf [EMAIL PROTECTED]
From: Joerg Roedel [EMAIL PROTECTED]
This patch fixes the reporting of cpu_mhz in /proc/cpuinfo on CPUs with
a constant TSC rate
mismatch
make[1]: *** [kvmctl.o] Error 1
I don't see a kvm-13 on the KVM website.
You will find the kvm-13 release in the SourceForge download area of
KVM[1]. Kvm-12 is still required for 2.6.20 kernels.
Joerg
[1] http://sourceforge.net/project/showfiles.php?group_id=180599
--
Joerg
From: Joerg Roedel [EMAIL PROTECTED]
This patch changes the SVM code to intercept SMIs and handle it
outside the guest.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
diff --git a/drivers/kvm/svm.c b/drivers/kvm/svm.c
index
is scanned and the first matching entry wins. The
patches should apply cleanly to recent Linus git kernels.
Regards,
Joerg
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL
From: Joerg Roedel [EMAIL PROTECTED]
This patch modifies the get_cycles_sync() function on i386 and x86_64 to
use the RDTSCP (if it is available) instruction to synchronize with the
CPU core and not CPUID. This is especially usefull when running Linux as
a KVM guest because CPUID is intercepted
[Patch 2/3] of course.
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read
From: Joerg Roedel [EMAIL PROTECTED]
This patch extends the alternative instruction framework to support an
arbitrary number of alternatives on the i386 architecture.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
diff
From: Joerg Roedel [EMAIL PROTECTED]
This patch adds the necessary changes to extend the i386 alternative
instruction framework extension on the x86_64 architecture.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
diff --git
From: Joerg Roedel [EMAIL PROTECTED]
This patch replaces the deprecated functions in drivers/char/riscom8.c
and fixes the compile warnings they produced.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
diff --git a/drivers/char/riscom8.c b/drivers/char/riscom8.c
index 7014525..54cb8cb 100644
On Mon, Mar 26, 2007 at 10:52:09PM +0400, Alexey Dobriyan wrote:
On Sun, Mar 25, 2007 at 10:36:02PM +0200, Joerg Roedel wrote:
This patch replaces the deprecated functions in drivers/char/riscom8.c
and fixes the compile warnings they produced.
That's not the point of exercise. Make it SMP
On Mon, Feb 19, 2007 at 10:29:22PM +0100, Andi Kleen wrote:
On Monday 19 February 2007 20:07, Joerg Roedel wrote:
From: Joerg Roedel [EMAIL PROTECTED]
This patch adds the necessary changes to extend the i386 alternative
instruction framework extension on the x86_64 architecture.
Looks
= 0x5a5a;
break;
default:
BUG();
Is it a good idea to call BUG() on an undefined hypercall? I assume its
better to inform the guest that is has issued an illegal call...
Joerg
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
() function. The optimization changes this function to
use RDTSCP instead of CPUID;RDTSC if this instruction is available.
Don't use CPUID there is really important if the kernel runs as a KVM
guest, because this instruction is intercepted and causes an expensive
VMEXIT.
--
Joerg Roedel
Operating System
From: Joerg Roedel [EMAIL PROTECTED]
This patch extends the alternative instructions framework to support 2
alternative instructions.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
diff --git a/arch/i386/kernel
From: Joerg Roedel [EMAIL PROTECTED]
In this patch updates the x86_64 architecture to work with the changes
to alternative instructions in i386
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
diff --git a/arch/x86_64/lib
From: Joerg Roedel [EMAIL PROTECTED]
This patch simplifies the get_cycles_sync() function by removing the
#ifdefs from it. Further it introduces an optimization for AMD
processors. There the RDTSCP instruction is used instead of CPUID;RDTSC
which is helpfull if the kernel runs as a KVM guest
From: Joerg Roedel [EMAIL PROTECTED]
This patch adds the X86_FEATURE_SYNC_RDTSC to the i386 architecture.
This is very helpfull to simplify the get_cycles_sync() function and
remove the #ifdefs from it.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
--
Joerg Roedel
Operating System Research
From: Joerg Roedel [EMAIL PROTECTED]
This patch removes the reporting of the constant_tsc flag from the
power management field in /proc/cpuinfo. The NULL value there was
replaced by because the former would result in a printout of [8] if
the flag is set.
Signed-off-by: Joerg Roedel [EMAIL
This small patch series introduces the reporting of the SVM revision,
the number of ASIDs and the flags in /proc/cpuinfo on SVM capable AMD
processors. The patches are for i386 and for x86_64 architectures.
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
From: Joerg Roedel [EMAIL PROTECTED]
This patch adds the advanced SVM reporting to /proc/cpuinfo on the x86_64
architecture.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
diff --git a/arch/x86_64/kernel/setup.c b/arch
From: Joerg Roedel [EMAIL PROTECTED]
This patch adds the advanced SVM reporting in /proc/cpuinfo to the i386
architecture.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
--
Joerg Roedel
Operating System Research Center
AMD Saxony LLC Co. KG
diff --git a/arch/i386/kernel/cpu/amd.c b/arch/i386
On Mon, Mar 05, 2007 at 05:46:27PM +0300, Michael Tokarev wrote:
Joerg Roedel wrote:
+++ b/arch/x86_64/kernel/setup.c
+ static char *x86_svm_flags[] = {
+ npt,
+ lbrv,
+ svmlock,
+ };
A nitpick:
static const char *const x86_svm_flags
On Mon, Mar 05, 2007 at 04:25:31PM +0100, Andi Kleen wrote:
On Monday 05 March 2007 15:39, Joerg Roedel wrote:
From: Joerg Roedel [EMAIL PROTECTED]
This patch adds the advanced SVM reporting to /proc/cpuinfo on the x86_64
architecture.
Adding more fields to /proc/cpuinfo is always
On Fri, Sep 14, 2007 at 11:57:59AM +0400, Manu Abraham wrote:
What do you think about IOMMU?
Just because AMD or INTEL want to invent some whizzy new technology it
doesn't say anything about the TV card development and retail business.
Intel and AMD have teams of Linux engineers
On Fri, Sep 14, 2007 at 04:00:30PM +0400, Manu Abraham wrote:
Joerg Roedel wrote:
On Fri, Sep 14, 2007 at 11:57:59AM +0400, Manu Abraham wrote:
What do you think about IOMMU?
Just because AMD or INTEL want to invent some whizzy new technology it
doesn't say anything about the TV card
On Fri, Sep 14, 2007 at 08:56:40PM +0400, Manu Abraham wrote:
I do understand that (an earlier reply from Grant Grundler on the same
[1], while working on something else), but that wasn't exactly what i
was getting at. The bridges are in fact tied up with a certain CPU class.
Though your
On Wed, Nov 07, 2007 at 04:21:02PM +0200, Amit Shah wrote:
@@ -1649,6 +1913,15 @@ int kvm_emulate_hypercall(struct kvm_vcpu *vcpu)
}
switch (nr) {
+ case KVM_PV_DMA_MAP:
+ ret = pv_map_hypercall(vcpu, a0, a1);
+ break;
+ case KVM_PV_DMA_UNMAP:
On Mon, Nov 12, 2007 at 12:50:01PM +0200, Muli Ben-Yehuda wrote:
On Wed, Nov 07, 2007 at 04:21:04PM +0200, Amit Shah wrote:
We make the dma_mapping_ops structure to point to our structure so
that every DMA access goes through us. (This is the reason this only
works for 64-bit guest.
This patch replaces the manual permission setup for pages in ioremap_64.c with
the pre-defined __PAGE_KERNEL_EXEC value.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
arch/x86/mm/ioremap_64.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/arch/x86/mm/ioremap_64.c
This small series of patches cleans up the definition and use of the _PAGE_*
and __PAGE_KERNEL* permissions for the x86_64 code in the x86 architecture a
little bit.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More
This patch defines the _PAGE_* paging attributes in pgtable_64.h in terms of
the former defined _PAGE_BIT_* values.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
include/asm-x86/pgtable_64.h | 27 +++
1 files changed, 15 insertions(+), 12 deletions(-)
diff --git
This minor cleanup replaces _KERNPG_TABLE with the __PAGE_KERNEL* for 2MB PTEs
in the x86_64 memory initialization code. The __PAGE_KERNEL* defines are more
appropriate for PTEs.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
arch/x86/mm/init_64.c |4 ++--
1 files changed, 2 insertions
On Tue, Dec 04, 2007 at 01:38:09PM +0100, Ingo Molnar wrote:
* Joerg Roedel [EMAIL PROTECTED] wrote:
+#define _PAGE_PRESENT (_AC(1,UL)_PAGE_BIT_PRESENT)
please run patches through scripts/checkpatch.pl, it gives:
total: 10 errors, 0 warnings, 42 lines checked
(please send
This minor cleanup replaces _KERNPG_TABLE with the __PAGE_KERNEL* for 2MB PTEs
in the x86_64 memory initialization code. The __PAGE_KERNEL* defines are more
appropriate for PTEs.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
arch/x86/mm/init_64.c |4 ++--
1 files changed, 2 insertions
This small series of patches cleans up the definition and use of the _PAGE_*
and __PAGE_KERNEL* permissions for the x86_64 code in the x86 architecture a
little bit.
The scripts are checked by checkpatch.pl now.
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the
This patch defines the _PAGE_* paging attributes in pgtable_64.h in terms of
the former defined _PAGE_BIT_* values.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
include/asm-x86/pgtable_64.h | 27 +++
1 files changed, 15 insertions(+), 12 deletions(-)
diff --git
This patch replaces the manual permission setup for pages in ioremap_64.c with
the pre-defined __PAGE_KERNEL_EXEC value.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
arch/x86/mm/ioremap_64.c |3 +--
1 files changed, 1 insertions(+), 2 deletions(-)
diff --git a/arch/x86/mm/ioremap_64.c
On Tue, Dec 04, 2007 at 10:07:54PM +0100, Ingo Molnar wrote:
* Joerg Roedel [EMAIL PROTECTED] wrote:
On Tue, Dec 04, 2007 at 01:38:09PM +0100, Ingo Molnar wrote:
* Joerg Roedel [EMAIL PROTECTED] wrote:
+#define _PAGE_PRESENT (_AC(1,UL)_PAGE_BIT_PRESENT)
please run
This patch does some whitespace cleanups in the paging code to fix some
checkpatch.pl warnings of my formerly merged cleanup patches.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
arch/x86/mm/init_64.c|2 +-
include/asm-x86/pgtable_64.h | 22 +++---
2 files
On Wed, Dec 05, 2007 at 05:37:24PM +0100, Heiko Carstens wrote:
On Wed, Dec 05, 2007 at 03:40:12PM +0100, Joerg Roedel wrote:
This patch does some whitespace cleanups in the paging code to fix some
checkpatch.pl warnings of my formerly merged cleanup patches
On Wed, Dec 05, 2007 at 12:27:58PM +0100, Ingo Molnar wrote:
* Joerg Roedel [EMAIL PROTECTED] wrote:
please run patches through scripts/checkpatch.pl, it gives:
total: 10 errors, 0 warnings, 42 lines checked
(please send a followup cleanup patch - i picked up your
On Tue, Dec 11, 2007 at 03:27:17PM +0100, Ingo Molnar wrote:
* Dor Laor [EMAIL PROTECTED] wrote:
Here [include/asm-x86/tsc.h]:
/* Like get_cycles, but make sure the CPU is synchronized. */
static __always_inline cycles_t get_cycles_sync(void)
{
unsigned long long ret;
Hi,
I would like to answer what the special purpose of the get_cycles_sync()
function is in the x86 architecture. In special I ask myself why
this function has to be *sync*?
I mean, the sync should guarantee here that the CPU does not execute the
RDTSC instruction out-of-order, thats clear. But
Hi Andi,
On Tue, Oct 30, 2007 at 09:21:02PM +0100, Andi Kleen wrote:
Joerg Roedel [EMAIL PROTECTED] writes:
I would like to answer what the special purpose of the get_cycles_sync()
function is in the x86 architecture. In special I ask myself why
this function has to be *sync*?
Vojtech
Hi Vojtech,
On Tue, Oct 30, 2007 at 11:02:09PM +0100, Vojtech Pavlik wrote:
The K8's still guarantee that subsequent RDTSCs return increasing
values, even if the processor reorders them.
What could have been happening then was that the RDTSC instruction might
have been reordered by the CPU
This patchset includes two patches.
1. Checks for the MCA fetures as early as possible and signedness fixup.
2. Minor coding style cleanup.
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at
Egger [EMAIL PROTECTED]
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
arch/x86/kernel/cpu/mcheck/k7.c |6 +-
arch/x86/kernel/cpu/mcheck/mce.c | 12 ++--
arch/x86/kernel/cpu/mcheck/mce.h |2 +-
arch/x86/kernel/cpu/mcheck/p5.c |4
arch/x86/kernel/cpu/mcheck/p6.c
remove one indent level
Signed-off-by: Christoph Egger [EMAIL PROTECTED]
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
arch/x86/kernel/cpu/mcheck/mce.c | 40 +++---
1 files changed, 20 insertions(+), 20 deletions(-)
diff --git a/arch/x86/kernel/cpu/mcheck
On Fri, Oct 19, 2007 at 02:40:25PM +0200, Muli Ben-Yehuda wrote:
On Fri, Oct 19, 2007 at 02:38:11PM +0200, Joerg Roedel wrote:
This patch renames the include file asm-x86/iommu.h to asm-x86/gart.h to
make
clear to which IOMMU implementation it belongs. The patch also adds GART
This patch renames the include file asm-x86/iommu.h to asm-x86/gart.h to make
clear to which IOMMU implementation it belongs. The patch also adds GART to
the Kconfig line.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
arch/x86/kernel/aperture_64.c |2 +-
arch/x86/kernel/early
This patch renames the IOMMU config option to GART_IOMMU because in fact it
means the GART and not general support for an IOMMU on x86.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
arch/x86/kernel/Makefile_64|2 +-
arch/x86/kernel/early-quirks.c |2 +-
arch/x86/kernel/pci-dma_64
This patch renames the 4 symbols iommu_hole_init(), iommu_aperture,
iommu_aperture_allowed, iommu_aperture_disabled.
It replaces the iommu_ with gart_ in the symbol name. All these symbols are
only used for the GART implementation of IOMMUs.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED
This patch makes some functions and variables static in pci-gart_64.c which are
not used somewhere else.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
arch/x86/kernel/pci-gart_64.c |8
1 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/x86/kernel/pci-gart_64.c b
This series of patches contains cleanups for the GART IOMMU implementation in
Linux. It mostly renames symbols to make clear that they belong only to the
GART and not to the implementations for other IOMMUs. It also makes some
unnecessary exported functions static in the GART implementation.
-
This patch renames the include file asm-x86/iommu.h to asm-x86/gart.h to make
clear to which IOMMU implementation it belongs. The patch also adds GART to
the Kconfig line.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
arch/x86/kernel/aperture_64.c |2 +-
arch/x86/kernel/early
On Tue, Oct 23, 2007 at 07:43:47PM +0200, Andi Kleen wrote:
On Tuesday 23 October 2007 19:41:33 Joerg Roedel wrote:
This patch renames the 4 symbols iommu_hole_init(), iommu_aperture,
iommu_aperture_allowed, iommu_aperture_disabled.
It replaces the iommu_ with gart_ in the symbol name
On Tue, Oct 23, 2007 at 08:10:54PM +0200, Andi Kleen wrote:
On Tuesday 23 October 2007 20:04:36 H. Peter Anvin wrote:
Dave Jones wrote:
On Tue, Oct 23, 2007 at 07:47:11PM +0200, Joerg Roedel wrote:
On Tue, Oct 23, 2007 at 07:43:47PM +0200, Andi Kleen wrote:
On Tuesday 23 October
This patch renames the include file asm-x86/iommu.h to asm-x86/gart.h to make
clear to which IOMMU implementation it belongs. The patch also adds GART to
the Kconfig line.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
Acked-by: Muli Ben-Yehuda [EMAIL PROTECTED]
---
arch/x86/kernel/aperture_64.c
This patch makes some functions and variables static in pci-gart_64.c which are
not used somewhere else.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
Acked-by: Muli Ben-Yehuda [EMAIL PROTECTED]
---
arch/x86/kernel/pci-gart_64.c |8
1 files changed, 4 insertions(+), 4 deletions
This patch renames the IOMMU config option to GART_IOMMU because in fact it
means the GART and not general support for an IOMMU on x86.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
Acked-by: Muli Ben-Yehuda [EMAIL PROTECTED]
---
arch/x86/kernel/Makefile_64|2 +-
arch/x86/kernel/early
-
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
This patch renames the 4 symbols iommu_hole_init(), iommu_aperture,
iommu_aperture_allowed, iommu_aperture_disabled. All these symbols are only
used for the GART implementation of IOMMUs.
It adds and additional gart_ prefix to them.
Signed-off-by: Joerg Roedel [EMAIL PROTECTED]
---
arch/x86
This series of patches contains cleanups for the GART IOMMU implementation in
Linux. It mostly renames symbols to make clear that they belong only to the
GART and not to the implementations for other IOMMUs. It also makes some
unnecessary exported functions static in the GART implementation. The
On Sun, Aug 26, 2012 at 08:21:26PM +0200, Sebastian Andrzej Siewior wrote:
On Mon, Aug 20, 2012 at 03:55:58PM +0200, Joerg Roedel wrote:
--- a/arch/x86/include/asm/hw_irq.h
+++ b/arch/x86/include/asm/hw_irq.h
@@ -101,6 +101,7 @@ static inline void set_io_apic_irq_attr(struct
On Sun, Aug 26, 2012 at 08:16:03PM +0200, Sebastian Andrzej Siewior wrote:
On Mon, Aug 20, 2012 at 03:55:56PM +0200, Joerg Roedel wrote:
diff --git a/drivers/iommu/irq_remapping.c b/drivers/iommu/irq_remapping.c
index 827f8a1..71824fc 100644
--- a/drivers/iommu/irq_remapping.c
+++ b
On Sun, Aug 26, 2012 at 08:27:55PM +0200, Sebastian Andrzej Siewior wrote:
On Mon, Aug 20, 2012 at 03:56:01PM +0200, Joerg Roedel wrote:
--- a/arch/x86/kernel/apic/io_apic.c
+++ b/arch/x86/kernel/apic/io_apic.c
@@ -2981,8 +2981,8 @@ void destroy_irq(unsigned int irq
On Sun, Aug 26, 2012 at 08:30:30PM +0200, Sebastian Andrzej Siewior wrote:
On Mon, Aug 20, 2012 at 03:56:02PM +0200, Joerg Roedel wrote:
--- a/arch/x86/include/asm/irq_remapping.h
+++ b/arch/x86/include/asm/irq_remapping.h
@@ -44,6 +44,9 @@ extern void compose_remapped_msi_msg(struct
On Sun, Aug 26, 2012 at 08:41:40PM +0200, Sebastian Andrzej Siewior wrote:
On Mon, Aug 20, 2012 at 03:56:03PM +0200, Joerg Roedel wrote:
This call-back points to the right function for initializing
the msi_msg structure.
So you pull out the compose_remapped_msi_msg() out of msi_compose_msg
On Sun, Aug 26, 2012 at 08:52:33PM +0200, Sebastian Andrzej Siewior wrote:
Basically you shuffle the code from up there, down there and call from behind
a function pointer. There are two things different this time:
- no version check = 0x20
I belive this is obsolete since this runs only on
Move these checks to IRQ remapping code by introducing the
panic_on_irq_remap() function.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/irq_remapping.h |5 +
arch/x86/kernel/apic/io_apic.c |6 ++
drivers/iommu/irq_remapping.c|6
This function does irq-remapping specific interrupt setup
like modifying the chip defaults.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/irq_remapping.h |9 +
arch/x86/kernel/apic/io_apic.c | 13 +++--
drivers/iommu/irq_remapping.c
Move the three easy to move checks in the x86' apic.c file
into the IRQ-remapping code.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/kernel/apic/apic.c |9 +++--
drivers/iommu/irq_remapping.c | 11 +--
2 files changed, 12 insertions(+), 8 deletions(-)
diff
Hi,
here is the third version of the patch-set to improve the abstraction
of interrupt remapping in the x86 core code. A more detailed description
can be found in the original post at:
https://lkml.org/lkml/2012/8/7/317
Changes from v2-v3:
* Rebased to v3.6-rc7
* Worked
The function is called unconditionally now in IO-APIC code
removing another irq_remapped() check from x86 core code.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/kernel/apic/io_apic.c |4 ++--
drivers/iommu/irq_remapping.c |5 -
2 files changed, 6 insertions(+), 3
This function pointer can be overwritten by the IRQ
remapping code. The irq_remapping_enabled check can be
removed from default_setup_hpet_msi.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/hpet.h |5 +++--
arch/x86/include/asm/x86_init.h |1 +
arch/x86
The irq_remapped function is only used in IOMMU code after
the last patch. So move its definition there too.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/irq_remapping.h | 10 --
drivers/iommu/irq_remapping.c|5 +
2 files changed, 5
Use seperate routines to setup MSI IRQs for both
irq_remapping_enabled cases.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/irq_remapping.h | 12 ---
arch/x86/include/asm/pci.h |2 ++
arch/x86/kernel/apic/io_apic.c | 26
This call-back is used to dump IO-APIC entries for debugging
purposes into the kernel log. VT-d needs a special routine
for this and will overwrite the default.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/io_apic.h |3 +
arch/x86/include/asm/x86_init.h
IO-APIC and PIC use the same resume routines when IRQ
remapping is enabled or disabled. So it should be safe to
mask the other APICs for the IRQ-remapping-disabled case
too.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/kernel/apic/apic.c | 19 +--
1 file
This callback replaces the old __eoi_ioapic_pin function
which needs a special path for interrupt remapping.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/io_apic.h |5 +
arch/x86/include/asm/x86_init.h |1 +
arch/x86/kernel/apic/io_apic.c | 20
Move all the code to either to the header file
asm/irq_remapping.h or to drivers/iommu/.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/hw_irq.h|1 +
arch/x86/include/asm/io_apic.h |5
arch/x86/include/asm/irq_remapping.h | 17
This function is only called when irq-remapping is disabled.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/kernel/apic/io_apic.c |3 ---
1 file changed, 3 deletions(-)
diff --git a/arch/x86/kernel/apic/io_apic.c b/arch/x86/kernel/apic/io_apic.c
index 360f62b..d5cd6e1 100644
Remove the last left-over from this flag from x86 code.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/irq_remapping.h |4
drivers/iommu/dmar.c |2 ++
drivers/iommu/intel-iommu.c |2 ++
drivers/iommu/irq_remapping.h
This function is only called from default_ioapic_set_affinity()
which is only used when interrupt remapping is disabled
since the introduction of the set_affinity function pointer.
So the check will always evaluate as true and can be
removed.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
With interrupt remapping a special function is used to
change the affinity of an IO-APIC interrupt. Abstract this
with a function pointer.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/io_apic.h |4
arch/x86/include/asm/irq_remapping.h |9
as necessary by interrupt remapping.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/io_apic.h |2 ++
arch/x86/include/asm/x86_init.h |9 +
arch/x86/kernel/apic/io_apic.c | 41 ++-
arch/x86/kernel/x86_init.c |9
This pointer is changed to a different function when IRQ
remapping is enabled.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/io_apic.h |5 +
arch/x86/include/asm/x86_init.h |5 +
arch/x86/kernel/apic/io_apic.c | 14 +-
arch/x86/kernel
Add a data structure to store information the IOMMU driver
can use to get from a 'struct irq_cfg' to the remapping
entry.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/hw_irq.h | 12 +++-
1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/arch/x86
.
Otherwise the default function is called.
Signed-off-by: Joerg Roedel joerg.roe...@amd.com
---
arch/x86/include/asm/io_apic.h |4 +++
arch/x86/include/asm/x86_init.h |4 +++
arch/x86/kernel/apic/io_apic.c | 59 ---
arch/x86/kernel/x86_init.c |1
1 - 100 of 6605 matches
Mail list logo