From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from DU2PR03CU002.outbound.protection.outlook.com (mail-northeuropeazon11021117.outbound.protection.outlook.com [52.101.65.117]) by lore.virtuozzo.com (Postfix) with ESMTPS id 849E1802E5 for ; Fri, 4 Sep 2026 19:05:07 +0000 (UTC) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=f8QxoXNBmMQIujntU9Xe3ElwV0Q8omM6VGxVRNBb0jPyFYT3DSA0JUmcuXzYPmw0c2MrmfWP4c9JcyRW1XAS59s2Y58l/enaBpKItXIaD/MWdyBlAiVnXoxIUpwVOCQRtW3L8kAG+3yKVBqILKONFL19FTSMSPtfLVnu97Fh5bnzsP+9MoQK/8IKjfr2AdDprcXSJxtjSrbCo8JG6It++WHprp5CfETui2cOLfRShDKPnlg3vidYep4EXJBd/Z5tAIrhNSsX6VmPsoRXalF7OYDeZ+4KmZW0jB1s9QYjVQHglzSY/H/0AxomAv5lW80DSuOHqYDR68V5/njANRYHFA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=bbQSGwWUukjoZSuKXuwafYtFSe29cWc/+4CcRMfwddc=; b=blXN3h5Xhf4ZRB9peYhICsOrbsMfaQ8DUeMjeaUMe2cNoTsIJtiJT1DSH429HBZWzogJVWjZj5ndPUnC3zWJxT3hAwXw93WMEQU1sJC0a/LhgPk5Gxu3GQ5R6Gk4fB5Hw68o8GOdHjJp8AVDsln1uouKv13nhw5opAUCWupfAnv/cZ8auY7DVQLdfbjB2uSqIRiPzOXXxi+fjDreVFrdRpdxzBS1eVpuC8oPh9eRvbdYGZCdqARs4eanjsZbDDoaufKrxT/DJnnbVpWQ713IRhLUsFuoN/6c/eEhgj1WPgmElznHPhkowZpP0ZylkPKfG2pQKRplYWqbKI8wgQdxBw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=virtuozzo.com; dmarc=pass action=none header.from=virtuozzo.com; dkim=pass header.d=virtuozzo.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=virtuozzo.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=bbQSGwWUukjoZSuKXuwafYtFSe29cWc/+4CcRMfwddc=; b=z3qirYWfncgUIEJ0qHmBwarUKVVZJYOxr4H7afasEJNtwOho4bq/kIoWwSKqt70lIU405Q+yGusGGb4jh8L1KUSFauMaEK41owzTIH1sxSpXPHaIws+BlKlnJUW76iNQpxA4v+tjBaLHVd/Ys+546D2NW4JeoQcEjjtbNrUwXo+PgD8Pc3T0+VOMTYoxmQ/ba69a5rFOJgidxB2S4sgNK7cT8T7xEcZIpLxrAh2lTLQptrvXtd/EioTiwLZQpJvSqpv+9L5RdHuDjUfnoZUsqS0Vw2N3jBNkxyHmXVWFyvVPd0Vw+iQ4lWiU1JUWl2jOw7n7dBtjBgL77m/L0XwwDw== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=virtuozzo.com; Received: from DBBPR08MB10650.eurprd08.prod.outlook.com (2603:10a6:10:52d::7) by PR3PR08MB5756.eurprd08.prod.outlook.com (2603:10a6:102:90::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.21.382.10; Fri, 4 Sep 2026 19:05:05 +0000 Received: from DBBPR08MB10650.eurprd08.prod.outlook.com ([fe80::1b2a:832c:d136:a37c]) by DBBPR08MB10650.eurprd08.prod.outlook.com ([fe80::1b2a:832c:d136:a37c%5]) with mapi id 15.21.0382.007; Fri, 4 Sep 2026 19:05:05 +0000 From: Andrey Drobyshev To: svt-core@virtuozzo.com Cc: andrey.drobyshev@virtuozzo.com, den@openvz.org Subject: [QEMU HCI-8.0 PATCH v2 11/13] usb-host: reconfigure endpoints on CPR incoming #VSTOR-137800 Date: Fri, 4 Sep 2026 22:04:41 +0300 Message-ID: <20260904190443.795902-12-andrey.drobyshev@virtuozzo.com> X-Mailer: git-send-email 2.47.1 In-Reply-To: <20260904190443.795902-1-andrey.drobyshev@virtuozzo.com> References: <20260904190443.795902-1-andrey.drobyshev@virtuozzo.com> Content-Transfer-Encoding: 8bit Content-Type: text/plain X-ClientProxiedBy: VI1PR07CA0215.eurprd07.prod.outlook.com (2603:10a6:802:58::18) To DBBPR08MB10650.eurprd08.prod.outlook.com (2603:10a6:10:52d::7) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DBBPR08MB10650:EE_|PR3PR08MB5756:EE_ X-MS-Office365-Filtering-Correlation-Id: 8354795a-cd34-4c1f-3a6f-08df0ab76b5a List-Id: svt-core@virtuozzo.com X-LD-Processed: 0bc7f26d-0264-416e-a6fc-8352af79c58f,ExtAddr X-MS-Exchange-Transport-Rules-Loop: 1 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|1800799024|366016|23010399003|376014|52116014|18002099003|22082099003|5113699003|56012099006|10067099003|6133799003|5023799004|38350700014; X-Microsoft-Antispam-Message-Info: Mz6Bff+FJISc41hOgqYz2e8Xb0gychO+mhbDx5Kdy991gQuCkk/X3JGpEi5JFuT3e2z09JyBEmb+cdCRjSsPGQj3YZrqG3sYcBb70yqEoQhpDCdDjKkmm+tgRl6LU5ipiezdxmAwSyX4rEvmlKKPyrFQDgyKWNaEnIXoF7qvkSmHz7wpJmeP7iXgfSVVt2nUElVWKa1JnXWjWuCs8QlEeqNsftgYIZR6iXlLW0YVwotb87ShmSGuu7wa2whShxlsYNEak/B0s4DZBrJXDLxodbHUUUOftOV3JBz2SZtAgFTIylSxYD+2ySMbXsSRXlYUj0mrNaR5F3s2S5DtzESSFiMRrxJLMm2IQ2nP3/SHTOetmG6X/WYW7bae4AcFOIlYbgxhspNvS5yZPPEkoJTBOXg+cJ9HhPqNqXY2Kdykp8iaFdVs4W6C2F8doEqF8wADk4itT3GmNZdebXnpwDNfOFLYW9pUOEIvX7unAWRXmspo7NSSXTeh4vIOi/eMPRrDeLDoFOxpDa0d1KI7lwZFG6Gjlle2CUukiD9PIj0LJnO/5CuBwjCRDcGPLMPHsskNwMi53bSSiF8OEgkN7dbH9GVqnH4d+p8smNoARsBDsF8acHCkqmautZp5qB29yIcpQNvD8sFXUibNg06WFpisx5Ep5/kzS04wY7uCN9iRdrqy2T2WaDYtbYcVkOPcNtxnXUFerfTzhT4gjkXJutGgmgCcvnITs8odapPOSOGbh4c= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:DBBPR08MB10650.eurprd08.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(1800799024)(366016)(23010399003)(376014)(52116014)(18002099003)(22082099003)(5113699003)(56012099006)(10067099003)(6133799003)(5023799004)(38350700014);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?arFS8fXQTIPiGYbV/XPnVfSRUyiwannscPDw7G7ZCodUdpMY3hvBL3KzS49s?= =?us-ascii?Q?HdgRRoL+flfkBeHnEhLgH57MZl5xRNzKAMvQPVISNoLR3+2YCvP+aBzrOp43?= =?us-ascii?Q?cgf/xHz3kPhKy2LgZ+LPPKq+nb72p1Pk2tSfLDkRIXab427VRuuqTOVo5oyQ?= =?us-ascii?Q?4nqtfDG6ywZGPtJvYGSh1Lctiw6at1BXAsBa/CjSeVFJaURJVrCeqz8ZX70m?= =?us-ascii?Q?uiLsZKgH8mE6AYu3d/M9gm4Zk9JYc3FNRe884Xzv1KE/nG30q+BEtpNU1d2y?= =?us-ascii?Q?pXb78gGTztVImhgG8LslUb6BOwDCmhANA+caSpycxX2oYx78bpkI/v+l79gR?= =?us-ascii?Q?qto9a21S+leApXDO5d4VNP9QWGSLL2qQLTjRXQ1qAIXEr8B2xZIkp/gfjhsv?= =?us-ascii?Q?emk13pGkjE23dB0sMlGLEygVsozhNr4fUTTIDEOj1flG0p79RgtcKkHfGp7g?= =?us-ascii?Q?ww24yFn9YG2IpXIqs+ZfUaFcoBREzA+g1BfUAlfWkREjeUsZoSdOQKFQaUom?= =?us-ascii?Q?GN6H9fb8SaDiI1cbqi4JR1uZjwhzQ8srKdr7fES7a8tTrd4s7AeO+5624u6p?= =?us-ascii?Q?2vkvchwQbRyQ5QpGpjz8e+AktoT4dolFTvcq2axALiwzbIk3i5TXHhAdNay+?= =?us-ascii?Q?o0P+J7CAinK20mxBdkWYaXugaoW7EuqwLQbHxGO8mO2OA9APur1CPBdbcpPp?= =?us-ascii?Q?a5DDdxVlKp21JklBfsJtPg0ct4mxLjMxiEf34CyBI4eI8H/5ftlgMhXN9yZR?= =?us-ascii?Q?0ZFQZgAvZRaoofRHNMq8PNLgmCd2sX5A+XIGUISX+cTia9R5xq2l94UGQa2Q?= =?us-ascii?Q?1N3COJTYXkXavydls6waAwu9bB9LXIUx/C30Qmd14tHwqate8cWNA17eXL97?= =?us-ascii?Q?taDzXUBmPbn0M9Qo/5C3IMPrM+py3VmS4B3fIgy6Z5AewQ6TcfDlUnzK9mt0?= =?us-ascii?Q?OJpunwFmep0gQewpIp6zzgsEIZmR4MP0G91A1e4baKbgCz4njBc0t5PLDz51?= =?us-ascii?Q?u8mNOAuFZZlmKN3wF0u987xKdPWMLARl45cQly8ehttRP/wx5j88FOiVOf9X?= =?us-ascii?Q?4wr1QHkREbrGtLr/6DeeznD3lz8OFaDFx4x4bEauLbr/R8IFVz0K89Jg+Nb4?= =?us-ascii?Q?FQFiJt8fkF8rlBuJdlcjSWDZlcIhygE0049yT6c6NXF/ak7nlSEbexYOpnP0?= =?us-ascii?Q?SZaiE8qz4XvoB53bK+2ydp1KdJUiVMHzdyZri3W1kpArBoZJtTFcfb+gItb8?= =?us-ascii?Q?yknsOsXm2vJJXHXxkU2PK+uKXT/s0FZPjdn4hOoFFUAxo+yRz6ikTuj2lNCL?= =?us-ascii?Q?2iZ2/U3+6qwdrGBuItDgijOkIiAhpuFSE8BXVxWu5jJz5Fwg5azqVWvV5vFV?= =?us-ascii?Q?mBEB3saMm3INAwAG/j4tOZQJcQUoUQXpvuJQw0MS2wKaQOfTJG9xAVxD7WmB?= =?us-ascii?Q?H2UM24yRwnydEofWLK+OAXpOd5nKl31Y0N+s5ikAvXcWlfI2wrn2B0DV4sEs?= =?us-ascii?Q?dGUjU3JQOAl890RvfCCP403MxAmNtt1OAGitLmh8waQVuzbyCDfbl2lA3s7B?= =?us-ascii?Q?oeMofxmTosrUSYSy/qncrvUN44okxqxrJFLbnooFvMMza0YIDDUYyPrDyjDt?= =?us-ascii?Q?KkGblV/3hyc/i6CcqzMIwJH3WrT9PVrilhrKxNOXMoU/hYY8PaEVuuL4G3tI?= =?us-ascii?Q?sB6rBsvDXc7lcea1QeP5ojOUm8nakDE6oUgnkF6CvFnz+gbinc3l2mavzJvT?= =?us-ascii?Q?v9xyFnMqSSi7+E1jgMHLaRQJhckCX/s=3D?= X-OriginatorOrg: virtuozzo.com X-MS-Exchange-CrossTenant-Network-Message-Id: 8354795a-cd34-4c1f-3a6f-08df0ab76b5a X-MS-Exchange-CrossTenant-AuthSource: DBBPR08MB10650.eurprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Sep 2026 19:05:00.8675 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 0bc7f26d-0264-416e-a6fc-8352af79c58f X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 9DjuxV4YcOMrItA2IEc7aGWQv41EudI7hafTRO2BrxuFNWG2jAdKZs0aOYfd75cePjifjZ3/K4oAyiRuXizDkWKvi+9ZF5k2uC+nelYSe4k= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PR3PR08MB5756 On CPR incoming we skip the usb_host_open() rescan, so libusb only sets up the endpoints from the device's default configuration. If the guest had enabled an endpoint by selecting a non-zero altsetting (e.g. an isochronous streaming endpoint), that endpoint is now missing, and the guest's transfers to it simply stall. The altsetting itself is restored by vmload now, so let's call usb_host_ep_update() on the CPR target to rebuild the endpoints out of it. The rescan is also the only place where we claim the interfaces in libusb. The kernel claims are still there on the preserved FD, but libusb's own handle knows nothing about them, so a later SET_INTERFACE would fail with LIBUSB_ERROR_NOT_FOUND. Let's re-claim the interfaces on the target as well to bring libusb's view back in sync - on the preserved FD this is a no-op at the usbfs level anyway. Signed-off-by: Andrey Drobyshev --- hw/usb/host-libusb.c | 35 +++++++++++++++++++++++++++++++++++ 1 file changed, 35 insertions(+) diff --git a/hw/usb/host-libusb.c b/hw/usb/host-libusb.c index 59f2b151cd2..d12e04c2768 100644 --- a/hw/usb/host-libusb.c +++ b/hw/usb/host-libusb.c @@ -1956,6 +1956,34 @@ static void usb_host_post_load_bh(void *opaque) usb_host_auto_check(NULL); } +/* + * On the CPR target libusb knows nothing about the interface claims the + * preserved fd still holds, so a later SET_INTERFACE would fail with + * NOT_FOUND. Re-claim them to bring libusb's view back in sync; on the + * preserved fd it's a no-op at the usbfs level anyway. + */ +static void usb_host_cpr_reclaim_interfaces(USBHostDevice *s) +{ + USBDevice *udev = USB_DEVICE(s); + struct libusb_config_descriptor *conf; + int i, n; + + if (libusb_get_active_config_descriptor(s->dev, &conf) != 0) { + return; + } + + n = MIN(conf->bNumInterfaces, USB_MAX_INTERFACES); + for (i = 0; i < n; i++) { + if (libusb_claim_interface(s->dh, i) == 0) { + s->ifs[i].claimed = true; + } + } + + udev->ninterfaces = conf->bNumInterfaces; + udev->configuration = conf->bConfigurationValue; + libusb_free_config_descriptor(conf); +} + static int usb_host_post_load(void *opaque, int version_id) { USBHostDevice *dev = opaque; @@ -1967,6 +1995,13 @@ static int usb_host_post_load(void *opaque, int version_id) * Skip it for CPR. */ if (cpr_is_incoming()) { + /* + * We kept the device open across CPR, so libusb still needs its + * interface claims and endpoints rebuilt here. And we only start + * reaping now, once the source has handed the fd off to us. + */ + usb_host_cpr_reclaim_interfaces(dev); + usb_host_ep_update(dev); usb_host_cpr_restart_events(); return 0; } -- 2.47.1