[all-commits] [llvm/llvm-project] e07e1d: [test] Increase test assertion for reasonable auxv...

Jordan Rupprecht via All-commits all-commits at lists.llvm.org
Mon Sep 12 19:41:24 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: e07e1d4425f837e5b38336664a246588880834b3
      https://github.com/llvm/llvm-project/commit/e07e1d4425f837e5b38336664a246588880834b3
  Author: Jordan Rupprecht <rupprecht at google.com>
  Date:   2022-09-12 (Mon, 12 Sep 2022)

  Changed paths:
    M lldb/test/API/tools/lldb-server/TestGdbRemoteAuxvSupport.py

  Log Message:
  -----------
  [test] Increase test assertion for reasonable auxv values.

While auxv keys are usually small, e.g. less than 50, they can sometimes be larger, especially on a downstream kernel where a custom auxv entry is intentionally high to avoid conflicting with the standard lower numbers. This test fails on a system with an auxv value bigger than 1000, but instead of putting this test at that value plus one, it looks like 2023 (i.e. `AT_SUN_CAP_HW2`) is another large one out there. Use 2500 as a limit to still have this be a reasonable "small" check but still allow all known auxv keys.

Semi-related change: this test case prints the auxv dict at the trace level, but only _after_ the assertion fails, making it not print what the offending value is as the test case aborts. Move it earlier so we can see what the "unreasonable" auxv value is.




More information about the All-commits mailing list