Skip to content

Commit 9bca7c4

Browse files
committed
Merge tag 's390-5.7-4' of git://git.kernel.org/pub/scm/linux/kernel/git/s390/linux
Pull s390 fixes from Vasily Gorbik: - Add missing R_390_JMP_SLOT relocation type in KASLR code. - Fix set_huge_pte_at for empty ptes issue which has been uncovered with arch page table helper tests. - Correct initrd location for kdump kernel. - Fix s390_mmio_read/write with MIO in PCI code. * tag 's390-5.7-4' of git://git.kernel.org/pub/scm/linux/kernel/git/s390/linux: s390/kaslr: add support for R_390_JMP_SLOT relocation type s390/mm: fix set_huge_pte_at() for empty ptes s390/kexec_file: fix initrd location for kdump kernel s390/pci: Fix s390_mmio_read/write with MIO
2 parents e644645 + 4c1cbcb commit 9bca7c4

File tree

5 files changed

+227
-8
lines changed

5 files changed

+227
-8
lines changed

arch/s390/include/asm/pci_io.h

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,10 @@
88
#include <linux/slab.h>
99
#include <asm/pci_insn.h>
1010

11+
/* I/O size constraints */
12+
#define ZPCI_MAX_READ_SIZE 8
13+
#define ZPCI_MAX_WRITE_SIZE 128
14+
1115
/* I/O Map */
1216
#define ZPCI_IOMAP_SHIFT 48
1317
#define ZPCI_IOMAP_ADDR_BASE 0x8000000000000000UL
@@ -140,7 +144,8 @@ static inline int zpci_memcpy_fromio(void *dst,
140144

141145
while (n > 0) {
142146
size = zpci_get_max_write_size((u64 __force) src,
143-
(u64) dst, n, 8);
147+
(u64) dst, n,
148+
ZPCI_MAX_READ_SIZE);
144149
rc = zpci_read_single(dst, src, size);
145150
if (rc)
146151
break;
@@ -161,7 +166,8 @@ static inline int zpci_memcpy_toio(volatile void __iomem *dst,
161166

162167
while (n > 0) {
163168
size = zpci_get_max_write_size((u64 __force) dst,
164-
(u64) src, n, 128);
169+
(u64) src, n,
170+
ZPCI_MAX_WRITE_SIZE);
165171
if (size > 8) /* main path */
166172
rc = zpci_write_block(dst, src, size);
167173
else

arch/s390/kernel/machine_kexec_file.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,7 @@ static int kexec_file_add_initrd(struct kimage *image,
151151
buf.mem += crashk_res.start;
152152
buf.memsz = buf.bufsz;
153153

154-
data->parm->initrd_start = buf.mem;
154+
data->parm->initrd_start = data->memsz;
155155
data->parm->initrd_size = buf.memsz;
156156
data->memsz += buf.memsz;
157157

arch/s390/kernel/machine_kexec_reloc.c

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ int arch_kexec_do_relocs(int r_type, void *loc, unsigned long val,
2828
break;
2929
case R_390_64: /* Direct 64 bit. */
3030
case R_390_GLOB_DAT:
31+
case R_390_JMP_SLOT:
3132
*(u64 *)loc = val;
3233
break;
3334
case R_390_PC16: /* PC relative 16 bit. */

arch/s390/mm/hugetlbpage.c

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -159,10 +159,13 @@ void set_huge_pte_at(struct mm_struct *mm, unsigned long addr,
159159
rste &= ~_SEGMENT_ENTRY_NOEXEC;
160160

161161
/* Set correct table type for 2G hugepages */
162-
if ((pte_val(*ptep) & _REGION_ENTRY_TYPE_MASK) == _REGION_ENTRY_TYPE_R3)
163-
rste |= _REGION_ENTRY_TYPE_R3 | _REGION3_ENTRY_LARGE;
164-
else
162+
if ((pte_val(*ptep) & _REGION_ENTRY_TYPE_MASK) == _REGION_ENTRY_TYPE_R3) {
163+
if (likely(pte_present(pte)))
164+
rste |= _REGION3_ENTRY_LARGE;
165+
rste |= _REGION_ENTRY_TYPE_R3;
166+
} else if (likely(pte_present(pte)))
165167
rste |= _SEGMENT_ENTRY_LARGE;
168+
166169
clear_huge_pte_skeys(mm, rste);
167170
pte_val(*ptep) = rste;
168171
}

arch/s390/pci/pci_mmio.c

Lines changed: 211 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,113 @@
1111
#include <linux/mm.h>
1212
#include <linux/errno.h>
1313
#include <linux/pci.h>
14+
#include <asm/pci_io.h>
15+
#include <asm/pci_debug.h>
16+
17+
static inline void zpci_err_mmio(u8 cc, u8 status, u64 offset)
18+
{
19+
struct {
20+
u64 offset;
21+
u8 cc;
22+
u8 status;
23+
} data = {offset, cc, status};
24+
25+
zpci_err_hex(&data, sizeof(data));
26+
}
27+
28+
static inline int __pcistb_mio_inuser(
29+
void __iomem *ioaddr, const void __user *src,
30+
u64 len, u8 *status)
31+
{
32+
int cc = -ENXIO;
33+
34+
asm volatile (
35+
" sacf 256\n"
36+
"0: .insn rsy,0xeb00000000d4,%[len],%[ioaddr],%[src]\n"
37+
"1: ipm %[cc]\n"
38+
" srl %[cc],28\n"
39+
"2: sacf 768\n"
40+
EX_TABLE(0b, 2b) EX_TABLE(1b, 2b)
41+
: [cc] "+d" (cc), [len] "+d" (len)
42+
: [ioaddr] "a" (ioaddr), [src] "Q" (*((u8 __force *)src))
43+
: "cc", "memory");
44+
*status = len >> 24 & 0xff;
45+
return cc;
46+
}
47+
48+
static inline int __pcistg_mio_inuser(
49+
void __iomem *ioaddr, const void __user *src,
50+
u64 ulen, u8 *status)
51+
{
52+
register u64 addr asm("2") = (u64 __force) ioaddr;
53+
register u64 len asm("3") = ulen;
54+
int cc = -ENXIO;
55+
u64 val = 0;
56+
u64 cnt = ulen;
57+
u8 tmp;
58+
59+
/*
60+
* copy 0 < @len <= 8 bytes from @src into the right most bytes of
61+
* a register, then store it to PCI at @ioaddr while in secondary
62+
* address space. pcistg then uses the user mappings.
63+
*/
64+
asm volatile (
65+
" sacf 256\n"
66+
"0: llgc %[tmp],0(%[src])\n"
67+
" sllg %[val],%[val],8\n"
68+
" aghi %[src],1\n"
69+
" ogr %[val],%[tmp]\n"
70+
" brctg %[cnt],0b\n"
71+
"1: .insn rre,0xb9d40000,%[val],%[ioaddr]\n"
72+
"2: ipm %[cc]\n"
73+
" srl %[cc],28\n"
74+
"3: sacf 768\n"
75+
EX_TABLE(0b, 3b) EX_TABLE(1b, 3b) EX_TABLE(2b, 3b)
76+
:
77+
[src] "+a" (src), [cnt] "+d" (cnt),
78+
[val] "+d" (val), [tmp] "=d" (tmp),
79+
[len] "+d" (len), [cc] "+d" (cc),
80+
[ioaddr] "+a" (addr)
81+
:: "cc", "memory");
82+
*status = len >> 24 & 0xff;
83+
84+
/* did we read everything from user memory? */
85+
if (!cc && cnt != 0)
86+
cc = -EFAULT;
87+
88+
return cc;
89+
}
90+
91+
static inline int __memcpy_toio_inuser(void __iomem *dst,
92+
const void __user *src, size_t n)
93+
{
94+
int size, rc = 0;
95+
u8 status = 0;
96+
mm_segment_t old_fs;
97+
98+
if (!src)
99+
return -EINVAL;
100+
101+
old_fs = enable_sacf_uaccess();
102+
while (n > 0) {
103+
size = zpci_get_max_write_size((u64 __force) dst,
104+
(u64 __force) src, n,
105+
ZPCI_MAX_WRITE_SIZE);
106+
if (size > 8) /* main path */
107+
rc = __pcistb_mio_inuser(dst, src, size, &status);
108+
else
109+
rc = __pcistg_mio_inuser(dst, src, size, &status);
110+
if (rc)
111+
break;
112+
src += size;
113+
dst += size;
114+
n -= size;
115+
}
116+
disable_sacf_uaccess(old_fs);
117+
if (rc)
118+
zpci_err_mmio(rc, status, (__force u64) dst);
119+
return rc;
120+
}
14121

15122
static long get_pfn(unsigned long user_addr, unsigned long access,
16123
unsigned long *pfn)
@@ -46,6 +153,20 @@ SYSCALL_DEFINE3(s390_pci_mmio_write, unsigned long, mmio_addr,
46153

47154
if (length <= 0 || PAGE_SIZE - (mmio_addr & ~PAGE_MASK) < length)
48155
return -EINVAL;
156+
157+
/*
158+
* Only support read access to MIO capable devices on a MIO enabled
159+
* system. Otherwise we would have to check for every address if it is
160+
* a special ZPCI_ADDR and we would have to do a get_pfn() which we
161+
* don't need for MIO capable devices.
162+
*/
163+
if (static_branch_likely(&have_mio)) {
164+
ret = __memcpy_toio_inuser((void __iomem *) mmio_addr,
165+
user_buffer,
166+
length);
167+
return ret;
168+
}
169+
49170
if (length > 64) {
50171
buf = kmalloc(length, GFP_KERNEL);
51172
if (!buf)
@@ -56,7 +177,8 @@ SYSCALL_DEFINE3(s390_pci_mmio_write, unsigned long, mmio_addr,
56177
ret = get_pfn(mmio_addr, VM_WRITE, &pfn);
57178
if (ret)
58179
goto out;
59-
io_addr = (void __iomem *)((pfn << PAGE_SHIFT) | (mmio_addr & ~PAGE_MASK));
180+
io_addr = (void __iomem *)((pfn << PAGE_SHIFT) |
181+
(mmio_addr & ~PAGE_MASK));
60182

61183
ret = -EFAULT;
62184
if ((unsigned long) io_addr < ZPCI_IOMAP_ADDR_BASE)
@@ -72,6 +194,78 @@ SYSCALL_DEFINE3(s390_pci_mmio_write, unsigned long, mmio_addr,
72194
return ret;
73195
}
74196

197+
static inline int __pcilg_mio_inuser(
198+
void __user *dst, const void __iomem *ioaddr,
199+
u64 ulen, u8 *status)
200+
{
201+
register u64 addr asm("2") = (u64 __force) ioaddr;
202+
register u64 len asm("3") = ulen;
203+
u64 cnt = ulen;
204+
int shift = ulen * 8;
205+
int cc = -ENXIO;
206+
u64 val, tmp;
207+
208+
/*
209+
* read 0 < @len <= 8 bytes from the PCI memory mapped at @ioaddr (in
210+
* user space) into a register using pcilg then store these bytes at
211+
* user address @dst
212+
*/
213+
asm volatile (
214+
" sacf 256\n"
215+
"0: .insn rre,0xb9d60000,%[val],%[ioaddr]\n"
216+
"1: ipm %[cc]\n"
217+
" srl %[cc],28\n"
218+
" ltr %[cc],%[cc]\n"
219+
" jne 4f\n"
220+
"2: ahi %[shift],-8\n"
221+
" srlg %[tmp],%[val],0(%[shift])\n"
222+
"3: stc %[tmp],0(%[dst])\n"
223+
" aghi %[dst],1\n"
224+
" brctg %[cnt],2b\n"
225+
"4: sacf 768\n"
226+
EX_TABLE(0b, 4b) EX_TABLE(1b, 4b) EX_TABLE(3b, 4b)
227+
:
228+
[cc] "+d" (cc), [val] "=d" (val), [len] "+d" (len),
229+
[dst] "+a" (dst), [cnt] "+d" (cnt), [tmp] "=d" (tmp),
230+
[shift] "+d" (shift)
231+
:
232+
[ioaddr] "a" (addr)
233+
: "cc", "memory");
234+
235+
/* did we write everything to the user space buffer? */
236+
if (!cc && cnt != 0)
237+
cc = -EFAULT;
238+
239+
*status = len >> 24 & 0xff;
240+
return cc;
241+
}
242+
243+
static inline int __memcpy_fromio_inuser(void __user *dst,
244+
const void __iomem *src,
245+
unsigned long n)
246+
{
247+
int size, rc = 0;
248+
u8 status;
249+
mm_segment_t old_fs;
250+
251+
old_fs = enable_sacf_uaccess();
252+
while (n > 0) {
253+
size = zpci_get_max_write_size((u64 __force) src,
254+
(u64 __force) dst, n,
255+
ZPCI_MAX_READ_SIZE);
256+
rc = __pcilg_mio_inuser(dst, src, size, &status);
257+
if (rc)
258+
break;
259+
src += size;
260+
dst += size;
261+
n -= size;
262+
}
263+
disable_sacf_uaccess(old_fs);
264+
if (rc)
265+
zpci_err_mmio(rc, status, (__force u64) dst);
266+
return rc;
267+
}
268+
75269
SYSCALL_DEFINE3(s390_pci_mmio_read, unsigned long, mmio_addr,
76270
void __user *, user_buffer, size_t, length)
77271
{
@@ -86,12 +280,27 @@ SYSCALL_DEFINE3(s390_pci_mmio_read, unsigned long, mmio_addr,
86280

87281
if (length <= 0 || PAGE_SIZE - (mmio_addr & ~PAGE_MASK) < length)
88282
return -EINVAL;
283+
284+
/*
285+
* Only support write access to MIO capable devices on a MIO enabled
286+
* system. Otherwise we would have to check for every address if it is
287+
* a special ZPCI_ADDR and we would have to do a get_pfn() which we
288+
* don't need for MIO capable devices.
289+
*/
290+
if (static_branch_likely(&have_mio)) {
291+
ret = __memcpy_fromio_inuser(
292+
user_buffer, (const void __iomem *)mmio_addr,
293+
length);
294+
return ret;
295+
}
296+
89297
if (length > 64) {
90298
buf = kmalloc(length, GFP_KERNEL);
91299
if (!buf)
92300
return -ENOMEM;
93-
} else
301+
} else {
94302
buf = local_buf;
303+
}
95304

96305
ret = get_pfn(mmio_addr, VM_READ, &pfn);
97306
if (ret)

0 commit comments

Comments
 (0)