Hello Stephen,
I'll fix it next version.
On 2023/9/18 10:33, Stephen Hemminger wrote:
On Mon, 4 Sep 2023 12:56:34 +0800
<wa...@3snic.com> wrote:
+void *
+sssnic_workq_consume(struct sssnic_workq *workq, uint16_t num_entries,
+ uint16_t *ci)
+{
+ void *e;
+ uint16_t current_ci;
+
+ if (workq->idle_entries + num_entries > workq->num_entries)
+ return NULL;
+
+ current_ci = sssnic_workq_ci_get(workq);
+ e = (void *)sssnic_workq_entry_get(workq, current_ci);
Minor nit: cast to void * is not normally necessary in C.
Compiler will do it for you.
--
Regards,
Renyong Wan