[OS-BUILD PATCH] [redhat] New configs in drivers/dma-buf

[CKI Gitlab (via Email Bridge)]

From: Fedora Kernel Team

[redhat] New configs in drivers/dma-buf

Hi,

As part of the ongoing rebase effort, the following configuration
options need to be reviewed.

As a reminder, the ARK configuration flow involves moving unreviewed
configuration options from the pending directory to the ark directory.
In the diff below, options are removed from the pending directory and
added to the ark hierarchy. The final options that need to be ACKed
are the files that are being added to the ark hierarchy.

If the value for a file that is added should be changed, please reply
with a better option.

CONFIG_DMABUF_DEBUG:

This option enables additional checks for DMA-BUF importers and
 exporters. Specifically it validates that importers do not peek at the
 underlying struct page when they import a buffer.

Symbol: DMABUF_DEBUG [=n]
 Type  : bool
 Defined at drivers/dma-buf/Kconfig:53
   Prompt: DMA-BUF debug checks
   Location:
     -> Device Drivers
       -> DMABUF options

---

Cc: Lyude Paul 
Signed-off-by: Fedora Kernel Team

diff a/redhat/configs/common/generic/CONFIG_DMABUF_DEBUG b/redhat/configs/common/generic/CONFIG_DMABUF_DEBUG
--- /dev/null
+++ b/redhat/configs/common/generic/CONFIG_DMABUF_DEBUG
@@ -0,0 +1 @@
+# CONFIG_DMABUF_DEBUG is not set
diff a/redhat/configs/pending-common/generic/CONFIG_DMABUF_DEBUG b/redhat/configs/pending-common/generic/CONFIG_DMABUF_DEBUG
--- a/redhat/configs/pending-common/generic/CONFIG_DMABUF_DEBUG
+++ /dev/null
@@ -1,17 +0,0 @@
-# CONFIG_DMABUF_DEBUG:
-# 
-# This option enables additional checks for DMA-BUF importers and
-# exporters. Specifically it validates that importers do not peek at the
-# underlying struct page when they import a buffer.
-# 
-# Symbol: DMABUF_DEBUG [=n]
-# Type  : bool
-# Defined at drivers/dma-buf/Kconfig:53
-#   Prompt: DMA-BUF debug checks
-#   Location:
-#     -> Device Drivers
-#       -> DMABUF options
-# 
-# 
-# 
-# CONFIG_DMABUF_DEBUG is not set

--
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/903

[Lyude Paul]

imho it'd be nice if others could take a look at this since DRM usually doesn't
make use of the dma buf debugging options, but I guess feel free to set this to
=n for non-debug, =y for debug

On Mon, 2021-02-22 at 12:56 +0000, CKI Gitlab (via Email Bridge) wrote:
> From: Fedora Kernel Team
>
> [redhat] New configs in drivers/dma-buf
>
> Hi,
>
> As part of the ongoing rebase effort, the following configuration
> options need to be reviewed.
>
> As a reminder, the ARK configuration flow involves moving unreviewed
> configuration options from the pending directory to the ark directory.
> In the diff below, options are removed from the pending directory and
> added to the ark hierarchy. The final options that need to be ACKed
> are the files that are being added to the ark hierarchy.
>
> If the value for a file that is added should be changed, please reply
> with a better option.
>
> CONFIG_DMABUF_DEBUG:
>
> This option enables additional checks for DMA-BUF importers and
>  exporters. Specifically it validates that importers do not peek at the
>  underlying struct page when they import a buffer.
>
> Symbol: DMABUF_DEBUG [=n]
>  Type  : bool
>  Defined at drivers/dma-buf/Kconfig:53
>    Prompt: DMA-BUF debug checks
>    Location:
>      -> Device Drivers
>        -> DMABUF options
>
> Cc: Lyude Paul 
> Signed-off-by: Fedora Kernel Team
>
> diff a/redhat/configs/common/generic/CONFIG_DMABUF_DEBUG
> b/redhat/configs/common/generic/CONFIG_DMABUF_DEBUG
> --- /dev/null
> +++ b/redhat/configs/common/generic/CONFIG_DMABUF_DEBUG
> @@ -0,0 +1 @@
> +# CONFIG_DMABUF_DEBUG is not set
> diff a/redhat/configs/pending-common/generic/CONFIG_DMABUF_DEBUG
> b/redhat/configs/pending-common/generic/CONFIG_DMABUF_DEBUG
> --- a/redhat/configs/pending-common/generic/CONFIG_DMABUF_DEBUG
> +++ /dev/null
> @@ -1,17 +0,0 @@
> -# CONFIG_DMABUF_DEBUG:
> -# 
> -# This option enables additional checks for DMA-BUF importers and
> -# exporters. Specifically it validates that importers do not peek at the
> -# underlying struct page when they import a buffer.
> -# 
> -# Symbol: DMABUF_DEBUG [=n]
> -# Type  : bool
> -# Defined at drivers/dma-buf/Kconfig:53
> -#   Prompt: DMA-BUF debug checks
> -#   Location:
> -#     -> Device Drivers
> -#       -> DMABUF options
> -# 
> -# 
> -# 
> -# CONFIG_DMABUF_DEBUG is not set
>
> --
> https://gitlab.com/cki-project/kernel-ark/-/merge_requests/903

[Patrick Talbert (via Email Bridge)]

From: Patrick Talbert on gitlab.com
https://gitlab.com/cki-project/kernel-ark/-/merge_requests/903#note_53985862...

These two config items were handled in MR !956 so this can be closed.