Mercurial > hg > nginx-quic
annotate src/os/unix/ngx_sunpro_amd64.il @ 6272:b6a665bf858a
HTTP/2: fix indirect reprioritization.
Previously, streams that were indirectly reprioritized (either because of
a new exclusive dependency on their parent or because of removal of their
parent from the dependency tree), didn't have their pointer to the parent
node updated.
This broke detection of circular dependencies and, as a result, nginx
worker would crash due to stack overflow whenever such dependency was
introduced.
Found with afl-fuzz.
Signed-off-by: Piotr Sikora <piotrsikora@google.com>
author | Piotr Sikora <piotrsikora@google.com> |
---|---|
date | Thu, 01 Oct 2015 20:25:55 -0700 |
parents | d620f497c50f |
children | 2cd019520210 |
rev | line source |
---|---|
561 | 1 / |
2 / Copyright (C) Igor Sysoev | |
4412 | 3 / Copyright (C) Nginx, Inc. |
561 | 4 / |
5 | |
6 / ngx_atomic_uint_t ngx_atomic_cmp_set(ngx_atomic_t *lock, | |
7 / ngx_atomic_uint_t old, ngx_atomic_uint_t set); | |
8 / | |
9 / the arguments are passed in %rdi, %rsi, %rdx | |
10 / the result is returned in the %rax | |
11 | |
12 .inline ngx_atomic_cmp_set,0 | |
13 movq %rsi, %rax | |
14 lock | |
15 cmpxchgq %rdx, (%rdi) | |
16 setz %al | |
17 movzbq %al, %rax | |
18 .end | |
19 | |
20 | |
21 / ngx_atomic_int_t ngx_atomic_fetch_add(ngx_atomic_t *value, | |
22 / ngx_atomic_int_t add); | |
23 / | |
24 / the arguments are passed in %rdi, %rsi | |
25 / the result is returned in the %rax | |
26 | |
27 .inline ngx_atomic_fetch_add,0 | |
28 movq %rsi, %rax | |
29 lock | |
30 xaddq %rax, (%rdi) | |
31 .end | |
1309 | 32 |
33 | |
34 / ngx_cpu_pause() | |
3479
bfd1912e55a9
use "rep; nop" instead of "pause" on Solaris/amd64
Igor Sysoev <igor@sysoev.ru>
parents:
1309
diff
changeset
|
35 / |
bfd1912e55a9
use "rep; nop" instead of "pause" on Solaris/amd64
Igor Sysoev <igor@sysoev.ru>
parents:
1309
diff
changeset
|
36 / the "rep; nop" is used instead of "pause" to avoid the "[ PAUSE ]" hardware |
bfd1912e55a9
use "rep; nop" instead of "pause" on Solaris/amd64
Igor Sysoev <igor@sysoev.ru>
parents:
1309
diff
changeset
|
37 / capability added by linker because Solaris/amd64 does not know about it: |
bfd1912e55a9
use "rep; nop" instead of "pause" on Solaris/amd64
Igor Sysoev <igor@sysoev.ru>
parents:
1309
diff
changeset
|
38 / |
bfd1912e55a9
use "rep; nop" instead of "pause" on Solaris/amd64
Igor Sysoev <igor@sysoev.ru>
parents:
1309
diff
changeset
|
39 / ld.so.1: nginx: fatal: hardware capability unsupported: 0x2000 [ PAUSE ] |
1309 | 40 |
41 .inline ngx_cpu_pause,0 | |
3479
bfd1912e55a9
use "rep; nop" instead of "pause" on Solaris/amd64
Igor Sysoev <igor@sysoev.ru>
parents:
1309
diff
changeset
|
42 rep; nop |
1309 | 43 .end |