@@ -70,8 +70,6 @@ struct cpu_core {
struct ovs_list list_node; /* In 'numa_node->cores' list. */
struct numa_node *numa; /* numa node containing the core. */
unsigned core_id; /* Core id. */
- bool available; /* If the core can be pinned. */
- bool pinned; /* If a thread has been pinned to the core. */
};
/* Contains all 'struct numa_node's. */
@@ -119,7 +117,6 @@ insert_new_cpu_core(struct numa_node *n, unsigned core_id)
ovs_list_insert(&n->cores, &c->list_node);
c->core_id = core_id;
c->numa = n;
- c->available = true;
return c;
}
@@ -340,18 +337,6 @@ ovs_numa_core_id_is_valid(unsigned core_id)
return found_numa_and_core && core_id < ovs_numa_get_n_cores();
}
-bool
-ovs_numa_core_is_pinned(unsigned core_id)
-{
- struct cpu_core *core = get_core_by_core_id(core_id);
-
- if (core) {
- return core->pinned;
- }
-
- return false;
-}
-
/* Returns the number of numa nodes. */
int
ovs_numa_get_n_numas(void)
@@ -396,97 +381,6 @@ ovs_numa_get_n_cores_on_numa(int numa_id)
return OVS_CORE_UNSPEC;
}
-/* Returns the number of cpu cores that are available and unpinned
- * on numa node. Returns OVS_CORE_UNSPEC if 'numa_id' is invalid. */
-int
-ovs_numa_get_n_unpinned_cores_on_numa(int numa_id)
-{
- struct numa_node *numa = get_numa_by_numa_id(numa_id);
-
- if (numa) {
- struct cpu_core *core;
- int count = 0;
-
- LIST_FOR_EACH(core, list_node, &numa->cores) {
- if (core->available && !core->pinned) {
- count++;
- }
- }
- return count;
- }
-
- return OVS_CORE_UNSPEC;
-}
-
-/* Given 'core_id', tries to pin that core. Returns true, if succeeds.
- * False, if the core has already been pinned, or if it is invalid or
- * not available. */
-bool
-ovs_numa_try_pin_core_specific(unsigned core_id)
-{
- struct cpu_core *core = get_core_by_core_id(core_id);
-
- if (core) {
- if (core->available && !core->pinned) {
- core->pinned = true;
- return true;
- }
- }
-
- return false;
-}
-
-/* Searches through all cores for an unpinned and available core. Returns
- * the 'core_id' if found and sets the 'core->pinned' to true. Otherwise,
- * returns OVS_CORE_UNSPEC. */
-unsigned
-ovs_numa_get_unpinned_core_any(void)
-{
- struct cpu_core *core;
-
- HMAP_FOR_EACH(core, hmap_node, &all_cpu_cores) {
- if (core->available && !core->pinned) {
- core->pinned = true;
- return core->core_id;
- }
- }
-
- return OVS_CORE_UNSPEC;
-}
-
-/* Searches through all cores on numa node with 'numa_id' for an
- * unpinned and available core. Returns the core_id if found and
- * sets the 'core->pinned' to true. Otherwise, returns OVS_CORE_UNSPEC. */
-unsigned
-ovs_numa_get_unpinned_core_on_numa(int numa_id)
-{
- struct numa_node *numa = get_numa_by_numa_id(numa_id);
-
- if (numa) {
- struct cpu_core *core;
-
- LIST_FOR_EACH(core, list_node, &numa->cores) {
- if (core->available && !core->pinned) {
- core->pinned = true;
- return core->core_id;
- }
- }
- }
-
- return OVS_CORE_UNSPEC;
-}
-
-/* Unpins the core with 'core_id'. */
-void
-ovs_numa_unpin_core(unsigned core_id)
-{
- struct cpu_core *core = get_core_by_core_id(core_id);
-
- if (core) {
- core->pinned = false;
- }
-}
-
/* Given the 'numa_id', returns dump of all cores on the numa node. */
struct ovs_numa_dump *
ovs_numa_dump_cores_on_numa(int numa_id)
@@ -632,68 +526,6 @@ ovs_numa_dump_destroy(struct ovs_numa_dump *dump)
free(dump);
}
-/* Reads the cpu mask configuration from 'cmask' and sets the
- * 'available' of corresponding cores. For unspecified cores,
- * sets 'available' to false. */
-void
-ovs_numa_set_cpu_mask(const char *cmask)
-{
- int core_id = 0;
- int i;
-
- if (!found_numa_and_core) {
- return;
- }
-
- /* If no mask specified, resets the 'available' to true for all cores. */
- if (!cmask) {
- struct cpu_core *core;
-
- HMAP_FOR_EACH(core, hmap_node, &all_cpu_cores) {
- core->available = true;
- }
-
- return;
- }
-
- for (i = strlen(cmask) - 1; i >= 0; i--) {
- char hex = toupper((unsigned char)cmask[i]);
- int bin, j;
-
- if (hex >= '0' && hex <= '9') {
- bin = hex - '0';
- } else if (hex >= 'A' && hex <= 'F') {
- bin = hex - 'A' + 10;
- } else {
- bin = 0;
- VLOG_WARN("Invalid cpu mask: %c", cmask[i]);
- }
-
- for (j = 0; j < 4; j++) {
- struct cpu_core *core;
-
- core = CONTAINER_OF(hmap_first_with_hash(&all_cpu_cores,
- hash_int(core_id++, 0)),
- struct cpu_core, hmap_node);
- core->available = (bin >> j) & 0x1;
-
- if (core_id >= hmap_count(&all_cpu_cores)) {
- return;
- }
- }
- }
-
- /* For unspecified cores, sets 'available' to false. */
- while (core_id < hmap_count(&all_cpu_cores)) {
- struct cpu_core *core;
-
- core = CONTAINER_OF(hmap_first_with_hash(&all_cpu_cores,
- hash_int(core_id++, 0)),
- struct cpu_core, hmap_node);
- core->available = false;
- }
-}
-
int ovs_numa_thread_setaffinity_core(unsigned core_id OVS_UNUSED)
{
if (dummy_numa) {
@@ -42,17 +42,10 @@ void ovs_numa_init(void);
void ovs_numa_set_dummy(const char *dummy_config);
bool ovs_numa_numa_id_is_valid(int numa_id);
bool ovs_numa_core_id_is_valid(unsigned core_id);
-bool ovs_numa_core_is_pinned(unsigned core_id);
int ovs_numa_get_n_numas(void);
-void ovs_numa_set_cpu_mask(const char *cmask);
int ovs_numa_get_n_cores(void);
int ovs_numa_get_numa_id(unsigned core_id);
int ovs_numa_get_n_cores_on_numa(int numa_id);
-int ovs_numa_get_n_unpinned_cores_on_numa(int numa_id);
-bool ovs_numa_try_pin_core_specific(unsigned core_id);
-unsigned ovs_numa_get_unpinned_core_any(void);
-unsigned ovs_numa_get_unpinned_core_on_numa(int numa_id);
-void ovs_numa_unpin_core(unsigned core_id);
struct ovs_numa_dump *ovs_numa_dump_cores_on_numa(int numa_id);
struct ovs_numa_dump *ovs_numa_dump_cores_with_cmask(const char *cmask);
struct ovs_numa_dump *ovs_numa_dump_n_cores_per_numa(int n);
ovs-numa doesn't need to keep the state of the pmd threads, it is an implementation detail of dpif-netdev. Signed-off-by: Daniele Di Proietto <diproiettod@vmware.com> --- lib/ovs-numa.c | 168 --------------------------------------------------------- lib/ovs-numa.h | 7 --- 2 files changed, 175 deletions(-)