Skip to content
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

FR: support errno #46

Merged
merged 3 commits into from
Dec 14, 2023
Merged

FR: support errno #46

merged 3 commits into from
Dec 14, 2023

Conversation

yajwu
Copy link
Collaborator

@yajwu yajwu commented Nov 23, 2023

No description provided.

@yajwu yajwu force-pushed the FR_errno_new branch 2 times, most recently from 5d03e86 to e277162 Compare November 29, 2023 06:08
@yajwu yajwu force-pushed the FR_errno_new branch 5 times, most recently from 03b71f0 to 936679b Compare December 13, 2023 02:21
Define error number for virtio vdpa driver functions.
Unify all RPC return value as:

{
  "errno": 0,
  "errstr": "Success"
}

for fail:
{
  "errno": -1008,
  "errstr": "PF device already add"
}

Fix of device status resetting timeout no return error.

FR:3664517

Signed-off-by: Yajun Wu <yajunw@nvidia.com>
Fix wrong required features check for adding PF.
Also remove the use of rte_errno, rte_errno is changed in virtio_pci_dev_free.

Signed-off-by: Yajun Wu <yajunw@nvidia.com>
usleep(1000) seems take more time on executing code instead of sleep.
For 120s sleep, may take 125s to complete.

Change to usleep 100ms for temp fix.
@yajwu yajwu changed the title [DON'T MERGE] Fr errno new FR: support errno Dec 13, 2023
@kailiangz1 kailiangz1 merged commit 75be30e into Mellanox:main Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants