On Mon, 15 Apr 2024 16:08:18 +0000 Abhinav Jain <jain.abhinav177@xxxxxxxxx> wrote: > This patch addresses TODO in "./drivers/md/md.c:6752". > Add journal struct member to mdu_array_info_t in "md_u.h". > Add the journal field in get_array_info function in "md.c". > Need feedback on if the update_array_info function needs to be updated. > > Signed-off-by: Abhinav Jain <jain.abhinav177@xxxxxxxxx> > --- Hello, The field you added here is unused. I believe it works but I cannot determine that as the filed you added in not exposed to user (probably --detail is right place). Please do some tests and include the example test in commit description. Please also explain why you see it useful. For the question you asked, having it in update_array_info requires possibility to modify/set from userspace, so user interface and/or corresponding metadata property is needed. Please explain why it would be useful to have possibility to set it from userspace, what are the benefits. Thanks, Mariusz
