Re: [Qemu-devel] NetBSD qemu block device support

[Blue Swirl]

On 3/17/10, haad <address@hidden> wrote:
> Hi folks,
>
>  This patch at [1] add support for NetBSD block ioctl calls to qemu
>  block-raw.c file. It was written for xen version of qemu but basically
>  it will work with vanilla qemu, too. Would anyone like to commit this
>  patch so it can be included into the base xen distribution ?

The patch does not apply to current development repository.

There is no description of the change suitable for changelog without
any editing.

Signed-off-by: line is missing.

The patch combines formatting (whitespace) changes with functional
changes. The formatting changes seem useless.

The essence of the patch is twofold, It adds NetBSD specific includes
and NetBSD specific code to raw_getlength(). These look OK, except
formatting may be a bit off (no space after if) and currently the code
probably should be surrounded by
#ifdef DIOCGWEDGEINFO
#endif
or just #ifdef __NetBSD__.