[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH] Use STATUS_PNP_REBOOT_REQUIRED in IRP_MN_START_DEVICE when a reboot is required
- To: <win-pv-devel@xxxxxxxxxxxxxxxxxxxx>
- From: Owen Smith <owen.smith@xxxxxxxxxx>
- Date: Mon, 21 Mar 2022 14:23:07 +0000
- Authentication-results: esa6.hc3370-68.iphmx.com; dkim=none (message not signed) header.i=none
- Cc: Owen Smith <owen.smith@xxxxxxxxxx>
- Delivery-date: Mon, 21 Mar 2022 14:23:21 +0000
- Ironport-data: A9a23:yh7m46MvltG1ZKfvrR30l8FynXyQoLVcMsEvi/4bfWQNrUor1jUGn 2NOX2iPOfaMa2T8eN9/b4qy9BtUsZDcn4AwQQto+SlhQUwRpJueD7x1DKtR0wB+jCHnZBg6h ynLQoCYdKjYdleF+lH1dOKJQUBUjclkfJKlYAL/En03FFcMpBsJ00o5wbZi2Ncw2LBVPivW0 T/Mi5yHULOa82Yc3lI8s8pvfzs24ZweEBtB1rAPTagjUG32zhH5P7pGTU2FFFPqQ5E8IwKPb 72rIIdVXI/u10xF5tuNyt4Xe6CRK1LYFVDmZnF+A8BOjvXez8CbP2lS2Pc0MC9qZzu1c99Z8 MV9pbC+c1wSMJLlntw0a0dINSdGMvgTkFPHCSDXXc27ykTHdz3nwul0DVFwNoodkgp1KTgQr 7pCcmlLN03dwbLtqF64YrAEasALK9bmOsUEv3B8zRnSDOo8QICFSKLPjTNd9Glu2JAeRK6CD yYfQRBxdR7ESEx9A1xNWaNgxsKQwULhQiIN/Tp5ooJoujOOnWSdyoPFL9vTP8SUAMlYgEucj mbH5HjiRAEXMsSFzjiI+W7qgfXA9R4XQ6pLSuf+rKQzxgTOmCpDU3X6SGdXv9GBmn6ud9xlc HcaxQMolYEKqlyNSOHyCkjQTGG/gjYQXN9ZEusf4Q6Ly7bJ7wvxOlXoXgKte/R96pZoGGVCO kuh2oqwWGcx6OH9pWe1rO/8kN+kBcQCwYbujwcgRBBN3dTsqZpbYvnnHoc6S/7dYjEY9FjNL 9G2QMoW2u57YS0jjfzTEbX7b9WE/MShc+LNzl+LNl9JFysgDGJfW6Sm6ELA8dFLJ5uDQ1+Ks RAswpbCs7BXVcjUxHXWEY3h+Y1FAN7fbVXhbaNHRcF9p1xBBVb9FWyv3N2ODBgwaZtVEdMYS ETSpRlQ9Pdu0IiCNsdKj3aKI51yl8DITI29PtiNN4YmSsUhJWevoXA1DWbNjj+FraTZufxmU XttWZ33Vihy5GUO5GfeetrxJpdwn3FgnzyPHc6np/lluJLHDEOopX4+GAPmRogEAGms+W05L /432xO29ihi
- Ironport-hdrordr: A9a23:HSWt/qkrf/KAL7d+H7C3KDYV+rfpDfIm3DAbv31ZSRFFG/Fxl6 iV88jzsiWE7Qr5OUtQ/uxoV5PgfZqxz/NICOoqTNWftWvd2FdARbsKheCJ/9SJIVybygc378 ldmsZFZOEYdWIK7vrH3A==
- List-id: Developer list for the Windows PV Drivers subproject <win-pv-devel.lists.xenproject.org>
Failing IRP_MN_START_DEVICE can lead to WHQL distribution of drivers to be
rejected by MS, as too many VMs in the gradual rollout phase will report this
error on update. Using STATUS_PNP_REBOOT_REQUIRED will indicate that the VM
requires a reboot, and should not be treated as a gating factor (it should also
trigger the Windows Update process to report the reboot to users and/or trigger
an automatic reboot)
Signed-off-by: Owen Smith <owen.smith@xxxxxxxxxx>
---
src/xenvif/pdo.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/xenvif/pdo.c b/src/xenvif/pdo.c
index bc9b9ed..b3a4aae 100644
--- a/src/xenvif/pdo.c
+++ b/src/xenvif/pdo.c
@@ -1303,7 +1303,7 @@ PdoStartDevice(
if (Pdo->HasAlias) {
PdoUnplugRequest(Pdo, TRUE);
- status = STATUS_UNSUCCESSFUL;
+ status = STATUS_PNP_REBOOT_REQUIRED;
goto fail9;
}
--
2.33.0.windows.2
|