<div dir="ltr">We are not validating anything.<div>This is for our internal analysis and product requirements.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 19, 2023 at 1:56 PM Stephen Hemminger <<a href="mailto:stephen@networkplumber.org">stephen@networkplumber.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Fri, 19 May 2023 13:46:18 -0700<br>
Rushil Gupta <<a href="mailto:rushilg@google.com" target="_blank">rushilg@google.com</a>> wrote:<br>
<br>
> +#include <rte_version.h><br>
>  <br>
>  #include "../gve_logs.h"<br>
>  <br>
> +#ifdef __linux__<br>
> +#include <sys/utsname.h><br>
> +#endif<br>
> +<br>
>  typedef uint8_t u8;<br>
>  typedef uint16_t u16;<br>
>  typedef uint32_t u32;<br>
> @@ -73,6 +78,12 @@ typedef rte_iova_t dma_addr_t;<br>
>  <br>
>  #define msleep(ms)           rte_delay_ms(ms)<br>
>  <br>
> +#define OS_VERSION_STRLEN 128<br>
> +struct os_version_string {<br>
> +     char os_version_str1[OS_VERSION_STRLEN];<br>
> +     char os_version_str2[OS_VERSION_STRLEN];<br>
> +};<br>
> +<br>
<br>
Not sure this a good idea. Are you having the host validate<br>
against DPDK versions. This is a bad idea.<br>
<br>
Better to use feature bits like virtio and not be creating<br>
and validating strings about versions.  For example, ever minor<br>
stable release changes this.<br>
</blockquote></div>