Fix BSOD on RingDestroy

Zero Frontend->MaxQueues after calling RingDestroy, as RingDestroy will
query this value to free each BlkifRing, which will decrement an
unsigned value below 0.
Also adds an ASSERT to detect if FrontendGetMaxQueues returns 0.

Signed-off-by: Owen Smith <owen.smith@citrix.com>
Test that Index != 0 rather than > 0, since it is an unsigned quantity.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Conditionally package DPInst

Since DPInst.exe is not shipped with the Windows Driver Kit 10, an
environment variable must point to local copies. Make the inclusion of
DPInst conditional on DPINST_REDIST being defined and that path
existing. This simplifies building packages which do not require DPInst
for installation, and removes a required step to create a working build.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Remove bogus ASSERTion

In a checked build the code in BlkifRingSchedule() sometimes hits the
ASSERTion:

ASSERT3U(State->Count, ==, 0);

This check is there because this code was ported across from XENVIF. In
the context of that driver the check is valid because it should never be
possible to post a partial sequence of netif requests (since that would
violate the protocol). However, in the context of XENVBD posting blkif
requests, it is perfectly reasonable for a subset of blkif requests for
a single SRB to be posted, and hence __BlkifRingPostRequests() may exit
before State->Count falls to zero. Thus the ASSERTion is invalid in this
context and needs to be removed.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Add some sensible default overrides

Without overrides, "max-ring-page-order" and "multi-queue-max-queues"
use values that will consume large amounts of grant references.
"max-ring-page-order" will default to 4 (16 pages per ring)
"multi-queue-max-queues" will default to the lowest of guest vCPU count
or backend's vCPU count.

Override "max-ring-page-order" to 1 (2 pages per ring) and
"multi-queue-max-queues" to 2 (2 rings per block device)

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Work around bug in VS2017 SDV

XENBUS commit 868cd40f (of the same name) introduced a workaround for a
quoting bug in SDV. This commit applies a similar workaround for XENVBD.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Fix SDV release parameter

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Disable spectre mitigation warnings

These are not terribly useful and cause the build to fail.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Update EVTCHN, GNTTAB and STORE interface headers

Updates revision to 09000004
Fixes Evtchn Unmask calls
Updates xenbus interface headers to latest versions

This will resolve an issue where, under certain timing conditions,
xenvbd will request a second reboot before functioning correctly.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Fix SDV build error

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Fix VS2015 build

STOR_PERF_NO_SGL does not appear to exist in the older WDK.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Set StorPortInitializePerfOpts

Sets DPC_REDIRECTION, OPTIMIZE_FOR_COMPLETION_DURING_STARTIO and
CONCURRENT_CHANNELS (to "multi-queue-max-queues") to improve StorPort's
distribution of SRBs to vCPUs.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Move query of "multi-queue-max-queues" to driver.c

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Remove queue.h/.c

XENVBD_QUEUE has been replaced by LIST_ENTRY, and is no longer used.
Remove queue.h/.c and any include directives

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Implement multi-queues

Splits XENVBD_RING into multiple XENVBD_BLKIF_RINGs, one for each shared
ring. Up-to "multi-queue-max-queues" rings are used to pass
blkif_requests and blkif_responses between frontend and backend. Reworks
the ring interactions to remove the locks used by XENVBD_QUEUE,
implementing a queue system similar to XenVifs transmitter queues.

Signed-off-by: Owen Smith <owen.smith@citrix.com>
s/Packets/SRBs in ring.c

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Force single queue in crash/hibernate path

Remove "multi-queue-num-queues" from frontend area when the crashump
driver is active. This will force the backend to use a single queue,
which the crashdump frontend is supplying

Signed-off-by: Owen Smith <owen.smith@citrix.com>
Add whitespace after cast for consistency with other code.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Dont close ring during FrontendReset

Closing the ring (and destroying the shared pages, etc) is not required
when a HwStorResetBus or SRB_FUNCTION_RESET_DEVICE is triggered.
Disabling the ring will cause any outstanding blkif_requests and SRBs to
be failed.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Remove RingTrigger

RingTrigger calls XENBUS_EVTCHN(Trigger..) on during the suspend
callback. Just before this, the ring is recreated and enabled, which
also calls XENBUS_EVTCHN(Trigger..). The explicit call RingTrigger is
unneccessary

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Query "multi-queue-max-queues"

Query "multi-queue-max-queues", and override if neccessary, and work
out a suitable value for the number of queues used. Also adds the
commented out writing code to set "multi-queue-num-queues"

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Update Xen header files

The original patch from Owen contained too much whitespace damage. This
patch replaces it with the results of an invocation of get_xen_headers.py
and includes all resulting header updates.

NOTE: This patch adds one extra header (physdev.h) and a fix to remove
      now-duplicate definitions of DOMID_INVALID from frontend.c.

Suggested-by: Owen Smith <owen.smith@citrix.com>
Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Fix line endings

Some CRLF nastiness seems to have crept in.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Abort any partially completed SRBs

When disabling, a SRB may be comprised of requests that have been
prepared and completed. Set the SrbStatus as early as possible, so
that any partially completed SRB is always failed.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Remove FreshSrbs queue

Any SRB that fails to prepare will be completed with SRB_STATUS_BUSY,
so that storport can manage the queue. SRBs are prepared in the StartIo
handler, and appended to the PreparedReqs queue. All possible PreparedReqs
are submitted, and if there is more PreparedReqs, the DPC is triggered to
submit more when possible.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Pass SrbExt to RingPrepare* calls

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Rationalize WasQueued semantics

Several functions return values were inverted in the call stack,
Fix the meaning of TRUE/FALSE all through the call stack for submitting
requests

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Use a threaded DPC for ring processing

The current DPC back-off mechanism uses a timer to reschedule long
running DPCs. Timers seem to always have a minimum tick period, so
rescheduled DPCs always have an additional level of latency.
Using a threaded DPC (which executes at PASSIVE_LEVEL) will allow
other drivers to interrupt the vbd DPC, and maintain system
responsiveness. Once the vbd's DPC can be pre-empted by another
DPC, a DPC back-off mechanism is no longer required.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Do more optimization in release builds and add /Qspectre flag

Spectre mitigations apparently only work on optimized code.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Add support for building with Visual Studio 2017

Also remove mappings for obsolete versions of VS in build.py.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Make sure Mdl->StartVa is set by __AllocatePages()

wdm.h carries this comment:

// Notice that while in the context of the subject thread, the base virtual
// address of a buffer mapped by an MDL may be referenced using the
// following:
//
//      Mdl->StartVa | Mdl->ByteOffset
//

Hence it is important that a mapped MDL has a valid StartVa field as well
as a valid MappedSystemVa field. Unfortunately, for reasons best known to
Microsoft, MmMapLockedPagesSpecifyCache() does not ensure this is the
case, so it needs to be fixed up by __AllocatePages() itself.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Stop trying to parse a target structure as a text string

The Verbose() logging call in TargetCreate() is using %s where is means %p.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Report Inquiry 0xB0 BlockLimits

Reports discard-alignment and discard-granularity

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Rationalize SCSIOP_* handlers

* Use same error handling for invalid buffers, etc.
* Allow for SCSIOP_MODE_SENSE10
* Correctly report VDISK_READONLY flag

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Zero ring stats before ASSERTing ring struct is zeroed

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Remove unused code and general tidy up

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Remove RingReQueueRequests

Since the Frontend transitions the ring through Closed and back to
Enabled on a resume from suspend, which aborts all queued and
prepared requests, de-queueing and re-queuing all requests from the
Target suspend callback is unneccessary. Also removes the Target
suspend callback and interface which is now unused.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Fix BSOD on shutdown due to freeing NULL pointer

Unfortunately ExFreePoolWithTag() is not NULL-safe, and Base64Free() is
coded assuming it is. This patch adds an extra check for NULL to fix the
problem.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Use CACHE interface for REQUESTs, SEGMENTs and INDIRECTs

Replaces the ExLookasideList* calls with CACHE interface.
This allows the INDIRECT object constructor to allocate the indirect
page, and not allocate the indirect page on every object retrieval.
Also defers the indirect page free until the object is destroyed, and
not on every object put.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Tidy up srbext.h

* reorders struct definitions
* remove inline functions and unneccessary headers
* replace Request.Id with (ULONG64)(ULONG_PTR)Request

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Add override for MaxRingPageOrder

Allow a Windows admin to limit the ring page count without
affecting any other VMs on the host. The override can be used
to restrict blkback's generous 16 page ring to a smaller value.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Use CACHE interface for bounce buffers

When the incomming read/write buffer is not aligned, the data needs
bouncing via a page-aligned buffer. Use the CACHE interface to provide
this buffer instead of a custom lookaside collection. Also moves the
storage for mapping the source buffers to the bounce buffer structure,
which will reduce the allocation size of the segments used on the fast
path (aligned buffers). The Indirect pages are also pre-allocated in the
object construction, and not for every object retrieval, and likewise
indirect pages are only freed on object destruction, and not for every
object put.
This will remove the log spam caused by the custom implementation.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Query for CACHE interface

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Add missing Verbose() format argument

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Remove generic store accessors in frontend

Replace with specific use functions.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Move Inquiry handling into target.c

Removes pdoinquiry.[ch] and its opaque inquiry pointer in favour of
making target.c the location of all target SRB handlers.
Removes a XenServer specific extension to the page83 inquiry data.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Tidy up GranterDebugCallback

Make the granter register for its own debug callback, rather than it
being called from the frontend debug callback.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Tidy up FrontendDebugCallback

Make the frontend register for its own debug callback, rather than it
being called from the target debug callback.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Move request prepare/cleanup to ring

Keep the blkif protocol related functions in ring.[ch]. Keep target.[ch]
related to SCSI operations

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Use macros for Frontend data accessors

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Merge Notifier and BlockRing into Ring

Remove the seperation between the event channel and shared ring.
Fixes the debug callback and adds more error messages.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Add DPC timeout check

Check the DPC has not exceeded 1/2 its alloted time every
* 1/4 ring of responses processed
* all outstanding prepared requests submitted
* 1 queued SRB prepared

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Added a function called from DriverEntry where...

we can safely read registry keys and convert strings at PASSIVE_LEVEL

The MSDN documentation for various registry key access and string
conversion functions requires the caller to be at PASSIVE_LEVEL.
One of the reasons for this is that the string conversion tables used by
functions such as RtlAnsiStringToUnicodeString are stored in paged pool
memory. Both the page fault handler and the process scheduler run at
DISPATCH_LEVEL, therefore you must not touch memory at DISPATCH_LEVEL that
could be paged out. A process running at DISPATCH_LEVEL cannot be
preempted for the page fault handler to run.

To aid Static Driver Verifier code analysis and inform developers, I have
added SAL annotations that indicate PASSIVE_LEVEL is required on certain
registry access functions.

Signed-off-by: Eric Mackay <mackayem@amazon.com>
Re-based onto master and adjusted for style. Note this involved fixing
whitespace issues in frontend.c.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>
---

v2:
 - Remove unused FeatureName array
 - Make more use of const qualifier for override name

Make sure AdapterInitialize is called at PASSIVE_LEVEL

AdapterHwFindAdapter() can be called at DISPATCH_LEVEL and therefore it
is not safe to call AdapterInitialize() directly from there.

This patch, instead, calls StorPortEnablePassiveInitialization() from
AdapterHwInitialize() to request a PASSIVE_LEVEL initialization routine
is called immediately after AdapterHwFindAdapter() so that
AdapterInitialize() can be called from there.

Suggested-by: Owen Smith <owen.smith@citrix.com>
Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Tidy up target.h

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Move suspend callback into target.c

Remove chained call from adapter.c

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Scope TargetDebugCallback in target.c

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Refactor target.c

* Adds property accessors
* Moves PNP SRB handler to adapter.c
* Streamlines header

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Refactor BuildIo and StartIo calls

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Move scatter gather list parsing to adapter.c

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Inline MapSegmentBuffer()

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Pass PXENVBD_SRBEXT, not PSCSI_REQUEST_BLOCK

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Move StorPort*() calls to adapter.c

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Remove unused code

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Fix SDV

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Refactor Adapter.c

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Tidy up Driver.h/.c

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Rename Pdo -> Target

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Rename Fdo -> Adapter

Also remove reference counts

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Remove VS2012 and VS2013 build scripts

This patch removes the scripts for building under VS2013 and VS2013 and
also fixes the package destination when building using VS2015.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Reboot request keys should be volatile

When a driver makes a reboot request it should use a volatile registry key.
The monitor service will explicitly remove the key prior to reboot but,
if the reboot is initiated in some other way and the key is non-volatile,
the monitor service will then needlessly prompt for a second reboot.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Take notice of 'discard-enable' in xenstore

When enabling discard in the frontend, both the 'feature-discard' and
'discard-enable' flags should be taken into account.

Signed-off-by: Andreas Kinzler <ml-ak@posteo.de>
[Added commit comment]
Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Fix race setting SrbStatus

SrbStatus needs to be set to PENDING right at the beginning of the
prepare functions, before the SRB has been added to any queue, otherwise
we risk a race with the completion code.

Reported-by: Andreas Kinzler <ml-ak@posteo.de>
Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Fix race calculating SrbExt->Count

It is possible under heavy loads for the backend to
start completing sub-requests of a Srb before the
SrbExt Count is set. This would leave the count unable
to reach 0 (as 1 or more requests are skipped by count
being overridden)

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Ensure event channel is unmasked

If the Pdo is paused, the notifier dpc will not
unmask the event channel. When the Pdo is unpaused, the
event channel remains masked, so no more interrupts
get delivered.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Add registry overrides for features

Add a REG_DWORD to XenVbd's Parameters key, with names based
on the features, as per xenstore value, and set to 0 to
override use of that feature.

For example,
 "HKLM\System\CCS\Services\XenVbd\Parameters" "feature-barrier" 0
  will prevent the frontend issuing BLKIF_OP_WRITE_BARRIERs
 "HKLM\System\CCS\Services\XenVbd\Parameters" "feature-discard" 0
  will prevent the frontend issuing BLKIF_OP_DISCARDs

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Squashing build warnings is not enough for SDV

An ASSERT will hopefully keep it quiet.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Fix SDV error

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Squash build warning

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Unify the three variants of util.h

The new header is placed a new src/common sub-directory.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Fix memory leak in __FreePages()

The pool memory for the MDL also needs to be freed.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Fix pool leaks exposed by DriverVerifier

* RegistryCloseKey was not called in DriverRequestReboot
* RegistryTeardown was not being called in DriverUnload
* __RegistryFree was not being called in RegistryCreateKey
* Reordered DriverEntry slightly for improved code consistancy

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Make sure we don't leave SRBs in state PENDING

When we fail to prepare an SRB, it should not be left in state PENDING
otherwise Windows will expect a completion at some point.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Fix packaging bug

The xenvbd.inf file was missing from the package

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Remove spurious FORCEINLINE

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Add missing files

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Update BUILD.md with VS2015/WDK10 information

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Fixes for VS2015/WDK10 build

The package build was not working correctly and caused the overall build
to fail.
At least part of the reason for this is that Microsoft, in their infinite
wisdom, have removed the DIFx redist from WDK10. This patch makes use of
a new environment variable 'DPINST_REDIST' to find the copy of dpinst.exe
to package such that this can be pointed at an older WDK or alternative
location where dpinst.exe can be found.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Add support for building under VS2015/WDK10

Moving to the new toolchain also threw up a few new warnings, which this
patch either fixes or squashes. Also, SDV appears to be fragile in new
ways (and whinge about some new things) so there are fixes for that too.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Update driver version from 8.2.0 to 9.0.0

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Don't disconnect/connect ring when target is reset

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Add registry override to prevent interception of Trim requests

Add a REG_DWORD called 'InterceptTrim' to XENDISK's parameters key
(now created by xenvbd.inf). Set the value to zero to disable Trim
interception.

This patch also adds registry code to XENDISK, brings util.h up to date,
and fixes some cosmetic issues in thread.c.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Succeed SRBs that the backend doesnt support

If the backend does not support BLKIF_OP_DISCARD, BLKIF_OP_WRITE_BARRIER,
or BLKIF_OP_FLUSH_DISKCACHE, the first request will be failed and
subsiquent requests will be succeeded instead of being passed to the
backend. Bring the SRB status into line in both of these situations.

Signed-off-by: Owen Smith <owen.smith@citrix.com>
Remove rather scary warning and instead log any features being disabled.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Send BLKIF_OP_FLUSH_DISKCACHE

If the backend supports flush, send BLKIF_OP_FLUSH_DISKCACHE in
response to SCSIOP_SYNCHRONIZE_CACHE, and advertise the cache
to Windows. Also tracks count of cache operations. If the backend
does not support flush, but supports barrier, send
BLKIF_OP_WRITE_BARRIER instead, but do not advertise the cache to
Windows, as its assumed the backend will guarantee writes are
flushed if the storage requires flushing.

Read backend features and disk info before setting Connected

Setting connected should be the last step of the connection
state transitions.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Fix VS2012 build

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Fix build warnings

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Make use of the 'physical-sector-size' reported by the backend

blkif backends can report a 'physical-sector-size' as will as
'sector-size'. XENVBD currently reads this xenstore key but does
nothing with the value. This patch makes use of the value to set the
LogicalPerPhysicalExponent field in the READ_CAPACITY16_DATA structure
which is populated in response to a SCSIOP_READ_CAPACITY16 CDB.

The patch also modifies XENDISK to discover the logical block size by
sending a SCSIOP_READ_CAPACITY16 rather than relying on an IOCTL to
query StorageAccessAlignmentProperty being sent down the stack and
succeeded by STORPORT (which does not seem to be the case for Windows 10).

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Don't include null terminator in synthesized VendorId identifier

The VendorId identifier on SCSI page 83 is 16 bytes long (+header).
When synthesizing the inquiry data, either by global flag, or missing
xenstore data (sm-data/scsi/0x12/0x83), the NULL terminator on the
VendorId should not be included in the field. When this happens, any
query will decode 3 identifiers (VendorId, EUI64, VendorSpecific) instead
of the intended 2 (VendorId, VendorSpecific). This breaks the XenServer
VSS provider that uses the VendorSpecific identifier to retrieve the
vdi-uuid. This is only an issue when the inquiry data is synthesized and
additional identifiers are required, which is not a common use case.

Signed-off-by: Owen Smith <owen.smith@citrix.com>

Stop checked builds from crashing

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

The SOFTWARE registry hive is not accessible at boot time

Hence it cannot be used for the monitor request key, otherwise XENVBD
cannot necessarily access it.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Update registry code

Bring in new registry code from XENBUS to fix RegistryCreateKey()'s
semantics.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Fix SDV build

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>