07 Dec, 2011
7 commits
-
Don't open-code the OpenFirmware "dcr-reg" property lookup trying to map
DCR resources. This makes the code a bit easier to read.Signed-off-by: Kyle Moffett
Signed-off-by: Benjamin Herrenschmidt -
It turns out that there are only 2 in-tree platforms which use MPICs
which are not "primary": IBM Cell and PowerMac. To reduce the
complexity of the typical board setup code, invert the MPIC_PRIMARY bit
into MPIC_SECONDARY.Signed-off-by: Kyle Moffett
Signed-off-by: Benjamin Herrenschmidt -
Almost all PowerPC platforms use a standard "open-pic" device node so
the mpic_alloc() function now accepts NULL for the device-node. This
will cause it to perform a default search with of_find_matching_node().Signed-off-by: Kyle Moffett
Signed-off-by: Benjamin Herrenschmidt -
The MPIC code can already perform an automatic OF address translation
step as part of mpic_alloc(), but several boards need to use that base
address when they perform mpic_assign_isu().The easiest solution is to save the computed physical address into the
"struct mpic" for later use by the board code.Signed-off-by: Kyle Moffett
Signed-off-by: Benjamin Herrenschmidt -
All of the existing callers of mpic_alloc() pass in a non-NULL
device-node pointer, so the checks for a NULL device-node may be
removed.Signed-off-by: Kyle Moffett
Signed-off-by: Benjamin Herrenschmidt -
Instead of using the open-coded "reg" property lookup and address
translation in mpic_alloc(), directly call of_address_to_resource().
This includes various workarounds for special cases which the naive
of_address_translate() does not.Afterwards it is possible to remove the copiously copy-pasted calls to
of_address_translate() from the 85xx/86xx/powermac platforms.Signed-off-by: Kyle Moffett
Cc: Benjamin Herrenschmidt
Cc: Paul Mackerras
Cc: Grant Likely
Cc: Kumar Gala
Signed-off-by: Benjamin Herrenschmidt -
This removes a bunch of "extern" declarations and CONFIG_SMP ifdefs.
Signed-off-by: Kyle Moffett
Cc: Benjamin Herrenschmidt
Cc: Paul Mackerras
Cc: Kumar Gala
Signed-off-by: Benjamin Herrenschmidt
28 Nov, 2011
6 commits
-
Since commit 8a0a9bd4db63bc45e301, this comment in mmap_rnd() does not
hold true as the value returned by get_random_int() will in fact bedifferent every single call. Remove the comment and simplify the code
back to its original desired form.This reverts commit a5adc91a4b44b5d1 which is no longer necessary and
also fixes the sparc code that copied this same adjustment.Signed-off-by: Dan McGee
Acked-by: David S. Miller
Signed-off-by: Benjamin Herrenschmidt -
During kdump stress testing I sometimes see the kdump kernel panic
with:Interrupt 0x306 (real) is invalid, disabling it.
Kernel panic - not syncing: bad return code EOI - rc = -4, value=ff000306Instead of panicing print the error message, dump the stack the first
time it happens and continue on. Add some more information to the
debug messages as well.Signed-off-by: Anton Blanchard
Signed-off-by: Benjamin Herrenschmidt -
On a 64bit book3s machine I have an oops from a system reset that
claims the book3e CE bit was set:MSR: 8000000000021032 CR: 24004082 XER: 00000010
On a book3s machine system reset sets IBM bit 46 and 47 depending on
the power saving mode. Separate the definitions by type and for
completeness add the rest of the bits in.Signed-off-by: Anton Blanchard
Signed-off-by: Benjamin Herrenschmidt -
In lieu of having multiple similiar lines, we can just have one
generic cpu-as line for CONFIG_ALTIVECSigned-off-by: Benjamin Herrenschmidt
-
CPC925/CPC945 use special window to access host bridge functionality of
u3-ht. Provide a way to access this device.Signed-off-by: Dmitry Eremin-Solenikov
Signed-off-by: Benjamin Herrenschmidt -
As described in the help text in the patch, this token restricts general
access to /dev/mem as a way of increasing the security. Specifically, access
to exclusive IOMEM and kernel RAM is denied unless CONFIG_STRICT_DEVMEM is
set to 'n'.Implement the 'devmem_is_allowed()' interface for Powerpc. It will be
called from range_is_allowed() when userpsace attempts to access /dev/mem.This patch is based on an earlier patch from Steve Best and with input from
Paul Mackerras and Scott Wood.[BenH] Fixed a typo or two and removed the generic change which should
be submitted as a separate patchSigned-off-by: Sukadev Bhattiprolu
Signed-off-by: Benjamin Herrenschmidt
25 Nov, 2011
27 commits
-
The patch below removes an extra semicolon.
Signed-off-by: Justin P. Mattock
CC: linuxppc-dev@lists.ozlabs.org
CC: Paul Mackerras
Signed-off-by: Benjamin Herrenschmidt -
On Fri, Nov 11, 2011 at 10:17:55AM +0530, Ananth N Mavinakayanahalli wrote:
> >
> > At this rate we're going to end up with no bits left for CPU features
> > way too quickly... Especially for something we only care about once at
> > boot time.
> >
> > Wouldn't CPU_FTR_PPCAS_ARCH_V2 be a good enough test ?
>
> /me checks Cell manuals... yes, that test would be good enough. I will
> cook up a patch to use this.Here it is...
Signed-off-by: Benjamin Herrenschmidt
-
This adds support for p7IOC (and possibly other IODA v1 IO Hubs)
using OPAL v2 interfaces.We completely take over resource assignment and assign them using an
algorithm that hands out device BARs in a way that makes them fit in
individual segments of the M32 window of the bridge, which enables us
to assign individual PEs to devices and functions.The current implementation gives out a PE per functions on PCIe, and a
PE for the entire bridge for PCIe to PCI-X bridges.This can be adjusted / fine tuned later.
We also setup DMA resources (32-bit only for now) and MSIs (both 32-bit
and 64-bit MSI are supported).The DMA allocation tries to divide the available 256M segments of the
32-bit DMA address space "fairly" among PEs. This is done using a
"weight" heuristic which assigns less value to things like OHCI USB
controllers than, for example SCSI RAID controllers. This algorithm
will probably want some fine tuning for specific devices or device
types.Signed-off-by: Benjamin Herrenschmidt
-
It advertises "host bridge" instead of "PCI to PCI bridge" which confuses
the Linux probe code.Signed-off-by: Benjamin Herrenschmidt
-
This is used for newer IO Hubs such as p7IOC.
Signed-off-by: Benjamin Herrenschmidt
-
When PCI_REASSIGN_ALL_RSRC is set, we used to clear all bus resources
at the beginning of survey and re-allocate them later.This changes it so instead, during early fixup, we mark all resources
as IORESOURCE_UNSET and move them down to be 0-based.Later, if bus resources are still unset at the beginning of the survey,
then we clear them.This shouldn't impact the re-assignment case on 4xx, but will enable
us to have the platform do some custom resource assignment before the
survey, by clearing individual resources IORESOURCE_UNSET bit.Also limits the clutter in the kernel log from fixup when re-assigning
since we don't care about the offset applied to the BAR values in this
case.Signed-off-by: Benjamin Herrenschmidt
-
Some platforms need to perform resource allocation using a custom algorithm
due to HW constraints, or may want to tweak things globally below a host
bridge. For example OPAL support for IODA will need to perform a
resource allocation pass that applies IODA specific segmentation
constraints to MMIO which cannot be done simply using the kernel generic
resource management code.Signed-off-by: Benjamin Herrenschmidt
-
This adds a pgprot combination required by some cache-enabled IO device
mappings, such as Freescale datapath (QMan and BMan) portals.Signed-off-by: Geoff Thorpe
Signed-off-by: Benjamin Herrenschmidt -
All interrupts which must be non threaded are marked
IRQF_NO_THREAD. So it's safe to allow force threaded handlers.Signed-off-by: Thomas Gleixner
Signed-off-by: Benjamin Herrenschmidt -
IPI handlers cannot be threaded. Remove the obsolete IRQF_DISABLED
flag (see commit e58aa3d2) while at it.Signed-off-by: Thomas Gleixner
Signed-off-by: Benjamin Herrenschmidt -
Cascade handlers must run in hard interrupt context.
Signed-off-by: Thomas Gleixner
Signed-off-by: Benjamin Herrenschmidt -
Cascade interrupt must run in hard interrupt context.
Signed-off-by: Thomas Gleixner
Signed-off-by: Benjamin Herrenschmidt -
The RTAS firmware flash update is conducted using an RTAS call that is
serialized by lock_rtas() which uses spin_lock. While the flash is in
progress, rtasd performs scan for any RTAS events that are generated by
the system. rtasd keeps scanning for the RTAS events generated on the
machine. This is performed via workqueue mechanism. The rtas_event_scan()
also uses an RTAS call to scan the events, eventually trying to acquire
the spin_lock before issuing the request.The flash update takes a while to complete and during this time, any other
RTAS call has to wait. In this case, rtas_event_scan() waits for a long time
on the spin_lock resulting in a soft lockup.Fix: Just before the flash update is performed, the queued rtas_event_scan()
work item is cancelled from the work queue so that there is no other RTAS
call issued while the flash is in progress. After the flash completes, the
system reboots and the rtas_event_scan() is rescheduled.Signed-off-by: Suzuki Poulose
Signed-off-by: Ravi Nittala
Reported-by: Divya Vikas
Signed-off-by: Benjamin Herrenschmidt -
This patch add the Chroma platform to WSP/PowerEN, which is a PCIe
card (a defconfig is included).The card includes an H8 service processor that is used to manage the
card. The H8 is connected over the second serial UART on the PowerEN
chip so this patch includes a simple 16550 driver to enable
communication, mostly for "power off" and "rebooting".This patch also includes a, WSP specific, "halt" method that will shut
of all A2 cores but still leave power on at the chip level. This is
desirable, especially if you wish to interrogate the chip with a
hardware probe after the halt.Signed-off-by: Jimi Xenidis
Signed-off-by: Benjamin Herrenschmidt -
Sorry, there was a typo in the #if
signed-off-by: Jimi Xenidis
Signed-off-by: Benjamin Herrenschmidt
-
The 'u' command will print the TLB on book3e parts and the SLB on
Book3s parts, but the help system doesn't say that correctly.Signed-off-by: Jimi Xenidis
Signed-off-by: Benjamin Herrenschmidt -
This patch adds a fault handler that responds to illegal Coprocessor
types. Currently all CTs are treated and illegal. There are two ways
to report the fault back to the application. If the application used
the record form ("icswx.") then the architected "reject" is emulated.
If the application did not used the record form ("icswx") then it is
selectable by config whether the failure is silent (as architected) or
a SIGILL is generated.In all cases pr_warn() is used to log the bad CT.
Signed-off-by: Jimi Xenidis
Signed-off-by: Benjamin Herrenschmidt -
ICSWX is also used by the A2 processor to access coprocessors,
although not all "chips" that contain A2s have coprocessors.Signed-off-by: Jimi Xenidis
Signed-off-by: Benjamin Herrenschmidt -
Some processors, like embedded, that already have a PID register that
is managed by the system. This patch separates the ACOP and PID
processing into separate files so that the ACOP code can be shared.Signed-off-by: Jimi Xenidis
Signed-off-by: Benjamin Herrenschmidt -
libio.h is not provided by uClibc, in order to be able to test the
definition of __UCLIBC__ we need to include stdlib.h, which also
includes stddef.h, providing the definition of 'NULL'.Signed-off-by: Florian Fainelli
Signed-off-by: Benjamin Herrenschmidt -
np is initialized to the result of calling a function that calls
of_node_get, so of_node_put should be called before the pointer is dropped.The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)//
@@
expression e,e1,e2;
@@* e = \(of_find_node_by_type\|of_find_node_by_name\)(...)
... when != of_node_put(e)
when != true e == NULL
when != e2 = e
e = e1
//Signed-off-by: Julia Lawall
Signed-off-by: Benjamin Herrenschmidt -
np is initialized to the result of calling a function that calls
of_node_get, so of_node_put should be called before the pointer is dropped.The semantic match that finds this problem is as follows:
(http://coccinelle.lip6.fr/)//
@@
expression e,e1,e2;
@@* e = \(of_find_node_by_type\|of_find_node_by_name\)(...)
... when != of_node_put(e)
when != true e == NULL
when != e2 = e
e = e1
//Signed-off-by: Julia Lawall
Acked-by: Arnd Bergmann
Signed-off-by: Benjamin Herrenschmidt -
Some pseries IOMMUs cache TCEs but don't snoop when the TCEs are changed
in memory, hence we need manually invalidate in software.This adds code to do the invalidate. It keys off a device tree property
to say where the to do the MMIO for the invalidate and some information
on what the format of the invalidate including some magic routing info.it_busno get overloaded with this magic routing info and it_index with
the MMIO address for the invalidate command.This then gets hooked into the building and freeing of TCEs.
This is only useful on bare metal pseries. pHyp takes care of this when
virtualised.Based on patch from Milton with cleanups from Mikey.
Signed-off-by: Milton Miller
Signed-off-by: Michael Neuling
Signed-off-by: Benjamin Herrenschmidt -
decrementer_check_overflow is called from arch_local_irq_restore so
we want to make it as light weight as possible. As such, turn
decrementer_check_overflow into an inline function.To avoid a circular mess of includes, separate out the two components
of struct decrementer_clock and keep the struct clock_event_device
part local to time.c.The fast path improves from:
arch_local_irq_restore
0: mflr r0
4: std r0,16(r1)
8: stdu r1,-112(r1)
c: stb r3,578(r13)
10: cmpdi cr7,r3,0
14: beq- cr7,24
...
24: addi r1,r1,112
28: ld r0,16(r1)
2c: mtlr r0
30: blrto:
arch_local_irq_restore
0: std r30,-16(r1)
4: ld r30,0(r2)
8: stb r3,578(r13)
c: cmpdi cr7,r3,0
10: beq- cr7,6c
...
6c: ld r30,-16(r1)
70: blrUnfortunately we still setup a local TOC (due to -mminimal-toc). Yet
another sign we should be moving to -mcmodel=medium.Signed-off-by: Anton Blanchard
Signed-off-by: Benjamin Herrenschmidt -
Fix some formatting issues and use the DECREMENTER_MAX
define instead of 0x7fffffff.Signed-off-by: Anton Blanchard
Signed-off-by: Benjamin Herrenschmidt -
The clockevents code uses max_delta_ns to avoid calling a
clockevent with too large a value.Remove the redundant version of this in the timer_interrupt
code.Signed-off-by: Anton Blanchard
Signed-off-by: Benjamin Herrenschmidt