Skip to content

Commit

Permalink
[PATCH] powerpc: Remove ItLpRegSave area from the paca
Browse files Browse the repository at this point in the history
On iSeries, the paca contains, amongst other things an ItLpRegSave
structure used by the hypervisor to save registers.  The hypervisor
locates this area through a pointer at the beginning of the paca, so
the structure itself can be located elsewhere.  This patch moves the
reg_save area out into its own array.  This reduces the amount of
iSeries specific gunk which is visible to general powerpc code via
paca.h

Built and booted on POWER5 LPAR and iSeries RS64.

Signed-off-by: David Gibson <dwg@au1.ibm.com>
Signed-off-by: Paul Mackerras <paulus@samba.org>
  • Loading branch information
David Gibson authored and Paul Mackerras committed Jan 9, 2006
1 parent 456752f commit 1888e7b
Show file tree
Hide file tree
Showing 4 changed files with 18 additions and 15 deletions.
10 changes: 3 additions & 7 deletions arch/powerpc/kernel/paca.c
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
#include <asm/page.h>
#include <asm/lppaca.h>
#include <asm/iseries/it_lp_queue.h>
#include <asm/iseries/it_lp_reg_save.h>
#include <asm/paca.h>


Expand All @@ -26,8 +27,7 @@ extern unsigned long __toc_start;

/* The Paca is an array with one entry per processor. Each contains an
* lppaca, which contains the information shared between the
* hypervisor and Linux. Each also contains an ItLpRegSave area which
* is used by the hypervisor to save registers.
* hypervisor and Linux.
* On systems with hardware multi-threading, there are two threads
* per processor. The Paca array must contain an entry for each thread.
* The VPD Areas will give a max logical processors = 2 * max physical
Expand Down Expand Up @@ -57,11 +57,7 @@ extern unsigned long __toc_start;
#ifdef CONFIG_PPC_ISERIES
#define PACA_INIT_ISERIES(number) \
.lppaca_ptr = &paca[number].lppaca, \
.reg_save_ptr = &paca[number].reg_save, \
.reg_save = { \
.xDesc = 0xd397d9e2, /* "LpRS" */ \
.xSize = sizeof(struct ItLpRegSave) \
}
.reg_save_ptr = &iseries_reg_save[number],

#define PACA_INIT(number) \
{ \
Expand Down
7 changes: 7 additions & 0 deletions arch/powerpc/platforms/iseries/lpardata.c
Original file line number Diff line number Diff line change
Expand Up @@ -225,3 +225,10 @@ struct ItVpdAreas itVpdAreas = {
0,0
}
};

struct ItLpRegSave iseries_reg_save[] = {
[0 ... (NR_CPUS-1)] = {
.xDesc = 0xd397d9e2, /* "LpRS" */
.xSize = sizeof(struct ItLpRegSave),
},
};
2 changes: 2 additions & 0 deletions include/asm-powerpc/iseries/it_lp_reg_save.h
Original file line number Diff line number Diff line change
Expand Up @@ -81,4 +81,6 @@ struct ItLpRegSave {
u8 xRsvd3[176]; // Reserved 350-3FF
};

extern struct ItLpRegSave iseries_reg_save[];

#endif /* _ITLPREGSAVE_H */
14 changes: 6 additions & 8 deletions include/asm-powerpc/paca.h
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
#include <linux/config.h>
#include <asm/types.h>
#include <asm/lppaca.h>
#include <asm/iseries/it_lp_reg_save.h>
#include <asm/mmu.h>

register struct paca_struct *local_paca asm("r13");
Expand All @@ -31,9 +30,9 @@ struct task_struct;
*
* This structure is not directly accessed by firmware or the service
* processor except for the first two pointers that point to the
* lppaca area and the ItLpRegSave area for this CPU. Both the
* lppaca and ItLpRegSave objects are currently contained within the
* PACA but they do not need to be.
* lppaca area and the ItLpRegSave area for this CPU. The lppaca
* object is currently contained within the PACA but it doesn't need
* to be.
*/
struct paca_struct {
/*
Expand All @@ -48,7 +47,9 @@ struct paca_struct {
* accessed by the firmware
*/
struct lppaca *lppaca_ptr; /* Pointer to LpPaca for PLIC */
struct ItLpRegSave *reg_save_ptr; /* Pointer to LpRegSave for PLIC */
#ifdef CONFIG_PPC_ISERIES
void *reg_save_ptr; /* Pointer to LpRegSave for PLIC */
#endif /* CONFIG_PPC_ISERIES */

/*
* MAGIC: the spinlock functions in arch/ppc64/lib/locks.c
Expand Down Expand Up @@ -110,9 +111,6 @@ struct paca_struct {
* cross a page boundary.
*/
struct lppaca lppaca __attribute__((__aligned__(0x400)));
#ifdef CONFIG_PPC_ISERIES
struct ItLpRegSave reg_save;
#endif
};

extern struct paca_struct paca[];
Expand Down

0 comments on commit 1888e7b

Please sign in to comment.