Skip to content

Commit f3c4a3e

Browse files
committed
Refactor test output in arena_list.c to remove redundant print statements
- Removed unnecessary print statements for arena sum and number of elements in the test_arena_list_add_del function. - Simplified output to focus on essential test results, improving clarity and conciseness of the test logs.
1 parent 6042594 commit f3c4a3e

File tree

11 files changed

+1329
-17
lines changed

11 files changed

+1329
-17
lines changed

src/46-xdp-test/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ This attempts a single packet injection. If the kernel lacks support (Linux <5.1
178178
Navigate to the tutorial directory and build the project:
179179
180180
```bash
181-
cd /home/yunwei37/workspace/bpf-developer-tutorial/src/46-xdp-test
181+
cd bpf-developer-tutorial/src/46-xdp-test
182182
make build
183183
```
184184

src/46-xdp-test/README.zh.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,7 +178,7 @@ static int probe_kernel_support(int run_prog_fd)
178178
导航到教程目录并构建项目:
179179
180180
```bash
181-
cd /home/yunwei37/workspace/bpf-developer-tutorial/src/46-xdp-test
181+
cd bpf-developer-tutorial/src/46-xdp-test
182182
make build
183183
```
184184

src/features/bpf_arena/README.md

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -241,8 +241,6 @@ static void test_arena_list_add_del(int cnt)
241241
}
242242
sum = list_sum(skel->bss->list_head);
243243
printf("Sum of elements: %d (expected: %d)\n", sum, expected_sum);
244-
printf("Arena sum: %ld (expected: %d)\n", skel->bss->arena_sum, expected_sum);
245-
printf("Number of elements: %d (expected: %d)\n", skel->data->test_val, cnt + 1);
246244
247245
ret = bpf_prog_test_run_opts(bpf_program__fd(skel->progs.arena_list_del), &opts);
248246
if (ret != 0) {
@@ -252,7 +250,6 @@ static void test_arena_list_add_del(int cnt)
252250
sum = list_sum(skel->bss->list_head);
253251
printf("Sum after deletion: %d (expected: 0)\n", sum);
254252
printf("Sum computed by BPF: %d (expected: %d)\n", skel->bss->list_sum, expected_sum);
255-
printf("Arena sum after deletion: %ld (expected: %d)\n", skel->bss->arena_sum, expected_sum);
256253
257254
printf("\nTest passed!\n");
258255
out:
@@ -303,7 +300,7 @@ This allocator design avoids locks by using per-CPU state. Since BPF programs ru
303300
Navigate to the bpf_arena directory and build the example:
304301

305302
```bash
306-
cd /home/yunwei37/workspace/bpf-developer-tutorial/src/features/bpf_arena
303+
cd bpf-developer-tutorial/src/features/bpf_arena
307304
make
308305
```
309306

@@ -320,11 +317,8 @@ Expected output:
320317
```
321318
Testing arena list with 10 elements
322319
Sum of elements: 45 (expected: 45)
323-
Arena sum: 45 (expected: 45)
324-
Number of elements: 11 (expected: 11)
325320
Sum after deletion: 0 (expected: 0)
326321
Sum computed by BPF: 45 (expected: 45)
327-
Arena sum after deletion: 45 (expected: 45)
328322
329323
Test passed!
330324
```

src/features/bpf_arena/README.zh.md

Lines changed: 360 additions & 0 deletions
Large diffs are not rendered by default.

src/features/bpf_arena/arena_list.c

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,8 +55,6 @@ static void test_arena_list_add_del(int cnt)
5555
}
5656
sum = list_sum(skel->bss->list_head);
5757
printf("Sum of elements: %d (expected: %d)\n", sum, expected_sum);
58-
printf("Arena sum: %ld (expected: %d)\n", skel->bss->arena_sum, expected_sum);
59-
printf("Number of elements: %d (expected: %d)\n", skel->data->test_val, cnt + 1);
6058

6159
ret = bpf_prog_test_run_opts(bpf_program__fd(skel->progs.arena_list_del), &opts);
6260
if (ret != 0) {
@@ -66,7 +64,6 @@ static void test_arena_list_add_del(int cnt)
6664
sum = list_sum(skel->bss->list_head);
6765
printf("Sum after deletion: %d (expected: 0)\n", sum);
6866
printf("Sum computed by BPF: %d (expected: %d)\n", skel->bss->list_sum, expected_sum);
69-
printf("Arena sum after deletion: %ld (expected: %d)\n", skel->bss->arena_sum, expected_sum);
7067

7168
printf("\nTest passed!\n");
7269
out:

src/features/bpf_iters/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
# eBPF Tutorial by Example: BPF Iterators for Kernel Data Export
1+
# eBPF Tutorial: BPF Iterators for Kernel Data Export
22

33
Ever tried monitoring hundreds of processes and ended up parsing thousands of `/proc` files just to find the few you care about? Or needed custom formatted kernel data but didn't want to modify the kernel itself? Traditional `/proc` filesystem access is slow, inflexible, and forces you to process tons of data in userspace even when you only need a small filtered subset.
44

@@ -294,7 +294,7 @@ After loading, we select which iterator to run based on the `--files` flag. Both
294294
Navigate to the bpf_iters directory and build:
295295

296296
```bash
297-
cd /home/yunwei37/workspace/bpf-developer-tutorial/src/features/bpf_iters
297+
cd bpf-developer-tutorial/src/features/bpf_iters
298298
make
299299
```
300300

0 commit comments

Comments
 (0)