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

Jump to bottom

VPCI code cleanup #2431

Closed
donsheng opened this issue Jan 25, 2019 · 2 comments
Closed

VPCI code cleanup #2431

donsheng opened this issue Jan 25, 2019 · 2 comments
Assignees
@donsheng
Labels
priority: low status: new The issue status: new for creation type: feature New feature

Comments

@donsheng
Copy link
Contributor

donsheng commented Jan 25, 2019 •
edited
Loading

Changes include:

  • Scan all physical PCI devices and store all needed info in global pdev array
  • Use the cached pci device (pdev) info for sharing mode and partition mode
  • Simply vm_description for pci passthru, only vbdf and pbdf need to be specified by user
  • vdev stores a pointer to pdev

[External_System_ID] ACRN-2601
@donsheng donsheng mentioned this issue Jan 28, 2019
Closed
@Mingyuan18 Mingyuan18 assigned dongyaozu and donsheng and unassigned dongyaozu Jan 28, 2019
@Mingyuan18 Mingyuan18 added priority: low type: feature New feature status: new The issue status: new for creation labels Jan 28, 2019
@donsheng donsheng mentioned this issue Feb 2, 2019
Merged
@acrnsi
Copy link

acrnsi commented Feb 2, 2019

[External_System_ID] ACRN-2601

acrnsi pushed a commit that referenced this issue Feb 2, 2019
@donsheng @acrnsi
HV: move struct pci_bar and pci_pdev to pci.h
65f9370 
Tracked-On: #2431
Signed-off-by: dongshen <dongsheng.x.zhang@intel.com>
Reviewed-by: Anthony Xu <anthony.xu@intel.com>
acrnsi pushed a commit that referenced this issue Feb 2, 2019
@donsheng @acrnsi
HV: scan all physical PCI devices and store all needed info in array
e0f9d14 
Tracked-On: #2431
Signed-off-by: dongshen <dongsheng.x.zhang@intel.com>
Signed-off-by: Victor Sun <victor.sun@intel.com>
Reviewed-by: Anthony Xu <anthony.xu@intel.com>
acrnsi pushed a commit that referenced this issue Feb 2, 2019
@donsheng @acrnsi
HV: use the cached pci device info for sharing mode
983b717 
Tracked-On: #2431
Signed-off-by: dongshen <dongsheng.x.zhang@intel.com>
Reviewed-by: Anthony Xu <anthony.xu@intel.com>
acrnsi pushed a commit that referenced this issue Feb 2, 2019
@jsun26intel @acrnsi
HV: init ptdev bar during runtime for partition mode
f082176 
Current pt devices bar info for partion mode is hardcoded in
vm_description.c, now we remove the hardcoded info and parse the bar
info during pt devices init.

Tracked-On: #2431
Signed-off-by: Victor Sun <victor.sun@intel.com>
Signed-off-by: dongshen <dongsheng.x.zhang@intel.com>
Reviewed-by: Anthony Xu <anthony.xu@intel.com>
acrnsi pushed a commit that referenced this issue Feb 2, 2019
@jsun26intel @acrnsi
HV: remove bar info in vm description
69d8835 
We have enabled bar decoding in init ptdev, so hard-coded bar info is not needed
any more.

Tracked-On: #2431
Signed-off-by: Victor Sun <victor.sun@intel.com>
Signed-off-by: dongshen <dongsheng.x.zhang@intel.com>
Reviewed-by: Anthony Xu <anthony.xu@intel.com>
acrnsi pushed a commit that referenced this issue Feb 2, 2019
@jsun26intel @acrnsi
HV: remove ptdev ops init in vm description
6ba07e6 
The ptdev ops will be initialized automatically during ptdev init, so
remove the hardcoded ops in vm_description.c

Tracked-On: #2431
Signed-off-by: Victor Sun <victor.sun@intel.com>
Signed-off-by: dongshen <dongsheng.x.zhang@intel.com>
Reviewed-by: Anthony Xu <anthony.xu@intel.com>
acrnsi pushed a commit that referenced this issue Feb 2, 2019
@donsheng @acrnsi
HV: store a pointer to pdev instead to save memory.
f4adb10 
As we store the physical pci device info in a global pdev array, the vdev struct can
store a pointer to pdev instead.

Tracked-On: #2431
Signed-off-by: dongshen <dongsheng.x.zhang@intel.com>
Reviewed-by: Anthony Xu <anthony.xu@intel.com>
@donsheng
Copy link
Contributor Author

Code merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@donsheng donsheng

Labels
priority: low status: new The issue status: new for creation type: feature New feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@dongyaozu @Mingyuan18 @donsheng @acrnsi