s390/pci: cleanup clp page allocation
authorSebastian Ott <sebott@linux.vnet.ibm.com>
Thu, 31 Jan 2013 18:54:03 +0000 (19:54 +0100)
committerMartin Schwidefsky <schwidefsky@de.ibm.com>
Thu, 14 Feb 2013 14:55:16 +0000 (15:55 +0100)
Use the __get_free_pages wrapper in clp_alloc_block. Also change the
allocation to use one page only. This page is used as CLP response
block e.g. to list available pci functions. Using one page we can
list > 250 pci functions at once and we have code to loop around this
CLP command (if not all functions fit into to the CLP block) already
in place.

Acked-by: Gerald Schaefer <gerald.schaefer@de.ibm.com>
Signed-off-by: Sebastian Ott <sebott@linux.vnet.ibm.com>
Signed-off-by: Martin Schwidefsky <schwidefsky@de.ibm.com>
arch/s390/include/asm/clp.h
arch/s390/pci/pci_clp.c

index 6c3aecc245ff3b17b9707d43bbd33ffd4258eec3..a0e71a501f7cecbf41252c0c549864659bd0e5f4 100644 (file)
@@ -2,7 +2,7 @@
 #define _ASM_S390_CLP_H
 
 /* CLP common request & response block size */
-#define CLP_BLK_SIZE                   (PAGE_SIZE * 2)
+#define CLP_BLK_SIZE                   PAGE_SIZE
 
 struct clp_req_hdr {
        u16 len;
index 702bd2693689acc99ab59202d2b1ea2222cf16ac..f339fe2feb15d0b5a786554ca0da960cbf9a2795 100644 (file)
@@ -37,8 +37,7 @@ static inline u8 clp_instr(void *data)
 
 static void *clp_alloc_block(void)
 {
-       struct page *page = alloc_pages(GFP_KERNEL, get_order(CLP_BLK_SIZE));
-       return (page) ? page_address(page) : NULL;
+       return (void *) __get_free_pages(GFP_KERNEL, get_order(CLP_BLK_SIZE));
 }
 
 static void clp_free_block(void *ptr)