[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH v4 17/20] ebpf: Simplify error handling
From: |
Akihiko Odaki |
Subject: |
[PATCH v4 17/20] ebpf: Simplify error handling |
Date: |
Sun, 15 Oct 2023 23:02:49 +0900 |
The error handling procedures required when rss_bpf__open() and
rss_bpf__load(rss_bpf_ctx) are different so it's better to implement
them separately.
Signed-off-by: Akihiko Odaki <akihiko.odaki@daynix.com>
---
ebpf/ebpf_rss.c | 10 +++-------
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/ebpf/ebpf_rss.c b/ebpf/ebpf_rss.c
index fe3184fed9..56c518c604 100644
--- a/ebpf/ebpf_rss.c
+++ b/ebpf/ebpf_rss.c
@@ -46,12 +46,13 @@ bool ebpf_rss_load(struct EBPFRSSContext *ctx)
rss_bpf_ctx = rss_bpf__open();
if (rss_bpf_ctx == NULL) {
trace_ebpf_error("eBPF RSS", "can not open eBPF RSS object");
- goto error;
+ return false;
}
if (rss_bpf__load(rss_bpf_ctx)) {
trace_ebpf_error("eBPF RSS", "can not load RSS program");
- goto error;
+ rss_bpf__destroy(rss_bpf_ctx);
+ return false;
}
ctx->obj = rss_bpf_ctx;
@@ -65,11 +66,6 @@ bool ebpf_rss_load(struct EBPFRSSContext *ctx)
rss_bpf_ctx->maps.tap_rss_map_toeplitz_key);
return true;
-error:
- rss_bpf__destroy(rss_bpf_ctx);
- ctx->obj = NULL;
-
- return false;
}
static void ebpf_rss_set_config(struct EBPFRSSContext *ctx,
--
2.42.0
- [PATCH v4 06/20] net: Remove flag propagation, (continued)
- [PATCH v4 06/20] net: Remove flag propagation, Akihiko Odaki, 2023/10/15
- [PATCH v4 07/20] tap: Shrink zeroed virtio-net header, Akihiko Odaki, 2023/10/15
- [PATCH v4 08/20] virtio-net: Copy header only when necessary, Akihiko Odaki, 2023/10/15
- [PATCH v4 09/20] virtio-net: Disable RSS on reset, Akihiko Odaki, 2023/10/15
- [PATCH v4 10/20] virtio-net: Unify the logic to update NIC state for RSS, Akihiko Odaki, 2023/10/15
- [PATCH v4 11/20] virtio-net: Return an error when vhost cannot enable RSS, Akihiko Odaki, 2023/10/15
- [PATCH v4 12/20] virtio-net: Always set populate_hash, Akihiko Odaki, 2023/10/15
- [PATCH v4 13/20] virtio-net: Do not clear VIRTIO_NET_F_RSS, Akihiko Odaki, 2023/10/15
- [PATCH v4 14/20] virtio-net: Do not clear VIRTIO_NET_F_HASH_REPORT, Akihiko Odaki, 2023/10/15
- [PATCH v4 15/20] ebpf: Fix RSS error handling, Akihiko Odaki, 2023/10/15
- [PATCH v4 17/20] ebpf: Simplify error handling,
Akihiko Odaki <=
- [PATCH v4 19/20] ebpf: Refactor tun_rss_steering_prog(), Akihiko Odaki, 2023/10/15
- [PATCH v4 16/20] ebpf: Use standard section name, Akihiko Odaki, 2023/10/15
- [PATCH v4 20/20] ebpf: Add a separate target for skeleton, Akihiko Odaki, 2023/10/15
- [PATCH v4 18/20] ebpf: Return 0 when configuration fails, Akihiko Odaki, 2023/10/15
- Re: [PATCH v4 00/20] virtio-net RSS/hash report fixes and improvements, Michael S. Tsirkin, 2023/10/18