Skip to content

Commit 0c11624

Browse files
authored
hdr_split: fix null slot frame addr (#685)
Signed-off-by: Frank Du <[email protected]>
1 parent 05396ad commit 0c11624

File tree

2 files changed

+16
-2
lines changed

2 files changed

+16
-2
lines changed

doc/experimental/header_split.md

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,22 @@ Header split is a hardware-based feature for RX memory copy offload. It can leve
66

77
## 2. Build DPDK with header split patch
88

9+
Note: please checkout DPDK to v23.03 as it is verified. It has known raw flow issues with DPDK v23.08 and v23.11.
10+
911
```bash
10-
git am patches/dpdk/23.07/hdr_split/0001-net-ice-support-hdr-split-mbuf-callback.patch
12+
cd dpdk
13+
# checkout v23.03 version
14+
git checkout v23.03
15+
git switch -c v23.03
16+
git am $imtl_source_code/patches/dpdk/23.03/*.patch
17+
git am $imtl_source_code/patches/dpdk/23.03/hdr_split/0001-net-ice-support-hdr-split-mbuf-callback.patch
18+
19+
# build dpdk
20+
rm build -rf
21+
meson setup build
22+
ninja -C build
23+
sudo ninja install -C build
24+
cd ..
1125
```
1226

1327
## 3. Update DDP package version with header split feature

lib/src/st2110/st_rx_video_session.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2341,7 +2341,7 @@ static int rv_handle_hdr_split_pkt(struct st_rx_video_session_impl* s,
23412341

23422342
/* find the target slot by tmstamp */
23432343
bool exist_ts = false;
2344-
struct st_rx_video_slot_impl* slot = rv_slot_by_tmstamp(s, tmstamp, NULL, &exist_ts);
2344+
struct st_rx_video_slot_impl* slot = rv_slot_by_tmstamp(s, tmstamp, payload, &exist_ts);
23452345
if (!slot || !slot->frame) {
23462346
if (exist_ts) {
23472347
s->stat_pkts_redundant_dropped++;

0 commit comments

Comments
 (0)