You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/09/01 01:23:30 UTC

[GitHub] [incubator-nuttx] masayuki2009 commented on a diff in pull request #6979: fs: procfs: Ignore to register Umem if CONFIG_BUILD_KERNEL=y

masayuki2009 commented on code in PR #6979:
URL: https://github.com/apache/incubator-nuttx/pull/6979#discussion_r960142446


##########
fs/procfs/fs_procfsmeminfo.c:
##########
@@ -578,6 +578,13 @@ static int meminfo_stat(FAR const char *relpath, FAR struct stat *buf)
 
 void procfs_register_meminfo(FAR struct procfs_meminfo_entry_s *entry)
 {
+#ifdef CONFIG_BUILD_KERNEL

Review Comment:
   @xiaoxiang781216 
   Hmm, the change does not show Kmem.
   
   the original PR
   
   ```
   nsh> free
                      total       used       free    largest  nused  nfree
           Kmem:  125677760      20864  125656896  125651936     60      3
           Page:  134217728    2256896  131960832  131960832
   ```
   
   after the changed applied
   
   ```
   nsh> free
                      total       used       free    largest  nused  nfree
           Page:  134217728    2256896  131960832  131960832
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org