pstore: Add priv field to pstore_record for backend specific use
authorArd Biesheuvel <ardb@kernel.org>
Wed, 22 Jun 2022 07:31:31 +0000 (09:31 +0200)
committerArd Biesheuvel <ardb@kernel.org>
Fri, 24 Jun 2022 18:40:04 +0000 (20:40 +0200)
The EFI pstore backend will need to store per-record variable name data
when we switch away from the efivars layer. Add a priv field to struct
pstore_record, and document it as holding a backend specific pointer
that is assumed to be a kmalloc()d buffer, and will be kfree()d when the
entire record is freed.

Acked-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
fs/pstore/inode.c
fs/pstore/platform.c
include/linux/pstore.h

index 14658b0..ffbadb8 100644 (file)
@@ -55,6 +55,7 @@ static void free_pstore_private(struct pstore_private *private)
                return;
        if (private->record) {
                kfree(private->record->buf);
+               kfree(private->record->priv);
                kfree(private->record);
        }
        kfree(private);
index e26162f..0c034ea 100644 (file)
@@ -769,6 +769,7 @@ void pstore_get_backend_records(struct pstore_info *psi,
                if (rc) {
                        /* pstore_mkfile() did not take record, so free it. */
                        kfree(record->buf);
+                       kfree(record->priv);
                        kfree(record);
                        if (rc != -EEXIST || !quiet)
                                failed++;
index e97a818..638507a 100644 (file)
@@ -57,6 +57,9 @@ struct pstore_info;
  * @size:      size of @buf
  * @ecc_notice_size:
  *             ECC information for @buf
+ * @priv:      pointer for backend specific use, will be
+ *             kfree()d by the pstore core if non-NULL
+ *             when the record is freed.
  *
  * Valid for PSTORE_TYPE_DMESG @type:
  *
@@ -74,6 +77,7 @@ struct pstore_record {
        char                    *buf;
        ssize_t                 size;
        ssize_t                 ecc_notice_size;
+       void                    *priv;
 
        int                     count;
        enum kmsg_dump_reason   reason;