[Differential] D13995: NVMe controller emulator for bhyve.
cem (Conrad Meyer)
phabric-noreply at FreeBSD.org
Sat Jan 20 18:36:02 UTC 2018
cem added a reviewer: imp.
cem added a comment.
Awesome!
I can't speak to the NVMe emulation itself (I know almost nothing about the protocol and nothing about Bhyve HW emu interfaces). Maybe Warner can.
INLINE COMMENTS
> nvme.h:127
> enum shn_value {
> + NVME_SHN_NOEFCT = 0x0,
> NVME_SHN_NORMAL = 0x1,
indentation looks off here. is it just phabricator?
> Makefile:65
>
> -CFLAGS+= -I${BHYVE_SYSDIR}/sys/dev/usb/controller
> +CFLAGS+= -I${BHYVE_SYSDIR}/sys/dev/usb/controller -I${BHYVE_SYSDIR}/sys/dev/nvme/
>
don't need trailing slash
> pci_nvme.c:19
> +#ifdef NVME_DEBUG
> +static FILE* dbg;
> +#define DPRINTF(format, arg...) \
style nit: `static FILE *dbg;` (asterisk goes on the right side of the space)
> pci_nvme.c:21
> +#define DPRINTF(format, arg...) \
> + do { \
> + fprintf(dbg, format, ##arg); \
style nit: Should be tab indents throughout (can't tell if that's just Phab making it look like 4 spaces or actually 4 spaces).
REVISION DETAIL
https://reviews.freebsd.org/D13995
EMAIL PREFERENCES
https://reviews.freebsd.org/settings/panel/emailpreferences/
To: sux2mfgj_gmail.com, grehan, trasz, imp
Cc: cem, freebsd-virtualization-list, chuck
More information about the freebsd-virtualization
mailing list