| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
| |
Fixes tautological-constant-out-of-range-compare warning.
Bug: 72331526
Test: presubmit
AOSP-Change: 1173604
Change-Id: Ie71f5a57fad298a9cb035b5e7430af617b438ab8
Merged-In: Ie71f5a57fad298a9cb035b5e7430af617b438ab8
(cherry picked from commit 7c0e9992d953850fd08c1f0f26cc3dcc24c61c4f)
|
| |\
| |
| |
| |
| |
| | |
9d9e852f62
Change-Id: I07e0c627a037913367035218d4a4746b0e57f92f
|
| | |\
| | |
| | |
| | | |
Change-Id: I46b2f09ed605e8d9c5960cd33a206ca880d448cf
|
| | |/|
| | |
| | |
| | | |
Change-Id: I847d80b9195ce5e43b4e2f880b3b01eaacdcf5fe
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
libapf is intended to be run on wireless chipsets so it must
never depend on Android platform internals. Set it to the SDK
version that was current when the interpreter was written.
Test: m libapf
Bug: 148753875
Change-Id: Ib20aa28f37f344032b0cc4fc104bda6a3b5a5919
|
| |\| |
| | |
| | |
| | |
| | |
| | | |
am: e5187d1a68
Change-Id: If8fd164050e13380f9925fda353debfe65e3f30b
|
| | |\|
| | |
| | |
| | |
| | |
| | | |
am: 1f847f05d8
Change-Id: I00c4097fd4df98d95263ba6141dc2f819172b69e
|
| | | |\
| | | |
| | | |
| | | |
| | | |
| | | | |
am: cdb38a5e3e
Change-Id: Id6afb62561ed5fb8cca7fc01ed78e4e8140b14ac
|
| | | |/
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
The change to print disassembled code in apf_run (aosp/1129074)
broke the output of apf_run by printing PC, R0, PC instead of
R0, R1, PC.
This was noticed because compiling the code with -O0 (which
happened when I put apf_run.c into a test target) results in a
compiler error. Unfortunately the error goes away with the
default compiler settings and -Werror=format-extra-args does not
help.
Test: existing tests were incorrect, modified
Change-Id: Iad1713ce253b5df62ad5cd42b6ddff28559dda8c
|
| |\| |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
am: 33a1edf8ec
am: 9d5cf97e14
Change-Id: Ie772087162a9be5fa2fcdbb8dc3973ba67ee21a6
|
| | |\ \
| | | |
| | | |
| | | |
| | | |
| | | | |
am: 33a1edf8ec
Change-Id: I0b08c923425a1a809dec79d47e7ee82863193708
|
| | | |\|
| | | |
| | | |
| | | |
| | | |
| | | | |
am: fed006431b
Change-Id: Ifca1a1f8f95f83a02128f500c122332ac5eda551
|
| | | | |\
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: 74ede83c56
Change-Id: Ic3962d1c8bad5f233c983e1718ba72a9ff454e38
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Currently, this will run NetworkStackTests (which run the
interpreter) and apf_run_test (which runs the disassembler).
Also disable apf_run_test on Windows because it's a shell script
that relies on bash.
Bug: 66928272
Test: treehugger
Change-Id: I0773887604a2c41e63e4c6c629ad81c3e6143bcc
|
| |\| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: 42383d6a21
Change-Id: I61caf48b26408d947591047a4ac20985a47509ea
|
| | |\| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: ad57e39f07
Change-Id: Ic83594bf763945c0f0f616bc9c0acb83605fc87a
|
| | | |\| |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: 37e5280ee8
Change-Id: I8f538f1b2534056b8507a70913c1652bc468e5c0
|
| | | | |\|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: 26dafcd858
Change-Id: Ib28de45a785965e930b92362b5b22a454a01f9b5
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
These tests cover both apf_run itself and the disassembler (and
of course the interpreter).
Test: atest apf_run_test
Change-Id: I622bb2bbd0ec48bdddb511963c374651705ff545
|
| |\| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: 687dc350f8
Change-Id: Ica93341d77ce0183751365357d1e74fb26b47813
|
| | |\| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: 18ce213d82
Change-Id: If1e9925d21ba40e674164f67cc4dfd1058e3d26f
|
| | | |\| |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: da6a0ef335
Change-Id: Idab03998dda6a8f99889d6a6892eba825a931862
|
| | | | |\|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: 983eb51423
Change-Id: I43418a911de2c0c0bf0f2ca56720aba3b974127e
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
apf_run is a great tool to debug APF programs, but it does not
currently print disassemled code. Fix this.
Before:
PC: 0 R0: 0 R1: 0
PC: 2 R0: 86dd R1: 0
PC: 7 R0: 86dd R1: 0
PC: 64 R0: 86dd R1: 0
PC: 141 R0: 86dd R1: 0
After:
R0 R1 PC Instruction
-------------------------------------------------
0 0 0: ldh r0, [12]
2 86dd 2: jlt r0, 0x600, drop
7 86dd 7: jne r0, 0x806, 64
40 86dd 64: jne r0, 0x800, 141
8d 86dd 141: jeq r0, 0x86dd, 161
...
Bug: 66928272
Test: manually compared output on a sample program
Change-Id: I1b93f70e0381d6e43d7f7d9785ab8c935baa1e4b
|
| |\| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: 377554ff83
Change-Id: If4e1ef9465257e94a1ea9cd32452c2d166a09721
|
| |/| | | |
| |/ / /
| | | |
| | | |
| | | |
| | | | |
am: 28176c5731
Change-Id: Ie7d5c581c4f9dab5750bb35f487fbce7c96b9e30
|
| |/| | |
| |/ /
| | |
| | |
| | |
| | | |
am: f8f0959556
Change-Id: I184e4a24c19ec7d3cfdc604899549c87305db471
|
| |/| |
| |/
| |
| |
| |
| | |
am: 90cde57df6
Change-Id: Ia4e4f9ccd4e482dfe74c4292c628a4470ad3a6c5
|
| |/
|
|
|
| |
Test: m apf_run apf_disassembler libapf
Change-Id: I43b975a6afa11f693268317de69967e7774a1c85
|
| |\
| |
| |
| |
| |
| |
| |
| | |
am: 3ae28a202f
am: b8a466afd4
Change-Id: Ie06a6071c83e0596a3d130053a0c3d9c8be017d8
|
| | |\
| | |
| | |
| | |
| | |
| | | |
am: 3ae28a202f
Change-Id: I7ac3ecc4b254f56fa98962f4c09b8f7c33b414a5
|
| | | |\
| | | |
| | | |
| | | |
| | | |
| | | | |
am: 6f73fbb3a0
Change-Id: Ib577d2959588e0e1cd6301fcf129d3347c1f3f6f
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This reverts commit 2af4ba42b5e3e9e16765979d169c7c6fe49e603d.
Reason for revert: APF uses a mix of 2-space and 4-space indentation,
and clang-format doesn't support this. Re-indenting the entire
codebase would interferes with 'git blame' and cause annoying merge
conflicts.
Change-Id: If77e4e2f189549cfd0635b52a32c89f14b8f99cf
|
| |\| | |
| | | |
| | | |
| | | |
| | | |
| | | | |
am: ee8232b1c0
Change-Id: I7bd1d151e9791c09ad2e77986b64f246823778e7
|
| | |\| |
| | | |
| | | |
| | | |
| | | | |
Bug: 126125118
Change-Id: I9ad60eb896816eb533b0c364d3cb34b23aee28e5
|
| | | |\ \
| | | | |
| | | | |
| | | | | |
stage-aosp-master
|
| | | | |\ \
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
Bug: 120848293
Change-Id: I5c91808a99e0412fd00de3b2a6ab0428417abd97
|
| |\| | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
39492ba12d
am: 98acc3d25c
Change-Id: Iea9148ec580b54b28736d2ac7583caa90e122252
|
| | |\| | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
am: 39492ba12d
Change-Id: I03bde339794a63f5625687381fbbf2d01d32ade7
|
| | | |\ \ \ \
| | | |/ / /
| | |/| | /
| | | | |/
| | | |/|
| | | | | |
am: 2af4ba42b5
Change-Id: I62ba263cd1c12ed1ae678c361c4086175f30f2b6
|
| | | | | |
| | | | |
| | | | |
| | | | | |
Change-Id: Ia98bf4f7e030541e1062effb8ba2ab8fd6e6050e
|
| |\| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: 87a315aff4
Change-Id: I1a7f254bf42a8b4939fbbf39e946cf862f1bccc4
|
| | |\| | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: adf9750f28
Change-Id: I898b468a2ed6bde1c8546bd74d7f091de01fc606
|
| | | |\| |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: d0174a67d8
Change-Id: I2e879d7f42469f98efaa7ba6aadb63fa8ddef268
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
This adds a --trace option to apf_run which dumps the PC and register
content during execution. This is a big aid in debugging misbehaving
APF programs.
It should be easy to disassemble the current instruction inline if we
refactor apf_disassembler to extract its body to a separate function.
Another easy feature would be dumping the memory locations that were
modified by the previous instruction.
Test: apf_run --trace --program <program> --packet <packet> --data <data>
Change-Id: I2a24bdfb84ce4e86409542332ee32082988a7698
|
| |\| | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
am: 71413c43c6
Change-Id: I1e500c6de4b07508041b70de924d13fe9b7bfeeb
|
| | |\| | |
| | |_|/
| |/| |
| | | |
| | | |
| | | | |
am: 49b916a860
Change-Id: Idb29fb501066048ead2e378969de2bc19edba7e3
|
| | | |\|
| | | |
| | | |
| | | |
| | | |
| | | | |
am: 5698836c72
Change-Id: I3ca8d27fd11721660bae27698367106d6488b86f
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Test: build/make/tools/checkowners.py hardware/google/apf/OWNERS
Change-Id: Ie6c815257523666740d0c648761083b357dd4cac
|
| |\| | |
| | | |
| | | |
| | | |
| | | |
| | | | |
am: f03e5d7819
Change-Id: Ie33ffe7d1a8f57c7d8c4cf0f1383a889e2748fb9
|