Skip to content

examples/power: allow use of more than 64 cores

David Hunt dpdk-dev 3 versions – Applied

Status from this message: Applied, thanks

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v3 Latest4 patches 2 repliesApplied
  2. v24 patches 4 repliesSuperseded
  3. v14 patches 8 repliesSuperseded

Changes in v3 from the cover letter

* Moved a spinlock to a more appropriate place. Was in a for-loop, made more sense to have it outside.

Patches in v3 Applied

#Subject and filesSizeRepliesApplied
1/4examples/power: change 64-bit masks to arraysexamples/vm_power_manager/channel_manager.c examples/vm_power_manager/channel_manager.h examples/vm_power_manager/vm_power_cli.c+55 -420 replies
2/4examples/power: remove mask functionsexamples/vm_power_manager/channel_monitor.c examples/vm_power_manager/power_manager.c examples/vm_power_manager/vm_power_cli.c+0 -2410 replies
3/4examples/power: allow vms to use lcores over 63examples/vm_power_manager/channel_manager.c examples/vm_power_manager/channel_manager.h examples/vm_power_manager/channel_monitor.c +1 more+57 -1060 replies
4/4examples/power: increase max cores to 256examples/vm_power_manager/channel_manager.h examples/vm_power_manager/power_manager.h+3 -70 replies

Reviews trailers per patch

1 of 4 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchAnatoly Burakov
SeriesR-b
1/4
2/4
3/4
4/4R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v3 thread

Cover letter [PATCH v3 0/4] examples/power: allow use of more than 64 cores

First of all we convert all the relevant uint64_t's to char arrays. Then we remove the unneeded mask functions that were limited to 64 cores. Also extend the guest functionality and finally rause the number of supported cores to something more sensible, i.e. 256.

v2 changes:
* Updates out of Review by Anatoly.
* Indentation fixes
* resolved missing spinlock around config updates
* simplified some loops
* fixed bug in non-equivalent code

v3 changes:
* Moved a spinlock to a more appropriate place. Was in a for-loop, made more sense to have it outside.

[1/4] examples/power: change 64-bit masks to arrays [2/4] examples/power: remove mask functions [3/4] examples/power: allow vms to use lcores over 63 [4/4] examples/power: increase max cores to 256