Skip to content

log: adjust more log entries to the new format#605

Merged
AltraMayor merged 3 commits intomasterfrom
eh-log
Sep 15, 2022
Merged

log: adjust more log entries to the new format#605
AltraMayor merged 3 commits intomasterfrom
eh-log

Conversation

@AltraMayor
Copy link
Owner

No description provided.

The new log format already includes the name of the block.
Prefix the log entries of process_single_policy() and
process_single_packet() with their names.
When dumping memory, one should use rte_be_to_cpu_64() instead of
rte_cpu_to_be_64() because one is dumping the memory in the same
order the bytes are in memory. That is, the first byte in memory
is the most significant when printed out, the second byte
the second most significant, and so forth; this is big endianess.

In practice, this change is purely semantic since
rte_cpu_to_be_64() and rte_be_to_cpu_64() are identical on
little and big endian hosts.
@AltraMayor AltraMayor added this to the Version 1.1 milestone Sep 14, 2022
@AltraMayor AltraMayor merged commit e639625 into master Sep 15, 2022
@AltraMayor AltraMayor deleted the eh-log branch September 15, 2022 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant