|   | 
      | 
  
  
      | 
      | 
  
 
     | 
    | 
  
  
     | 
    | 
  
  
    |   | 
      | 
  
  
    | 
         
xen-devel
[Xen-devel] [PATCH 05/10] x86/PCI: Clean up pci_cache_line_size
 
| 
To:  | 
Ingo Molnar <mingo@xxxxxxx> | 
 
| 
Subject:  | 
[Xen-devel] [PATCH 05/10] x86/PCI: Clean up pci_cache_line_size | 
 
| 
From:  | 
Jeremy Fitzhardinge <jeremy@xxxxxxxx> | 
 
| 
Date:  | 
Wed, 27 May 2009 00:24:37 -0700 | 
 
| 
Cc:  | 
Chris Wright <chrisw@xxxxxxxxxx>, Xen-devel <xen-devel@xxxxxxxxxxxxxxxxxxx>,	Matthew Wilcox <matthew@xxxxxx>,	Ky Srinivasan <ksrinivasan@xxxxxxxxxx>, kurt.hackel@xxxxxxxxxx,	the arch/x86 maintainers <x86@xxxxxxxxxx>,	Linux Kernel Mailing List <linux-kernel@xxxxxxxxxxxxxxx>,	Alex Nixon <alex.nixon@xxxxxxxxxx>,	Jeremy Fitzhardinge <jeremy.fitzhardinge@xxxxxxxxxx>,	Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx>,	Jens Axboe <jens.axboe@xxxxxxxxxx>,	Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx>,	Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>, Greg KH <gregkh@xxxxxxx> | 
 
| 
Delivery-date:  | 
Wed, 27 May 2009 00:39:43 -0700 | 
 
| 
Envelope-to:  | 
www-data@xxxxxxxxxxxxxxxxxxx | 
 
| 
In-reply-to:  | 
<1243409082-21349-1-git-send-email-jeremy@xxxxxxxx> | 
 
| 
List-help:  | 
<mailto:xen-devel-request@lists.xensource.com?subject=help> | 
 
| 
List-id:  | 
Xen developer discussion <xen-devel.lists.xensource.com> | 
 
| 
List-post:  | 
<mailto:xen-devel@lists.xensource.com> | 
 
| 
List-subscribe:  | 
<http://lists.xensource.com/mailman/listinfo/xen-devel>,	<mailto:xen-devel-request@lists.xensource.com?subject=subscribe> | 
 
| 
List-unsubscribe:  | 
<http://lists.xensource.com/mailman/listinfo/xen-devel>,	<mailto:xen-devel-request@lists.xensource.com?subject=unsubscribe> | 
 
| 
References:  | 
<1243409082-21349-1-git-send-email-jeremy@xxxxxxxx> | 
 
| 
Sender:  | 
xen-devel-bounces@xxxxxxxxxxxxxxxxxxx | 
 
 
 
From: Alex Nixon <alex.nixon@xxxxxxxxxx>
Separate out x86 cache_line_size initialisation code into its own
function (so it can be shared by Xen later in this patch series)
[ Impact: cleanup ]
Signed-off-by: Alex Nixon <alex.nixon@xxxxxxxxxx>
Signed-off-by: Jeremy Fitzhardinge <jeremy.fitzhardinge@xxxxxxxxxx>
Reviewed-by: "H. Peter Anvin" <hpa@xxxxxxxxx>
Reviewed-by: Matthew Wilcox <willy@xxxxxxxxxxxxxxx>
Reviewed-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx>
---
 arch/x86/include/asm/pci_x86.h |    1 +
 arch/x86/pci/common.c          |   17 +++++++++++------
 2 files changed, 12 insertions(+), 6 deletions(-)
diff --git a/arch/x86/include/asm/pci_x86.h b/arch/x86/include/asm/pci_x86.h
index e60fd3e..5401ca2 100644
--- a/arch/x86/include/asm/pci_x86.h
+++ b/arch/x86/include/asm/pci_x86.h
@@ -45,6 +45,7 @@ enum pci_bf_sort_state {
 extern unsigned int pcibios_max_latency;
 
 void pcibios_resource_survey(void);
+void pcibios_set_cache_line_size(void);
 
 /* pci-pc.c */
 
diff --git a/arch/x86/pci/common.c b/arch/x86/pci/common.c
index 2202b62..011ff45 100644
--- a/arch/x86/pci/common.c
+++ b/arch/x86/pci/common.c
@@ -412,26 +412,31 @@ struct pci_bus * __devinit pcibios_scan_root(int busnum)
 
 extern u8 pci_cache_line_size;
 
-int __init pcibios_init(void)
+void __init pcibios_set_cache_line_size(void)
 {
        struct cpuinfo_x86 *c = &boot_cpu_data;
 
-       if (!raw_pci_ops) {
-               printk(KERN_WARNING "PCI: System does not support PCI\n");
-               return 0;
-       }
-
        /*
         * Assume PCI cacheline size of 32 bytes for all x86s except K7/K8
         * and P4. It's also good for 386/486s (which actually have 16)
         * as quite a few PCI devices do not support smaller values.
         */
+
        pci_cache_line_size = 32 >> 2;
        if (c->x86 >= 6 && c->x86_vendor == X86_VENDOR_AMD)
                pci_cache_line_size = 64 >> 2;  /* K7 & K8 */
        else if (c->x86 > 6 && c->x86_vendor == X86_VENDOR_INTEL)
                pci_cache_line_size = 128 >> 2; /* P4 */
+}
+
+int __init pcibios_init(void)
+{
+       if (!raw_pci_ops) {
+               printk(KERN_WARNING "PCI: System does not support PCI\n");
+               return 0;
+       }
 
+       pcibios_set_cache_line_size();
        pcibios_resource_survey();
 
        if (pci_bf_sort >= pci_force_bf)
-- 
1.6.0.6
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
 |   
 
| <Prev in Thread] | 
Current Thread | 
[Next in Thread>
 |  
- [Xen-devel] [GIT PULL REPOST] xen/dom0/pci - Xen dom0 PCI access, Jeremy Fitzhardinge
- [Xen-devel] [PATCH 01/10] xen: Don't disable the I/O space, Jeremy Fitzhardinge
 
- [Xen-devel] [PATCH 03/10] Xen: Rename the balloon lock, Jeremy Fitzhardinge
 
- [Xen-devel] [PATCH 02/10] xen: Allow unprivileged Xen domains to	create iomap pages, Jeremy Fitzhardinge
 
- [Xen-devel] [PATCH 04/10] xen: Add xen_create_contiguous_region, Jeremy Fitzhardinge
 
- [Xen-devel] [PATCH 10/10] xen: define BIOVEC_PHYS_MERGEABLE(), Jeremy Fitzhardinge
 
- [Xen-devel] [PATCH 09/10] xen/pci: clean up Kconfig a bit, Jeremy Fitzhardinge
 
- [Xen-devel] [PATCH 08/10] x86/pci: make sure _PAGE_IOMAP it set on	pci mappings, Jeremy Fitzhardinge
 
- [Xen-devel] [PATCH 05/10] x86/PCI: Clean up pci_cache_line_size,
Jeremy Fitzhardinge <=
 
- [Xen-devel] [PATCH 06/10] x86/PCI: Enable scanning of all pci	functions, Jeremy Fitzhardinge
 
- [Xen-devel] [PATCH 07/10] Xen/x86/PCI: Add support for the Xen PCI	subsystem, Jeremy Fitzhardinge
 
 
 
 
 |  
  
 | 
    | 
  
  
    |   | 
    |