Skip to content

Commit 793d9ff

Browse files
committed
Remove redundant and unnecessary comments.
1 parent cadb02e commit 793d9ff

2 files changed

Lines changed: 0 additions & 26 deletions

File tree

lib/ff_config.c

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1025,10 +1025,6 @@ ini_parse_handler(void* user, const char* section, const char* name,
10251025
} else if (MATCH("kni", "udp_port")) {
10261026
pconfig->kni.udp_port= strdup(value);
10271027
} else if (MATCH("stack", "kernel_coexist")) {
1028-
/* Kernel-stack coexistence switch: "1"/"on"/"true"/"yes" enable it,
1029-
* anything else (incl. "0"/"off") keeps it disabled. When enabled, a
1030-
* socket created with SOCK_KERNEL additionally uses the host kernel
1031-
* stack alongside F-Stack; default per-fd semantics stay F-Stack. */
10321028
pconfig->stack.kernel_coexist =
10331029
(strcasecmp(value, "1") == 0 || strcasecmp(value, "on") == 0 ||
10341030
strcasecmp(value, "true") == 0 || strcasecmp(value, "yes") == 0)
@@ -1364,8 +1360,6 @@ ff_default_config(struct ff_config *cfg)
13641360
cfg->dpdk.promiscuous = 1;
13651361
cfg->dpdk.pkt_tx_delay = BURST_TX_DRAIN_US;
13661362

1367-
/* kernel-stack coexistence disabled by default (pure F-Stack).
1368-
* memset already zeroes it; set explicitly for clarity. */
13691363
cfg->stack.kernel_coexist = 0;
13701364

13711365
/* KNI ratelimit default disabled */

lib/ff_config.h

Lines changed: 0 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -319,14 +319,6 @@ struct ff_config {
319319
} kni;
320320

321321
struct {
322-
/* Kernel-stack coexistence capability switch (per process):
323-
* 0 = disabled (default): pure F-Stack, every socket uses the
324-
* F-Stack user-space stack.
325-
* 1 = enabled: a socket created with an explicit SOCK_KERNEL marker
326-
* (and without SOCK_FSTACK) additionally uses the host Linux
327-
* kernel stack, coexisting with F-Stack in the same process.
328-
* The default per-fd semantics stay F-Stack; this switch never makes
329-
* the whole process default to the kernel stack. */
330322
int kernel_coexist;
331323
} stack;
332324

@@ -362,18 +354,6 @@ int ff_load_config(int argc, char * const argv[]);
362354
* (config.ini [stack] kernel_coexist=1). */
363355
int ff_kernel_coexist_enabled(void);
364356

365-
/*
366-
* Free every heap-allocated field referenced by ff_global_cfg (port_cfgs,
367-
* vlan_cfgs, vdev_cfgs, bond_cfgs, rss_check_cfgs, freebsd.boot/sysctl
368-
* linked lists, dpdk_argv[], filename, etc.) and zero the structure so a
369-
* subsequent ff_load_config() call starts from a clean slate.
370-
*
371-
* Registered via atexit() at the end of the first successful
372-
* ff_load_config(), and also invoked at the start of any later
373-
* ff_load_config() to make the loader idempotent (FU-S2-2-CFG-UNLOAD).
374-
*
375-
* Safe to call multiple times; calling on an already-zeroed cfg is a no-op.
376-
*/
377357
void ff_unload_config(void);
378358

379359
#ifdef __cplusplus

0 commit comments

Comments
 (0)