-
Notifications
You must be signed in to change notification settings - Fork 56
enable gdr on 10.2 baseline #431
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: v0.10.2_next
Are you sure you want to change the base?
enable gdr on 10.2 baseline #431
Conversation
Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Harish Subramony <[email protected]>
Signed-off-by: Harish Subramony <[email protected]>
🚧 CI BlockedThe main CI workflow was not started for the following reason:
|
Signed-off-by: Harish Subramony <[email protected]>
🚧 CI BlockedThe main CI workflow was not started for the following reason:
|
vllm_gaudi/distributed/kv_transfer/kv_connector/v1/hpu_nixl_connector.py
Show resolved
Hide resolved
| i = 0 | ||
| global hpu_buffer | ||
| use_hpu_buffer = False | ||
| use_hpu_buffer = False # (len(src_slot_mapping) == hpu_buffer[0][0].size(0)) and (hpu_buffer is not None) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove line 4323
| hpu_buffer[i][0]=key_cache.index_select_(0, src_slot_mapping) | ||
| hpu_buffer[i][1]=value_cache.index_select_(0, src_slot_mapping) | ||
| else: | ||
| #import remote_pdb;remote_pdb.set_trace() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove line 4348 to 4352,
Signed-off-by: Harish Subramony <[email protected]>
🚧 CI BlockedThe main CI workflow was not started for the following reason:
|
Signed-off-by: Harish Subramony <[email protected]>
🚧 CI BlockedThe main CI workflow was not started for the following reason:
|
Signed-off-by: Harish Subramony <[email protected]>
🚧 CI BlockedThe main CI workflow was not started for the following reason:
|
| src_block_ids=src_block_ids, | ||
| dst_block_ids=dst_block_ids, | ||
| src_device=src_device, | ||
| dst_device=dst_device) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
from 4320 - 4325, seems only needed for is_hetero false, then add condition
Signed-off-by: Harish Subramony <[email protected]>
🚧 CI BlockedThe main CI workflow was not started for the following reason:
|
Signed-off-by: Harish Subramony <[email protected]>
🚧 CI BlockedThe main CI workflow was not started for the following reason:
|
🚧 CI BlockedThe main CI workflow was not started for the following reason:
|
🚧 CI BlockedThe main CI workflow was not started for the following reason:
|
No description provided.