mirror of
https://github.com/openwrt/openwrt.git
synced 2024-12-22 06:57:57 +00:00
kernel: fix MIPS highmem breakage
Signed-off-by: Felix Fietkau <nbd@openwrt.org> SVN-Revision: 48468
This commit is contained in:
parent
be5fcf9fd7
commit
e756c2bb07
@ -0,0 +1,95 @@
|
||||
From: Felix Fietkau <nbd@openwrt.org>
|
||||
Date: Sun, 24 Jan 2016 01:03:51 +0100
|
||||
Subject: [PATCH] MIPS: fix cache flushing for highmem pages
|
||||
|
||||
Most cache flush ops were no-op for highmem pages. This led to nasty
|
||||
segfaults and (in the case of page_address(page) == NULL) kernel
|
||||
crashes.
|
||||
|
||||
Fix this by always flushing highmem pages using kmap/kunmap_atomic
|
||||
around the actual cache flush. This might be a bit inefficient, but at
|
||||
least it's stable.
|
||||
|
||||
Signed-off-by: Felix Fietkau <nbd@openwrt.org>
|
||||
---
|
||||
|
||||
--- a/arch/mips/mm/cache.c
|
||||
+++ b/arch/mips/mm/cache.c
|
||||
@@ -14,6 +14,7 @@
|
||||
#include <linux/sched.h>
|
||||
#include <linux/syscalls.h>
|
||||
#include <linux/mm.h>
|
||||
+#include <linux/highmem.h>
|
||||
|
||||
#include <asm/cacheflush.h>
|
||||
#include <asm/processor.h>
|
||||
@@ -78,18 +79,29 @@ SYSCALL_DEFINE3(cacheflush, unsigned lon
|
||||
return 0;
|
||||
}
|
||||
|
||||
+static void
|
||||
+flush_highmem_page(struct page *page)
|
||||
+{
|
||||
+ void *addr = kmap_atomic(page);
|
||||
+ flush_data_cache_page((unsigned long)addr);
|
||||
+ kunmap_atomic(addr);
|
||||
+}
|
||||
+
|
||||
void __flush_dcache_page(struct page *page)
|
||||
{
|
||||
struct address_space *mapping = page_mapping(page);
|
||||
unsigned long addr;
|
||||
|
||||
- if (PageHighMem(page))
|
||||
- return;
|
||||
if (mapping && !mapping_mapped(mapping)) {
|
||||
SetPageDcacheDirty(page);
|
||||
return;
|
||||
}
|
||||
|
||||
+ if (PageHighMem(page)) {
|
||||
+ flush_highmem_page(page);
|
||||
+ return;
|
||||
+ }
|
||||
+
|
||||
/*
|
||||
* We could delay the flush for the !page_mapping case too. But that
|
||||
* case is for exec env/arg pages and those are %99 certainly going to
|
||||
@@ -105,6 +117,11 @@ void __flush_anon_page(struct page *page
|
||||
{
|
||||
unsigned long addr = (unsigned long) page_address(page);
|
||||
|
||||
+ if (PageHighMem(page)) {
|
||||
+ flush_highmem_page(page);
|
||||
+ return;
|
||||
+ }
|
||||
+
|
||||
if (pages_do_alias(addr, vmaddr)) {
|
||||
if (page_mapped(page) && !Page_dcache_dirty(page)) {
|
||||
void *kaddr;
|
||||
@@ -123,8 +140,10 @@ void __flush_icache_page(struct vm_area_
|
||||
{
|
||||
unsigned long addr;
|
||||
|
||||
- if (PageHighMem(page))
|
||||
+ if (PageHighMem(page)) {
|
||||
+ flush_highmem_page(page);
|
||||
return;
|
||||
+ }
|
||||
|
||||
addr = (unsigned long) page_address(page);
|
||||
flush_data_cache_page(addr);
|
||||
@@ -142,7 +161,12 @@ void __update_cache(struct vm_area_struc
|
||||
if (unlikely(!pfn_valid(pfn)))
|
||||
return;
|
||||
page = pfn_to_page(pfn);
|
||||
- if (page_mapping(page) && Page_dcache_dirty(page)) {
|
||||
+ if (!Page_dcache_dirty(page))
|
||||
+ return;
|
||||
+
|
||||
+ if (PageHighMem(page)) {
|
||||
+ flush_highmem_page(page);
|
||||
+ } else if (page_mapping(page)) {
|
||||
addr = (unsigned long) page_address(page);
|
||||
if (exec || pages_do_alias(addr, address & PAGE_MASK))
|
||||
flush_data_cache_page(addr);
|
@ -0,0 +1,95 @@
|
||||
From: Felix Fietkau <nbd@openwrt.org>
|
||||
Date: Sun, 24 Jan 2016 01:03:51 +0100
|
||||
Subject: [PATCH] MIPS: fix cache flushing for highmem pages
|
||||
|
||||
Most cache flush ops were no-op for highmem pages. This led to nasty
|
||||
segfaults and (in the case of page_address(page) == NULL) kernel
|
||||
crashes.
|
||||
|
||||
Fix this by always flushing highmem pages using kmap/kunmap_atomic
|
||||
around the actual cache flush. This might be a bit inefficient, but at
|
||||
least it's stable.
|
||||
|
||||
Signed-off-by: Felix Fietkau <nbd@openwrt.org>
|
||||
---
|
||||
|
||||
--- a/arch/mips/mm/cache.c
|
||||
+++ b/arch/mips/mm/cache.c
|
||||
@@ -14,6 +14,7 @@
|
||||
#include <linux/sched.h>
|
||||
#include <linux/syscalls.h>
|
||||
#include <linux/mm.h>
|
||||
+#include <linux/highmem.h>
|
||||
|
||||
#include <asm/cacheflush.h>
|
||||
#include <asm/processor.h>
|
||||
@@ -78,18 +79,29 @@ SYSCALL_DEFINE3(cacheflush, unsigned lon
|
||||
return 0;
|
||||
}
|
||||
|
||||
+static void
|
||||
+flush_highmem_page(struct page *page)
|
||||
+{
|
||||
+ void *addr = kmap_atomic(page);
|
||||
+ flush_data_cache_page((unsigned long)addr);
|
||||
+ kunmap_atomic(addr);
|
||||
+}
|
||||
+
|
||||
void __flush_dcache_page(struct page *page)
|
||||
{
|
||||
struct address_space *mapping = page_mapping(page);
|
||||
unsigned long addr;
|
||||
|
||||
- if (PageHighMem(page))
|
||||
- return;
|
||||
if (mapping && !mapping_mapped(mapping)) {
|
||||
SetPageDcacheDirty(page);
|
||||
return;
|
||||
}
|
||||
|
||||
+ if (PageHighMem(page)) {
|
||||
+ flush_highmem_page(page);
|
||||
+ return;
|
||||
+ }
|
||||
+
|
||||
/*
|
||||
* We could delay the flush for the !page_mapping case too. But that
|
||||
* case is for exec env/arg pages and those are %99 certainly going to
|
||||
@@ -105,6 +117,11 @@ void __flush_anon_page(struct page *page
|
||||
{
|
||||
unsigned long addr = (unsigned long) page_address(page);
|
||||
|
||||
+ if (PageHighMem(page)) {
|
||||
+ flush_highmem_page(page);
|
||||
+ return;
|
||||
+ }
|
||||
+
|
||||
if (pages_do_alias(addr, vmaddr)) {
|
||||
if (page_mapped(page) && !Page_dcache_dirty(page)) {
|
||||
void *kaddr;
|
||||
@@ -123,8 +140,10 @@ void __flush_icache_page(struct vm_area_
|
||||
{
|
||||
unsigned long addr;
|
||||
|
||||
- if (PageHighMem(page))
|
||||
+ if (PageHighMem(page)) {
|
||||
+ flush_highmem_page(page);
|
||||
return;
|
||||
+ }
|
||||
|
||||
addr = (unsigned long) page_address(page);
|
||||
flush_data_cache_page(addr);
|
||||
@@ -142,7 +161,12 @@ void __update_cache(struct vm_area_struc
|
||||
if (unlikely(!pfn_valid(pfn)))
|
||||
return;
|
||||
page = pfn_to_page(pfn);
|
||||
- if (page_mapping(page) && Page_dcache_dirty(page)) {
|
||||
+ if (!Page_dcache_dirty(page))
|
||||
+ return;
|
||||
+
|
||||
+ if (PageHighMem(page)) {
|
||||
+ flush_highmem_page(page);
|
||||
+ } else if (page_mapping(page)) {
|
||||
addr = (unsigned long) page_address(page);
|
||||
if (exec || pages_do_alias(addr, address & PAGE_MASK))
|
||||
flush_data_cache_page(addr);
|
Loading…
Reference in New Issue
Block a user