[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
RE: [PATCH v2] Limit the amount of work done for each Receiver DPC
- To: "paul@xxxxxxx" <paul@xxxxxxx>, Owen Smith <owen.smith@xxxxxxxxxx>
- From: Martin Harvey <martin.harvey@xxxxxxxxxx>
- Date: Mon, 6 Jun 2022 12:21:50 +0000
- Accept-language: en-US
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
- Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; 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=+le76VFoSTOAHMb8FiLC3lBSCpq+3sNAwa6Ukz31XyI=; b=POai5cLF1BWSjvtYU/FzZn9TvrMnS62f4ygxCpsQjGtVH1JAjMdqntqoTEzaCCCAWfGbQegn1HtJbkcG/0DH+5LlHJFRsQwnjFm/iklo7ds0nihlUO3/Qiy9lFrWsqafHHlAs1+RFvNwveTcGFKJzQ23NIxGNrvZ5vi33bjLMP6qLOs+PtwEKGw05WXr1LeFXb0ownamuBkTuSU/BlDaAj6plCwy8e9+x2Lw2HFcwBXjLJ6WrhVx7MPhH/8lso/4DOuZkDII5AwhFJPvB+KhCXGmvPNgVFJa7B6r76YM67gRmGSDKY9GXKOgDxtG9yQhtJ4Pl0i7gAq3lWgM4rW6Rw==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=C5RnE2booTMZLclbNmMp7i9l/eCmXUtfPfLYjfxPnsyFgRx3PX7Yd0Er6twpemj71o29VkYAhoe11wJfIdAvzDhYKXxypTEdKLTNK7e8gOD21EUkSsO9RT8aHvzuKhAVn292cawGw6O4GpJ25KdI6XCIWgJz87HH+3WTRSHaaKeiuAK/SQke9JzucIyu9SrXn1423aBkHEo95ibLLyTAq9sV+cs5gsXoLIRxg5CrKbr2qrtiLDl8ReAG31P85t5lTzFOQWs2YUmnIqjtziaNbuSbCeiOxerHlEoZRDSQIPJB0/N5wkJnruzOBhAhTTyY6hxbdR3rTaQvalhH+BW2AA==
- Authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=citrix.com;
- Cc: "win-pv-devel@xxxxxxxxxxxxxxxxxxxx" <win-pv-devel@xxxxxxxxxxxxxxxxxxxx>
- Delivery-date: Mon, 06 Jun 2022 12:21:59 +0000
- Ironport-data: A9a23:jTCs2KOEZkLkkITvrR3HlsFynXyQoLVcMsEvi/4bfWQNrUpz02BVz GAbW26OOa3cazOjLowlbtizp01UsZDXzdU2QQto+SlhQUwRpJueD7x1DKtR0wB+jCHnZBg6h ynLQoCYdKjYdleF+lH1dOKJQUBUjclkfJKlYAL/En03FFYMpBsJ00o5wbZn2tMw27BVPivW0 T/Mi5yHULOa82Yc3lI8s8pvfzs24ZweEBtB1rAPTagjUG32zhH5P7pGTU2FFFPqQ5E8IwKPb 72rIIdVXI/u10xF5tuNyt4Xe6CRK1LYFVDmZnF+A8BOjvXez8CbP2lS2Pc0MC9qZzu1c99Z5 9JXt4GydQISF7DNs8Q7cil2ARtYIvgTkFPHCSDXXc276WTjKiKp79AwSUY8MMsf5/p9BnxI+ boAMjcRYxufhuWwhrWmVu1rgcdlJ87uVG8dkig4kXeFUrB7HtaaHPiiCdxwhV/cguhLFOzCZ sxfaTN1chnRSxZOJk0WGNQ1m+LAanzXLGcJ8w7M+fZfD2778yMo0qbSbsfvI9WTdJh0ol2mr ULs8DGsav0dHJnFodafyVq0j/LLtTP2XsQVDrLQ3uFrhhiL2CkfBQMbUXO/oOKlkQiuVtRHM UsW9yEy668o+ySWosLVWhS5pDuIu0AaUt8JSukisljSl+zT/hqTAXUCQnhZcts6ucQqRDssk FiUg9fuAj8pu7qQIZ6AyoqpQfqJEXB9BQc/ieUsFGPpP/GLTFkPsy/y
- Ironport-hdrordr: A9a23:QmUJrqHrf9wtgBwvpLqFQ5HXdLJyesId70hD6qkvc3Fom52j/f xGws5x6fatskdrZJkh8erwW5VoMkmsj6KdhrNhcItKPTOW8ldASbsP0WKM+UyGJ8STzI9gPO JbAtBD4b7LfBRHZKTBkW+F+r8bqbHpnpxAx92utkuFJjsaCZ2Imj0JbjpzZXcGITWua6BYKL Osou584xawc3Ueacq2QlMfWfLYmtHNnJX6JTYbGh8O8mC1/H2VwY+/NyLd8gYVUjtJz7tn23 PCiRbF6qKqtOz+4gPA1lXU849dlLLau5p+7Y23+4gowwfX+0SVjbdaKvi/VfcO0aWSAWMR4Z rxStEbToNOAj3qDyeISFDWqnTdOX4VmgPfIBmj8DTeSIXCNU0HItsEioRDfhTD7U08+Nl6za JQxmqc84FaFBXagU3GlpH1vjxR5z2JSEAZ4JguZr1kIPgjQa4UqZZa8FJeEZ8GEi6/4Ic7EP N2BMWZ4PpNa1uVY33Qo2EqmbWXLz8ONwbDRlJHtt2e0jBQknw8x0wExNYHlnNF8J4mUZFL6+ nNL6wtnrBTSc0da757GY46MIOKI32IRQiJPHOZIFzhGq1CM3XRq4Tv6LFw/+2ucIxg9upHpH 0AaiIsiYcfQTOeNSTV5uw0zvnkehTMYR39jsdD+pN+prrwALL2LCzrciFdr/ed
- List-id: Developer list for the Windows PV Drivers subproject <win-pv-devel.lists.xenproject.org>
- Thread-index: AQHYdnXhM853CaX5dkS4JamzRTB2U607/02AgAYxeUCAABtgAIAAA1wAgAACyRA=
- Thread-topic: [PATCH v2] Limit the amount of work done for each Receiver DPC
-----Original Message-----
From: Durrant, Paul <xadimgnik@xxxxxxxxx>
Sent: 06 June 2022 13:09
To: Martin Harvey <martin.harvey@xxxxxxxxxx>; paul@xxxxxxx; Owen Smith
<owen.smith@xxxxxxxxxx>
Cc: win-pv-devel@xxxxxxxxxxxxxxxxxxxx
Subject: Re: [PATCH v2] Limit the amount of work done for each Receiver DPC
> Yes, that is a mistake.
>
> I'll rebase the 2nd patch on top and let's see how that looks.
If the first is broken, the second is unlikely to be any better...
Promise the original patches were well tested, and have been quite reliable.
Totally happy with variable renames and cosmetic changes, but some of the logic
to do with backpressure and flushing deliberately overlaps from one DPC to the
next, and it's not possible to simplify it into "state depends only on this
dpc".
What do you want to achieve here, and what's the best way of going about it?
MH.
|
