Miscellaneous Updates (const-correctness, logic fixes, etc.) (#126)

* Update lib/rocprofiler/hsa/hsa.cpp

- fix logic for constructing callback_contexts and buffered_contexts arrays

* Update include/rocprofiler/{agent,fwd,pc_sampling}.h

- remove rocprofiler_pc_sampling_config_array_t due to const problems
- update rocprofiler_agent_t to use arrays to const data
- remove redundant rocprofiler_query_pc_sampling_agent_configurations
  - this implementation is quite literally looking up info in the agent struct that was passed

* Update lib/rocprofiler/pc_sampling.cpp

- remove rocprofiler_query_pc_sampling_agent_configurations

* update lib/rocprofiler/agent.cpp

- handle const fields
- make mi200_pc_sampling_config variable static

* Update lib/rocprofiler/tests/agent.cpp

- tweak to pc_sampling_configs offset

* Update samples/pc_sampling

- Update sample to reflect minor tweaks to pc_sampling_configs in rocprofiler_agent_t

* Update CI workflow

- remove 'if: ${{ always() }}'
  - I suspect this is why the jobs do not cancel in progress correctly

[ROCm/rocprofiler-sdk commit: d1518c65b2]

projects/rocprofiler-sdk/source/include/rocprofiler/pc_sampling.h

@@ -63,19 +63,6 @@ struct rocprofiler_pc_sampling_configuration_s
     uint64_t                         flags;
 };
 
-/**
- * @brief Query PC Sampling Configuration.
- *
- * @param [in] agent
- * @param [out] config
- * @param [out] config_count
- * @return ::rocprofiler_status_t
- */
-rocprofiler_status_t ROCPROFILER_API
-rocprofiler_query_pc_sampling_agent_configurations(rocprofiler_agent_t                      agent,
-                                                   rocprofiler_pc_sampling_configuration_t* config,
-                                                   size_t* config_count) ROCPROFILER_NONNULL(2, 3);
-
 /** @} */
 
 ROCPROFILER_EXTERN_C_FINI