Hello Pallavi,

> +#include "eal_internal_cfg.h"
> +
> +/* sets up platform-specific runtime data dir */
> +int
> +eal_create_runtime_dir(void);
> +
> +/* returns runtime dir */
> +const char *
> +eal_get_runtime_dir(void);

Any reason not to #include "eal_filesystem.h"?

> +
> +static inline const char *
> +eal_runtime_config_path(void)
> +{
> +     static char buffer[PATH_MAX];  /* static so auto-zeroed */
> +     char  Directory[PATH_MAX];

Should be "directory" according to the style guide.

> +
> +     GetTempPathA(sizeof(Directory), Directory);
> +     snprintf(buffer, sizeof(buffer)-1, RUNTIME_CONFIG_FMT, Directory,

Style guide requires spaces around binary "-".

> +static inline const char *
> +eal_hugepage_info_path(void)
> +{
> +     static char buffer[PATH_MAX];  /* static so auto-zeroed */
> +     TCHAR  Directory[PATH_MAX];

Should be "directory", see above.

Reply via email to