Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:3 as permitted sender) client-ip=2620:137:e000::3:3;
From:   Javier Martinez Canillas <javierm@redhat.com>
To:     Zack Rusin <zackr@vmware.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Cc:     "corbet@lwn.net" <corbet@lwn.net>,
        "olvaffe@gmail.com" <olvaffe@gmail.com>,
        "daniel.vetter@ffwll.ch" <daniel.vetter@ffwll.ch>,
        "nunes.erico@gmail.com" <nunes.erico@gmail.com>,
        "airlied@redhat.com" <airlied@redhat.com>,
        "pekka.paalanen@collabora.com" <pekka.paalanen@collabora.com>,
        "tzimmermann@suse.de" <tzimmermann@suse.de>,
        "belmouss@redhat.com" <belmouss@redhat.com>,
        "mripard@kernel.org" <mripard@kernel.org>,
        "daniel@ffwll.ch" <daniel@ffwll.ch>,
        "gurchetansingh@chromium.org" <gurchetansingh@chromium.org>,
        "maarten.lankhorst@linux.intel.com" 
        <maarten.lankhorst@linux.intel.com>,
        "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>,
        "linux-doc@vger.kernel.org" <linux-doc@vger.kernel.org>,
        "kraxel@redhat.com" <kraxel@redhat.com>,
        "virtualization@lists.linux.dev" <virtualization@lists.linux.dev>,
        "airlied@gmail.com" <airlied@gmail.com>,
        Linux-graphics-maintainer <Linux-graphics-maintainer@vmware.com>,
        "contact@emersion.fr" <contact@emersion.fr>
Subject: Re: [PATCH v2 0/5] drm: Allow the damage helpers to handle buffer
 damage
In-Reply-To: <a16a61582f90a5b490fb7681b44864a4801c830a.camel@vmware.com>
References: <20231115131549.2191589-1-javierm@redhat.com>
 <a16a61582f90a5b490fb7681b44864a4801c830a.camel@vmware.com>
Date:   Thu, 16 Nov 2023 08:46:14 +0100
Message-ID: <87o7fu5eex.fsf@minerva.mail-host-address-is-not-set>
MIME-Version: 1.0
Content-Type: text/plain
Precedence: bulk

Zack Rusin <zackr@vmware.com> writes:

Hello Zack,

> On Wed, 2023-11-15 at 14:15 +0100, Javier Martinez Canillas wrote:

[...]

>>
>> Changes in v2:
>> - Add a struct drm_plane_state .ignore_damage_clips to set in the plane's
>>   .atomic_check, instead of having different helpers (Thomas Zimmermann).
>> - Set struct drm_plane_state .ignore_damage_clips in virtio-gpu plane's
>>   .atomic_check instead of using a different helpers (Thomas Zimmermann).
>> - Set struct drm_plane_state .ignore_damage_clips in vmwgfx plane's
>>   .atomic_check instead of using a different helpers (Thomas Zimmermann).
>
> The series looks good to me, thanks for tackling this. I'm surprised that we don't

Thanks. Can I get your r-b or a-b ?

> have any IGT tests for this. Seems like it shouldn't be too hard to test it in a
> generic way with just a couple of dumb buffers.
>

Yes, I haven't looked at it but also think that shouldn't be that hard.

> z

-- 
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat