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 2023/01/13 15:10:24 UTC

[GitHub] [nuttx] jlaitine commented on a diff in pull request #8109: Add shmfs

jlaitine commented on code in PR #8109:
URL: https://github.com/apache/nuttx/pull/8109#discussion_r1069571383


##########
fs/shm/shmfs_private.h:
##########
@@ -0,0 +1,60 @@
+/****************************************************************************
+ * fs/shm/shmfs_private.h
+ *
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.  The
+ * ASF licenses this file to you under the Apache License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance with the
+ * License.  You may obtain a copy of the License at
+ *
+ *   http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+ * WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.  See the
+ * License for the specific language governing permissions and limitations
+ * under the License.
+ *
+ ****************************************************************************/
+
+#ifndef __FS_SHM_SHMFS_PRIVATE_H
+#define __FS_SHM_SHMFS_PRIVATE_H
+
+/****************************************************************************
+ * Included Files
+ ****************************************************************************/
+
+#include <stdlib.h>
+
+/****************************************************************************
+ * Public Data
+ ****************************************************************************/
+
+extern const struct file_operations shmfs_operations;
+
+/****************************************************************************
+ * Public Types
+ ****************************************************************************/
+
+struct shmfs_object_s
+{
+  /* Total number of bytes needed from physical memory. */
+
+  size_t length;
+
+  /* Vector of allocations from physical memory.
+   * - In flat and protected builds there is just one pointer to the
+   *   allocated memory in paddr[0].
+   * - In kernel build each address points to one page in page pool
+   *   and the length of the vector is MM_NPAGES(length)
+   */
+
+  FAR void *paddr[];

Review Comment:
   Sure! I'll modify within few days



-- 
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