On 03/05/2023 19:02, Mukesh Ojha wrote: > Add support to update client's region physical/virtual addresses, > which is useful for dynamic loadable modules, dynamic address > changing clients like if we want to collect current stack > information for each core and the current stack is changing on > each sched_switch event, So here virtual/physical address of > the current stack is changing. So, to cover such use cases > add the update region support in minidump driver. > > Signed-off-by: Mukesh Ojha <quic_mojha@xxxxxxxxxxx> > --- > drivers/soc/qcom/qcom_minidump.c | 57 ++++++++++++++++++++++++++++++++++++++++ > include/soc/qcom/qcom_minidump.h | 7 +++++ > 2 files changed, 64 insertions(+) > > diff --git a/drivers/soc/qcom/qcom_minidump.c b/drivers/soc/qcom/qcom_minidump.c > index 6d29371..853bdda 100644 > --- a/drivers/soc/qcom/qcom_minidump.c > +++ b/drivers/soc/qcom/qcom_minidump.c > @@ -561,6 +561,63 @@ int qcom_apss_minidump_region_unregister(const struct qcom_apss_minidump_region > } > EXPORT_SYMBOL_GPL(qcom_apss_minidump_region_unregister); > > +/** > + * qcom_apss_minidump_update_region() - Update region in APSS minidump table. Description of the function is exact copy of function name. This is a hint for two things: 1. Your function has wrong name. 2. Your description is not helping. Best regards, Krzysztof
