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.

Sign up for GitHub

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

Move allocated pci deletion in defer of del cmd #278

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Move allocated pci deletion in defer of del cmd #278

wants to merge 1 commit into from
+3 −6

Commits on Jul 26, 2023

  1. Move allocated pci deletion in defer of del cmd 

    The allocated pci is supposed to be deleted when the del command
is completed. However, placing this logic at the end of the function
does not always guaranteed its execution. For instance, if ResetVfConfig
or ReleaseVF fail, the err variable is overwritten which causes the cache
to be deleted in the defer function and the next del retry to exit
without deleting the allocated pci.
By placing this logic along the deletion of the cache will ensure that
all operations in the del command are done.
    @mlguerrero12
    mlguerrero12 committed Jul 26, 2023
    Configuration menu
    Copy the full SHA
    0cca275 View commit details
    Browse the repository at this point in the history