[lvc-project] [PATCH 5.10] drm/shmem-helper: Fix BUG_ON() on mmap(PROT_WRITE, MAP_PRIVATE)

[Fedor Pchelkin]

Для 5.10 ветки запросов по бэкпорту на текущий момент не было - задача всё
ещё актуальна.

Отправлять предварительно стоит всё же в lvc-patches at linuxtesting.org, а не
lvc-project at linuxtesting.org.

lvc-project предназначен для включения в копию письма при взаимодействии с
международным сообществом.

lvc-patches предназначен для нашего внутреннего взаимодействия.

On Fri, 18. Oct 12:18, Artem Sdvizhkov wrote:
> From: "Artem Sdvizhkov" <a.sdvizhkov at securitycode.ru>

Такая же проблема с добавляемым From выше была у Георгия Рюрикова. Вашего
>From в теле отправляемого письма быть не должно. Скорее всего он
добавляется автоматически git send email'ом, т.к. From, который Вы
прописываете в *заголовке* письма, отличается от того, что задан в
~/.gitconfig

> 
> From: "Wachowski, Karol" <karol.wachowski at intel.com>
> 
> commit 39bc27bd688066a63e56f7f64ad34fae03fbe3b8 upstream.
> 
> Lack of check for copy-on-write (COW) mapping in drm_gem_shmem_mmap
> allows users to call mmap with PROT_WRITE and MAP_PRIVATE flag
> causing a kernel panic due to BUG_ON in vmf_insert_pfn_prot:
> BUG_ON((vma->vm_flags & VM_PFNMAP) && is_cow_mapping(vma->vm_flags));
> 
> Return -EINVAL early if COW mapping is detected.
> 
> This bug affects all drm drivers using default shmem helpers.
> It can be reproduced by this simple example:
> void *ptr = mmap(0, size, PROT_WRITE, MAP_PRIVATE, fd, mmap_offset);
> ptr[0] = 0;
> 
> Fixes: 2194a63a818d ("drm: Add library for shmem backed GEM objects")
> 
> Signed-off-by: Wachowski, Karol <karol.wachowski at intel.com>
> Signed-off-by: Jacek Lawrynowicz <jacek.lawrynowicz at linux.intel.com>
> Signed-off-by: Daniel Vetter <daniel.vetter at ffwll.ch>
> [Artem: In order to adapt this patch to branch 5.10
> add header file mm/internal.h]
> Signed-off-by: Artem Sdvizhkov <a.sdvizhkov at securitycode.ru>

Всё описание исходного коммита остаётся без изменений. Пустой строки после
тега Fixes быть не должно. Вами зачем-то были убраны множественные теги Cc:
из зоны подписей.

Ваша подпись с комментарием должны идти в самом-самом конце, после всего,
что есть в описании исходного коммита (т.е. после Link: ниже).

> Link: https://patchwork.freedesktop.org/patch/msgid/20240520100514.925681-1-jacek.lawrynowicz@linux.intel.com
> ---
>  drivers/gpu/drm/drm_gem_shmem_helper.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/drivers/gpu/drm/drm_gem_shmem_helper.c b/drivers/gpu/drm/drm_gem_shmem_helper.c
> index e8f07305e279..37f347f39c88 100644
> --- a/drivers/gpu/drm/drm_gem_shmem_helper.c
> +++ b/drivers/gpu/drm/drm_gem_shmem_helper.c
> @@ -17,6 +17,8 @@
>  #include <drm/drm_prime.h>
>  #include <drm/drm_print.h>
> 
> +#include "../../../mm/internal.h"   /* is_cow_mapping() */

В https://lore.kernel.org/stable/3469eb7f-96b5-4efa-b3de-82c956809038@redhat.com/
сделано такое же преобразование для stable. Хоть как-то и "костыльно"
выглядит, но makes sense.

Компилируемость проверяли? Всё нормально?

> +
>  /**
>   * DOC: overview
>   *
> @@ -630,6 +632,9 @@ int drm_gem_shmem_mmap(struct drm_gem_object *obj, struct vm_area_struct *vma)
>                 return ret;
>         }
> 
> +       if (is_cow_mapping(vma->vm_flags))
> +               return -EINVAL;
> +
>         shmem = to_drm_gem_shmem_obj(obj);
> 
>         ret = drm_gem_shmem_get_pages(shmem);
> --
> 2.43.0
> 
> Заявление о конфиденциальности
> 
> Данное электронное письмо и любые приложения к нему являются конфиденциальными и предназначены исключительно для адресата. Если Вы не являетесь адресатом данного письма, пожалуйста, уведомите немедленно отправителя, не раскрывайте содержание другим лицам, не используйте его в каких-либо целях, не храните и не копируйте информацию любым способом.

По поводу добавляемой (судя по всему Вашим почтовым сервером) вот этой
плашки выше мэйнтейнеры и разработчики из международного сообщества уже не
раз высказывались в негативном ключе.

https://lore.kernel.org/lkml/2024091631-quaintly-scouting-3c29@gregkh/
https://lore.kernel.org/stable/87tte545pc.fsf@intel.com/

Боюсь, из-за неё будут возникать постоянные проблемы с принятием патчей.
Не знаю, имеется ли у Вас возможность сделать, чтобы таких строк не
добавлялось в письмо?...