Re: [UNIKRAFT PATCH v2 1/7] lib/ukblkdev: blkdev.h: Include <uk/assert.h>

[Laurentiu Barbulescu <lrbarbulescu@xxxxxxxxx>]

Hi Simon,

Looks good to me.

Reviewed-by: Laurentiu Barbulescu <lrbarbulescu@xxxxxxxxx>

On Tue, Dec 22, 2020 at 5:38 PM Simon Kuenzer <simon.kuenzer@xxxxxxxxx> wrote:

The header `blkdev.h` uses UK_ASSERT but did not include
`<uk/assert.h>`.

Signed-off-by: Simon Kuenzer <simon.kuenzer@xxxxxxxxx>
---
 lib/ukblkdev/include/uk/blkdev.h | 1 +
 1 file changed, 1 insertion(+)

diff --git a/lib/ukblkdev/include/uk/blkdev.h b/lib/ukblkdev/include/uk/blkdev.h
index c8de0038d..7aa099092 100644
--- a/lib/ukblkdev/include/uk/blkdev.h
+++ b/lib/ukblkdev/include/uk/blkdev.h
@@ -66,6 +66,7 @@
 #include <stdio.h>
 #include <errno.h>
 #include <limits.h>
+#include <uk/assert.h>
 #include <uk/list.h>
 #include <uk/errptr.h>

--
2.20.1