<div dir="ltr">Hello, <div><br>Just proposed a new patch version for the same topic<div><a href="https://lore.kernel.org/ltp/20221026140408.471609-1-alessandro.carminati@gmail.com/">https://lore.kernel.org/ltp/20221026140408.471609-1-alessandro.carminati@gmail.com/</a><br></div><div><br></div><div>cheers<br clear="all"><div><div dir="ltr" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr">Alessandro Carminati<br><br></div></div></div></div></div></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Il giorno mer 26 ott 2022 alle ore 12:56 Jan Stancek <<a href="mailto:jstancek@redhat.com" target="_blank">jstancek@redhat.com</a>> ha scritto:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Tue, Oct 25, 2022 at 4:59 PM Richard Palethorpe <<a href="mailto:rpalethorpe@suse.com" target="_blank">rpalethorpe@suse.com</a>> wrote:<br>
><br>
> mountinfo doesn't always contain the correct device name. For example<br>
> /dev/root may be displayed, but not exist in devtmpfs[1].<br>
><br>
> The unevent file in sysfs is another way of finding the device name<br>
         ^^ small typo here<br>
<br>
> from the major and minor numbers. Possibly it always displays the<br>
> proper device name.<br>
<br>
major/minor numbers may still fail for btrfs, but that issue exists<br>
outside of scope of this patch.<br>
<br>
><br>
> One caveat is the sysfs can be disabled, so this commit does not<br>
> remove the mountinfo method altogether, but leaves it as a fallback.<br>
<br>
Could we add some TINFO message to output, so we know where dev_name<br>
came from? (uevent vs mountinfo)<br>
<br>
><br>
> Alessandro Carminati originally sent two patches[1] which added the<br>
> uevent file method as a fallback. However it seems like the better<br>
> method.<br>
><br>
> [1]: <a href="https://lore.kernel.org/ltp/Y0023HcAOlhfAcJw@lab.hqhome163.com/" rel="noreferrer" target="_blank">https://lore.kernel.org/ltp/Y0023HcAOlhfAcJw@lab.hqhome163.com/</a><br>
><br>
> Signed-off-by: Richard Palethorpe <<a href="mailto:rpalethorpe@suse.com" target="_blank">rpalethorpe@suse.com</a>><br>
> Suggested-by: Alessandro Carminati <<a href="mailto:alessandro.carminati@gmail.com" target="_blank">alessandro.carminati@gmail.com</a>><br>
> Reported-by: Alessandro Carminati <<a href="mailto:alessandro.carminati@gmail.com" target="_blank">alessandro.carminati@gmail.com</a>><br>
<br>
Acked-by: Jan Stancek <<a href="mailto:jstancek@redhat.com" target="_blank">jstancek@redhat.com</a>><br>
<br>
</blockquote></div>