From: Naman Jain <namjain@linux.microsoft.com>
To: "K . Y . Srinivasan" <kys@microsoft.com>,
Haiyang Zhang <haiyangz@microsoft.com>,
Wei Liu <wei.liu@kernel.org>, Dexuan Cui <decui@microsoft.com>
Cc: linux-hyperv@vger.kernel.org, linux-kernel@vger.kernel.org,
Naman Jain <namjain@linux.microsoft.com>,
John Starks <jostarks@microsoft.com>,
jacob.pan@linux.microsoft.com,
Easwar Hariharan <eahariha@linux.microsoft.com>,
Michael Kelley <mhklinux@outlook.com>,
Saurabh Singh Sengar <ssengar@linux.microsoft.com>
Subject: [PATCH v2 0/2] Drivers: hv: vmbus: Wait for offers and log missing offers
Date: Tue, 29 Oct 2024 01:01:45 -0700 [thread overview]
Message-ID: <20241029080147.52749-1-namjain@linux.microsoft.com> (raw)
After VM requests for channel offers during boot or resume from
hibernation, host offers the configured channels and then sends a
separate message when all the essential channel offers are delivered.
Wait for this message to make this offers request and receipt process
synchronous.
Without this, user mode can race with VMBus initialization and miss
channel offers. User mode has no way to work around this other than
sleeping for a while, since there is no way to know when VMBus has
finished processing offers.
This is in analogy to a PCI bus not returning from probe until it has
scanned all devices on the bus.
As part of this implementation, some code cleanup is also done for the
logic which becomes redundant due to this change.
Second patch prints the channels which are not offered when resume
happens from hibernation to supply more information to the end user.
Changes since v1:
https://lore.kernel.org/all/20241018115811.5530-1-namjain@linux.microsoft.com/
* Added Easwar's Reviewed-By tag
* Addressed Michael's comments:
* Added explanation of all offers delivered message in comments
* Removed infinite wait for offers logic, and changed it wait once.
* Removed sub channel workqueue flush logic
* Added comments on why MLX device offer is not expected as part of
this essential boot offer list. I refrained from adding too many
details on it as it felt like it is beyond the scope of this patch
series and may not be relevant to this. However, please let me know if
something needs to be added.
* Addressed Saurabh's comments:
* Changed timeout value to 10000 ms instead of 10*1000
* Changed commit msg as per suggestions
* Added a comment for warning case of wait_for_completion timeout
* Added a note for missing channel cleanup in comments and commit msg
John Starks (1):
Drivers: hv: vmbus: Log on missing offers
Naman Jain (1):
Drivers: hv: vmbus: Wait for offers during boot
drivers/hv/channel_mgmt.c | 55 ++++++++++++++++++++++++++++-----------
drivers/hv/connection.c | 4 +--
drivers/hv/hyperv_vmbus.h | 14 +++-------
drivers/hv/vmbus_drv.c | 31 +++++++++++-----------
4 files changed, 61 insertions(+), 43 deletions(-)
base-commit: 6fb2fa9805c501d9ade047fc511961f3273cdcb5
--
2.34.1
next reply other threads:[~2024-10-29 8:02 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-10-29 8:01 Naman Jain [this message]
2024-10-29 8:01 ` [PATCH v2 1/2] Drivers: hv: vmbus: Wait for offers during boot Naman Jain
2024-10-30 20:31 ` Easwar Hariharan
2024-11-05 5:40 ` Naman Jain
2024-10-31 19:13 ` Michael Kelley
2024-11-05 5:40 ` Naman Jain
2024-10-29 8:01 ` [PATCH v2 2/2] Drivers: hv: vmbus: Log on missing offers Naman Jain
2024-10-31 19:14 ` Michael Kelley
2024-11-07 5:44 ` Naman Jain
2024-11-11 5:43 ` Naman Jain
2024-11-12 3:13 ` Michael Kelley
2024-11-13 8:47 ` Naman Jain
2024-11-13 15:26 ` Michael Kelley
2024-11-14 6:53 ` Naman Jain
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20241029080147.52749-1-namjain@linux.microsoft.com \
--to=namjain@linux.microsoft.com \
--cc=decui@microsoft.com \
--cc=eahariha@linux.microsoft.com \
--cc=haiyangz@microsoft.com \
--cc=jacob.pan@linux.microsoft.com \
--cc=jostarks@microsoft.com \
--cc=kys@microsoft.com \
--cc=linux-hyperv@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mhklinux@outlook.com \
--cc=ssengar@linux.microsoft.com \
--cc=wei.liu@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.