On 5/16/23 15:07, Yajun Deng wrote: > + > +What: /sys/devices/system/edac/mc/mc*/node* > +Date: May 2023 > +Contact: Yajun Deng <yajun.deng@xxxxxxxxx> > + linux-edac@xxxxxxxxxxxxxxx > +Description: When CONFIG_NUMA is enabled, a symbolic link that points to the > + corresponding NUMA node directory. > + > + For example, the following symbolic link is created for node0 on mc0 > + and mc1: > + "For example, on node0 with two memory control directories mc0 and mc1 the symlinks are::" (I prefer using literal code block here). > + /sys/devices/system/edac/mc/mc0/node0 -> ../../../node/node0 > + /sys/devices/system/edac/mc/mc1/node0 -> ../../../node/node0 Or bullet lists should better fit listing above? > + > +What: /sys/devices/system/node/node*/mc* > +Date: May 2023 > +Contact: Yajun Deng <yajun.deng@xxxxxxxxx> > + linux-edac@xxxxxxxxxxxxxxx > +Description: When CONFIG_NUMA is enabled, a symbolic link that points to the > + corresponding memory control directory. > + > + For example, the following symbolic link is created for mc0 and mc1 > + on node0: > + > + /sys/devices/system/node/node0/mc0 -> ../../edac/mc/mc0 > + /sys/devices/system/node/node0/mc1 -> ../../edac/mc/mc1 Similar as my review above. Thanks. -- An old man doll... just what I always wanted! - Clara
