Use new monitor request key

The monitor service now uses a request key in registry under HKLM/SOFTWARE.
This patch modifies __DriverRequestReboot() to use the new key, which is
now set as a parameter by the INF file.

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

Fix logging format string

Should have been %Z rather than %s since the argument is ANSI_STRING.

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

Re-work identity stealing code

The change introduced by patch 04c391d9 "Replace copying network
settings with identity stealing" had a flaw...

If a PV device that aliases an emulated device is installed then it
will only steal linkage if that emulated device is operational at the
time. This works on initial driver install, but not if the PV network
device driver is removed and then re-installed without an interim
reboot (which would make the emulated device operational). Instead,
after re-installation, the PV device has a brand new stack binding and
thus any network settings changes that were previously made are
apparently lost.

To fix this problem, this patch breaks the process of stealing the
emulated device's stack binding into two phases:

1) Discovery: When the PV device driver is installed, if an aliasing
   emulated device is operational, a DWORD value called "VIF" is
   created under the emulated devices software key. The value is set
   to the PV device instance.

2) Theft: When the PV device is being brought online, the registry is
   checked to see if any network device has a software key containing
   a "VIF" value that patches the PV device instance.
   If such an entry is found then the stack binding is stolen from
   that device. Since the "VIF" value is in the *emulated* device's
   software key, it will not be removed if the PV device driver is
   removed and re-installed and so the problem described above will
   not occur.

Because this patch avoids stealing the emulated device's stack bindings
if it is actually operational (i.e. we're in phase 1) this results in the
class installer building a new set of stack bindings for the PV device
during initial installation.
These are then restored any time a PV device with a stolen stack binding
is offlined. So, when a PV device is offline there is no evidence that
any stack binding has been stolen and hence, during driver removal, there
is no risk of an emulated device's stack binding being torn down.

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

Don't qualify the check for the backend being online...

...with whether an eject request for the frontend has been issued. It's
either online or it's not.

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

Make sure the NICS unplug count does not run away

In a tight loop of network attach/detach it's possible for a mismatch in
the number of unplug requests and revocations to occur, leading to a
run-away unplug count.

This patch adjusts the location of the requests and revocations to make
sure this does not happen.

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

Make sure we don't end up with duplicate PDOs

In a tight loop of network attach/detach it's possible to end up with
duplicate PDOs because, as soon as a PDO gets set to 'missing' a new one
with the same name can be created. This leads to races in accessing the
vif areas in xenstore and things can get very confused.

This patch makes sure that a new PDO cannot be created until a previous
incarnation is in the 'deleted' state, which means there will definitely
be no conflict in interactions with xenstore.

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

Fix locking and teardown bugs in controller frontend

If the backend does not implement the control ring then the frontend will
eventually deadlock. Also, by inspection, the teardown code does not
release interfaces if the control ring was not present.

This patch fixes both issues.

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

Fix compatibility with older backends

There was a problem when XENVIF was brought up on backends that don't
implement a control ring. This patch fixes the issue.

Reported-by: Rafal Wojdyla <omeg@invisiblethingslab.com>
Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Use XENBUS_MONITOR service to request reboot rather than SetupAPI

This means more code can be removed from XENNET's co-installer and we get a
more meaningful message displayed to the user.

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

Add missing names to the INF file

The package should have a name and the XENVIF driver should also have a
DisplayName.

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

Fix function name

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

Fix SDV warnings

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

Defer transmit response event until the last possible moment

When sending packets, there is no need to enable a transmit reponse
event until the ring fills or there are no more packets to send.

This patch therefore defers advancing the ring event count until just
before a batch of packets are pushed.

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

Avoid transmitting on the wrong CPU

The transmit and receive rings have DPCs and event channels affinitized
to a particular CPU. Thus, when XENNET queues a new packet at the
transmit side, make sure the packet is prepared and posted from the
CPU to which the DPC and event channel are bound to avoid lock contention.

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

Affinitize watchdog threads

To avoid bouncing locks between CPUs, affinitize the ring watchdog thread
to the same CPU as the event channel and DPC.

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

Re-introduce mistakenly dropped frontend state logic...

...from commit c9b6e56e "Cope with unexpected initial backend states" to
handle a backend that is already connected (for cases where Windows is
booted via iSCSI).

Signed-off-by: Eytan Heidingsfeld <eytanh@gmail.com>
Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Send transmit side hash value to the backend

Backends capable of supporting packet hashing on the guest receive side
are also capable of accepting hash values in an extra info fragment on
the guest transmit side. This patch adds code to construct and send the
extra info fragment if the packet metadata contains hash information and
the backend is able to process it.

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

Add new functionality to VIF interface to support RSS

This patch adds the necessary extra functionality into the VIF interface
to make use of the new control ring to support NDIS RSS in XENNET.

The VIF interface version is bumped to 6 and the PDO revision adjusted
accordingly.

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

Add frontend code for the new netif control ring

My recent patches to Xen's netif.h specify a control ring that can be
used by a frontend driver to configure packet hashing and steering in
a backend.
This patch adds the necessary code to XENVIF to drive this new ring,
however the rest of the code to link this up to new VIF interface
functionality (so that it may be used by XENNET) is deferred to a
subsequent patch.

This patch also pulls in an updated EVTCHN interface from XENBUS and
corrects the PDO binding.

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

Update Xen headers to 4.7.0-rc2

This pulls in the recent changes to netif.h which are required by
subsequent patches.

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

Make sure transmitter DPC does not try to unmask non-existent evtchn

In the case where the backend does not support split event channels (one
for RX and one for TX) then the transmitter code in XENVIF does not open
the combined channel; it is handled by the receiver code. So, the
transmitter DPC should not make any attempt to unmask the event channel at
the end of processing.

This patch adds the necessary check for FrontendIsSplit before doing the
unmask and also removes an unnecessary check for FrontendIsSplit in the
transmitter ring enable function.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>
Reported-by: Eytan Heidingsfeld <eytanh@gmail.com>
Tested-by: Eytan Heidingsfeld <eytanh@gmail.com>

Fix memory leak

This was breaking SDV builds.

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

Replace copying network settings with identity stealing

It is becoming increasingly hard to find all the places in the registry
that Windows uses to stash network settings and copy them from the
emulated NIC to the new PV device when it is first installed.

So, rather than fighting a losing battle, steal the identity of emulated
devices instead, The identity information is essentially contained in two
values stored in the 'software'registry key of a NIC (under Control/Class/
<NET GUID>): NetCfgInstanceId and the NET_LUID, made up of *IfType and
NetLuidIndex. By copying these values and the contents of the Linkage subkey
we can effectively impersonate the emulated NIC once it is unplugged after
reboot.

This patch amends the settings code accordingly and also does some tidying
in pdo.c and registry.c.

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

BootFlags should be a parameter, not a subkey

A misplaced comma in the INF file means that BootFlags is being created as
a subkey under the service key with a default DWORD value, rather than a
DWORD value with the name 'BootFlags'.

This patch moves the comma and fixes the problem.

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

Avoid ASSERTion failure on VIF plug

If the backend is not fast enough then it's possible that the frontend
code sees the backend in state Initialising rather than InitWait. This
causes an ASSERT to be hit.
This patch simply whitelists the Initialising state to avoid the problem.

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

Prevent VIF from being incorrectly considered offline...

...one resume from suspend.

My previous reversion of 765b7a6a exposed the original reason the
reverted patch.

On resume from suspend, the backend will have been re-created by the
toolstack and hence the frontend detects it as being offline. This
causes the frontend to request ejection of the disk (thinking that
the reason for the backend going offline is because the VIF is being
unplugged).

This patch works around this problem by introducing an early suspend
callback to note that the frontend has actually gone offline, which
will prevent the ejection from being requested.

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

Remove erroneous code to re-acquire backend path on resume

Commit 765b7a6a added code to release and re-acquire the backend
path (and domid) on resume from suspend. That code falsely
assumes that the frontend is not in a closed state at the time.
It's also not clear why it was added in the first place so
this patch removes it again, as it causes a BSOD when the
frontend is in the closed state.

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

Only veto PDO start when InitSafeBootMode is set

There is no need to prevent FDO or PDO creation to stop PV network devices
from coming up, we just need to prevent the PDOs from starting.

Thie patch removes the global veto and instead bails out of PdoStartDevice()
if the system is in safe mode.

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

Dynamic multicast control

Only use the multicast control feature of the backend if it supported
"feature-dynamic-multicast-control" otherwise attempts to enable the 'all
multicast' filter level in XENVIF will have no effect and the NDISTest 6.5
MultiCast Address test will fail.

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

Make transmitter code the canonical place where multicast control...

...feature is handled

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

Fix memory leak in error path

If a malformed large packet is received then it is possible that a packet
structure is leaked on the error path.

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

Cancel pending timers when rings are disabled

No new timers can be scheduled once rings are disabled but if the rings are
both disconnected and destroyed within the next 100us then it's possible
that the DPC might hit some unitialized data so make sure pending timers
are cancelled.

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

Add DPC watchdog avoidance to receiver and transmitter poll

Under HCK MPE testing it's possible for both receiver and transmitter
polling loops to run long enough to hit the DPC watchdog. This patch
avoids the problem by terminating the polling loops after a 'batch'
(currently 1/4 of the ring) and then checking the remaining time before the
watchdog fires. If the remaining time is still more than half the total
time then another poll is done, otherwise a timer is scheduled to complete
the work.

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

Avoid evaluating assertion expressions in free builds

The evaluations are pointless and the warnings generated by not evaluating
can be squashed with a couple of #pragmas.

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

Avoid falling foul of stupid registry values

Without this patch mistakenly setting 'FrontendMaxQueues' to 0 in the
registry will cause an unobvious BSOD.

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

Fix DPC accounting

The DPC count is not zeroed when the event count is zeroed leading to some
odd looking stats. Also the DPC count should really be incremented by the
DPC itself to ensure serialization.

While in the neighbourhood, the ring notify functions in transmitter and
receiver look a little superfluous; they can easily be folded directly
into the DPC if the receiver code triggers the transmmitter DPC in the case
of a combined event channel.

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

Fix WHQL InvalidPackets test

Because the transmit code now parses packets before queueing (for the
purposes of maybe needing to hash) the operation can fail of the headers
are invalid. The WHQL test does not expect this to happen.

This patch ignores the parse failure but instead fails the packet when it
is de-queued but only if absolutely no header (i.e. even the ethernet
header) could be parsed. This seems to be enough to keep the test happy.

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

Stop using prefix GSO

Since acknowledging that the netif response id is meaningless in the code,
we no longer need to use the prefix flavour of GSO.

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

Centralize compatibility code

This patch moves the code that maintains compatibility with older
VIF interface versions into the vif.c module, rather than having it
distributed amongst vif.c, transmitter.c and receiver.c.

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

Fix revision number to avoid collision with staging-8.1

My previous patch incorrectly used a PDO revision that was already in use
on the staging-8.1 branch.

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

Update VIF interface to version 5

Unfortunately VIF interface version 4 suffers from a shutdown race with
the TransmitterQueuePacket method. If a call to this races with the Disable
method then a NULL callback function pointer can be invoked.
This patch fixes the race by having TransmitterQueuePacket return a status
code to indicate whether a packet has been queued (rather than invoking the
completion callback if it has not), but this necessitates a interface change
so the VIF interface version is also updated to 5 and a new PDO revision is
added.

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

Re-synchrinize registry.c with XENBUS

The registry code in XENBUS has some fixes that are not present in the
XENVIF copy, so import the updated code from XENBUS.

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

Re-synchronize util.h with XENBUS and use __toupper()

A recent patch introduced __tolower() and __toupper() into util.h as
replacements for tolower() and toupper() respectively as the latter do
a hidden conversion to Unicode which make then unsafe at any IRQL other
than PASSIVE_LEVEL.
This patch imports util.h from XENBUS and fixes other code to be compatible.

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

Fix SDV build

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

Introduce VIF interface version 4

This greatly simplifies the interface with XENNET by queuing single packets
on both the transmit and receive side, negating the need for a shared packet
structures and bringing static hash calculation into XENVIF.
Crucially this also means that XENNET no longer needs to use the
XENBUS_CACHE interface which allows it to be patched to avoid the race
introduced by commit 026aa32c "Make sure XENBUS interfaces are released when
going into S4".

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

Stop caching MaxQueues, NumQueues and Split in Transmitter and Receiver

Just call the Frontend accessor when the values are required.

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

Move check for split event channels into Frontend module

Reduce code duplication and efficiency a little by checking the xenstore
key once in Frontend and then providing a function with Transmitter and
Receiver can use to grab the value.

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

Make sure XENBUS interfaces are released when going into S4

Because a transition into and out of S4 means a new domain is built, it's
crucial that all XENBUS interfaces are released (so that things like
event channels, grant tables and the xenstore ring get re-constructed).

This patch fixes code paths where this was not being done.

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

Fix SDV build

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

Fix xenstore cleanup

The distribution information is not cleared in the correct place and
cleanup of the hotplug feature flag is missing. This patch fixes both
issues.

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

Add XENBUS_STORE and XENBUS_SUSPEND interfaces to exported set

To advertise driver distribution information to xenstore XENNET needs to
make use of the XENBUS_STORE and XENBUS_SUSPEND interfaces. As XENVIF is
XENNET's parent driver, it must export them. Adding them to the exported
set also requires update of the PDO revision number.

Signed-off-by: Owen Smith <owen.smith@citrix.com>
Import updated store_interface.h from XENBUS and revised commit comment.

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

Publish network information to xenstore using documented paths

My recent patch series to Xen added documented paths for advertising vif
hotplug capability and paths for publishing network interface information
(name, IP addresses, etc.).

This patch modifies XENVIF to make use of those new paths.

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

Publish distribution information to xenstore

My recent patch series to Xen added a documented path and format for
publishing information about PV driver distributions to xenstore.

This patch adds code to populate the documented path (should it exist)
with information about the XENVIF driver package.

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

Add a registry override to veto driver installations

There are certain cases where a local installer package may wish to
prevent Windows Update installations of drivers. This can be achieved by
having the co-installer check for a registry value and fail it's pre-install
phase if the value is present.

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

Change coinstaller log level to TXTLOG_WARNING

Windows 10 does not enable logging of TXTLOG_DETAILS in setupapi.dev.log by
default, so use TXTLOG_WARNING to make sure the messages appear.

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

Fix error path in transmitter header preparation

The wrong value was being checked for NULL after allocation of a fragment.

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

Cope with zero-length binary values in the registry

It appears that it is legitimate for the registry to contain zero-length
binary values, so the registry access code needs to cope with them and
the settings copy code needs to allow for them too.

Reported-by: Rafal Wojdyla <omeg@invisiblethingslab.com>
Signed-off-by: Paul Durrant <paul.durrant@citrix.com>

Remove version 1 of the XENVIF_VIF interface

The revision has been gone from the PDO for a long time so version 1 of
the interface has been technically dead for a while. This patch cleans up
the code.

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

Cosmetic change to vif_interface.h

This does not affect the size, layout or semantics of anything so no
need to bump the version number.

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

Add code to set receive side backfill size

The NDIS header/data-split feature requires that the first data
MDL in a NET_BUFFER has sufficient headroom to take a copy of the
entire header MDL just in case something up the stack can't cope
with the split.
To this end NDIS specifies a 'backfill size' which must be
reserved. This patch implements support for that by copying and
adjusting the first data MDL if necessary.

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

Parameterize vendor prefix and PCI device id

The XenServer PV vendor prefix ('XS') and PCI device (C000) are still
hard-coded into the XENVIF package. These need to be stripped out and
replaced by values that can be customized at build time. This patch does
that.

The patch also reverts to building version.h and customizing xenvif.inf
directly in build.py.

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

Use multicast control if the backend supports it.

This required a bit of re-work in the transmitter code since using multicast
control necessitates sending dummy transmit requests with special extra
info fragments. As a knock-on I also re-worked the way the IP address table
is maintained, removing bits that were in the transmitter code into the
general frontend code.

Re-work was also required in the mac code since it is also necessary to
track addition and removal of individual multicast addresses, rather than
just handling a table update.

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

Don't treat a missing Driver key as a hard failure

When looking to see whether an incumbent child driver will patch the
PDO names created by the new version of XENVIF, ignore any cases where
we find that the Driver key referenced in the Device key is actually
missing.

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

Update to 8.2.0

I also removed CHANGELOG as it's pretty useless and will likely become
misleading.

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

Stop manipulating the old unplug value in the co-installer...

...and take the new ClearUnplugRequest() code from the XENVBD co-installer
instead.

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

Avoid holding a reference to the UNLPLUG interface

If the system goes into S3 or S4 then currently the reference will still be
held. This is not problematic for this interface but it is not desirable to
hold references in this circumstance and for other interfaces it can lead
to bugchecks.

This patch adds a function, PdoRequestUnplug(), which only acquires the
interface as needed and then makes use of that.

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

Don't NULL out fragment in pending array __TransmitterRingFakeResponses()

It is completion of the response in TransmitterRingPoll() that should NULL
out the entry in the array. Doing so causes the assertion at
transmitter.c:2111 to fire in a debug build and a page fault in a non-debug
build.

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

Fix SDV build

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

Registry string value types cannot be inferred

Following a similar fix in XENBUS this patch adds an explicit Type parameter
to RegistryUpdateSzValue(). It also adds a similar parameter to
RegistryQuerySzValue() so that code querying a string value can find out
what type it is.

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

Fix potential NULL-pointer dereference...

...in parsing SystemStartOptions.

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

Use prefix matching for IPv6 address copy code

...and also fix registry path and add support for similar IPv4 address
copying.

Windows puts static IP address values in some oddly named and located
registry keys. The key names are just GUIDs and the only useful part of
the values names (that allows use to find them) is that they prefix
match the interface LUID rendered as a 16 digit hex string.

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

Apply PnP fixes analogous to those recently made in XENBUS

Code inspection in XENBUS highlighted some issues with the PnP code. The
same issues need to be fixed in XENVIF.

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

Update docs and set version to 8.1.0

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

Don't trust the integrity of the shared rings after migration

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

Don't mess with DbgPrint filters

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

Move friendly name setting into driver

XENVIF can set the friendly name of each PDO during start device processing
which removes the need for the XENNET co-installer to handle this.

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

Fix co-installer compatibility check

It was using the wrong enumerator name.

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

Use new unplug mechanism

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

Remove interface subscriber checks from the co-installer...

...and replace with a scan of child driver software keys to check that
the revision number in the MatchingDeviceId value is supported.

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

Apply new PDO revision numbering policy

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

Remove PDO Luid field

It's not used any more.

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

Fix potential buffer overflow

The buffer allocation in FdoQueryDeviceRelations() is completely bogus. Fix
it.

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

Note lack of IPv6 addresses in the log

When we are copying settings, say whether we found any IPv6 address.

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

Remove test for HardwareInterface in FrontendGetInterfaceIndex()

Apparently no interface in the mib table on Windows Server 2008 SP2 is
a hardware interface, even if it has (virtual) hardware.

Also add a missing \n to a trace statement.

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

Modify Trace() macro to keep SDV happy

SDV's free build of the driver was getting upset when arguments to
trace were evaluated but not used.

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

Lookup active network interfaces by index rather than LUID

The frontend address lookup code was failing to check whether the interface
it was querying was actually active, and also using LUID is kind of overkill
when we could just use index.

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

Fix memory leak

...detected by SDV.

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

Avoid PDO namespace conflicts...

...by encoding the driver major version in the upper byte of the
revision.

This clearly implies that any future change in the driver major version
will start a new PDO namespace, but that it almost certainly the correct
thing to do in that case.

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

Update bindings

Refresh the interface headers (just suspend_interface.h in this case) and
add the XENBUS major version into the top byte of the revision, since this
is now used to avoid namespace conflicts.

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

Add functions to backup and restore network settings

In PdoStartDevice(), if an aliasing emulated device is found in the MIB
interface table, back up its network settings to a private area under the
the PDO software key before requesting reboot.

In a later boot, if the emulated device has been unplugged, restore the
network settings from the private area to the PDO such that it will start
with the same configuration as the emulated device it has replaced.

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

Stop synthesizing the NET_LUID

From Windows 10 the lifetime of the registry keys used to synthesize the
PDO's NET_LUID has changed. Instead of using a synthesized NET_LUID, look it
up in the MIB interface table (keyed by the permanent physical address)

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

Move reboot request code into driver.c

It doesn't really belong in pdo.c

Also this patch adds a check in PdoStartDevice() to fail if a reboot has
already been requested. This is because Windows 10 apparently has a couple
of goes at starting the PDO even after it failed to start the first time
(because we requested a reboot).

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

Only open the PDO software key once

There are currently two functions where the PDO software key is opened and
closed: __PdoSetCurrentAddress() and __PdoSetLuid(). Both of these
functions are called from PdoStartDevice() so it's more efficient to open
the software key there and pass it as an argument to the aforementioned
functions.

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

Infer REG_SZ or REG_MULTI_SZ type in RegistryUpdateSzValue()

The argument is a pointer to an array of ANSI_STRINGs in both cases so the
type can easily be inferred from the length of the array.

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

Add a type argument to the RegistryEnumerateValues() callback...

...and pass ANSI_STRINGs rather than PCHARs.

Also fix both RegistryEnumerateSubKeys() and RegistryEnumerateValues() to
not clobber their own buffer sizes when calling ZwEnumerateKey() or
ZwEnumerateValueKey() respectively.

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

Add functions to query and update binary values in the registry

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

Remove use of 'Addresses' subkey

XENVIF writes the permanent address of each vif into a subkey under its
service key. This is done to allow XENNET's co-installer to copy network
settings from any emulated device that currently has that address.

As of Windows 10, this functionality cannot be provided by a co-installer
and so this subkey has become useless. Hence this patch removes its use.

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

Remove erroneous use of transmitter DPC

If the backend does not support split event channels then the transmitter
DPC should not be used, and it contains an assertion to that effect.
The ring enable function, however, unconditionally queues the DPC leading
to an assertion failure. This patch makes sure the DPC is only queued if
the backend does indeed support split event channels.

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

Dispense with long PDO descriptions

Windows does not present them well and they are somewhat ugly. The relation-
ship between interface versions and PDO revision is now simply logged.

Also this patch applies appropriate limits to the HardwareIDs and
CompatibleIDs.

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

Release and re-acquire backend on resume

When resuming from suspend, any previously cached backend domid and path are
invalid. The frontend code should therefore release and re-acquire those
values before attempting to manipulate vif state.

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