linux-kernel - Re: [PATCH v6 01/10] drm/hdcp: Add drm_hdcp_atomic

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <A0271342-FB1E-4F7C-BD94-7506F4FACE6E@linaro.org>
Date:   Fri, 20 Jan 2023 23:33:40 +0300
From:   Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
To:     Sean Paul <sean@...rly.run>,
        Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
CC:     Mark Yacoub <markyacoub@...omium.org>, quic_khsieh@...cinc.com,
        linux-arm-msm@...r.kernel.org, dri-devel@...ts.freedesktop.org,
        freedreno@...ts.freedesktop.org, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org, intel-gfx@...ts.freedesktop.org,
        robdclark@...il.com, quic_abhinavk@...cinc.com, sean@...rly.run,
        airlied@...il.com, daniel@...ll.ch, robh+dt@...nel.org,
        krzysztof.kozlowski+dt@...aro.org, agross@...nel.org,
        andersson@...nel.org, konrad.dybcio@...ainline.org,
        jani.nikula@...ux.intel.com, joonas.lahtinen@...ux.intel.com,
        rodrigo.vivi@...el.com, tvrtko.ursulin@...ux.intel.com,
        tzimmermann@...e.de, ville.syrjala@...ux.intel.com,
        stanislav.lisovskiy@...el.com, matthew.d.roper@...el.com,
        imre.deak@...el.com, lucas.demarchi@...el.com,
        manasi.d.navare@...el.com, swati2.sharma@...el.com,
        bhanuprakash.modem@...el.com, javierm@...hat.com,
        jose.souza@...el.com, lyude@...hat.com, hbh25y@...il.com,
        arun.r.murthy@...el.com, ashutosh.dixit@...el.com,
        ankit.k.nautiyal@...el.com, maxime@...no.tech, swboyd@...omium.org,
        christophe.jaillet@...adoo.fr, quic_sbillaka@...cinc.com,
        johan+linaro@...nel.org, dianders@...omium.org, marex@...x.de,
        quic_jesszhan@...cinc.com, bjorn.andersson@...aro.org,
        abhinavk@...eaurora.org, seanpaul@...omium.org,
        Jani Nikula <jani.nikula@...el.com>
Subject: Re: [PATCH v6 01/10] drm/hdcp: Add drm_hdcp_atomic_check()



On 20 January 2023 18:32:47 GMT+03:00, Sean Paul <sean@...rly.run> wrote:
>On Thu, Jan 19, 2023 at 11:37:52AM +0100, Krzysztof Kozlowski wrote:
>> On 18/01/2023 20:30, Mark Yacoub wrote:
>> > From: Sean Paul <seanpaul@...omium.org>
>> > 
>> > This patch moves the hdcp atomic check from i915 to drm_hdcp so other
>> > drivers can use it. No functional changes, just cleaned up some of the
>> > code when moving it over.
>> > 
>> > Acked-by: Jani Nikula <jani.nikula@...el.com>
>> > Acked-by: Jani Nikula <jani.nikula@...el.com>
>> > Reviewed-by: Rodrigo Vivi <rodrigo.vivi@...el.com>
>> > Reviewed-by: Abhinav Kumar <abhinavk@...eaurora.org>
>> > Signed-off-by: Sean Paul <seanpaul@...omium.org>
>> > Signed-off-by: Mark Yacoub <markyacoub@...omium.org>
>> > Link: https://patchwork.freedesktop.org/patch/msgid/20210913175747.47456-2-sean@poorly.run #v1
>> > Link: https://patchwork.freedesktop.org/patch/msgid/20210915203834.1439-2-sean@poorly.run #v2
>> > Link: https://patchwork.freedesktop.org/patch/msgid/20211001151145.55916-2-sean@poorly.run #v3
>> > Link: https://patchwork.freedesktop.org/patch/msgid/20211105030434.2828845-2-sean@poorly.run #v4
>> > Link: https://patchwork.freedesktop.org/patch/msgid/20220411204741.1074308-2-sean@poorly.run #v5
>> 
>> It seems all your previous versions were sent not to correct people and
>> lists. Therefore we see it for the first time even though it is v6! 
>
>Hi Krzysztof,
>Thanks for your review comments.
>
>Here are the addresses the last version was sent to, who is missing?
>
>To: dri-devel@...ts.freedesktop.org, 
>    jani.nikula@...el.com,
>    intel-gfx@...ts.freedesktop.org,
>    freedreno@...ts.freedesktop.org,
>    rodrigo.vivi@...el.com
>Cc: bjorn.andersson@...aro.org, 
>    swboyd@...omium.org,
>    abhinavk@...eaurora.org,
>    markyacoub@...omium.org,
>    Sean Paul <seanpaul@...omium.org>,
>    Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
>    Maxime Ripard <mripard@...nel.org>,
>    Thomas Zimmermann <tzimmermann@...e.de>,
>    David Airlie <airlied@...ux.ie>,
>    Daniel Vetter <daniel@...ll.ch>,
>    Jani Nikula <jani.nikula@...ux.intel.com>,
>    Joonas Lahtinen <joonas.lahtinen@...ux.intel.com>,
>    Tvrtko Ursulin <tvrtko.ursulin@...ux.intel.com>
>
>> It's
>> not the first such weird CC list in chromium, so maybe your
>> organisational process could be improved? Not only for you but for
>> colleagues as well, so you all start using get_maintainers.pl on newest
>> kernel (not something ancient)?
>
>I can't really speak for others, but I use MAINTAINERS from drm-tip. The 
>previous patch sets were sent before 24df12013853 ("MAINTAINERS: Add 
>Dmitry as MSM DRM driver co-maintainer"), which might explain why you think
>there are absences?

Current iteration of the patchset got at least three addresses wrong. They have been changed for various reasons. Thus I also can suppose that the list is incomplete and/or incorrect.

>
>Thanks again,
>
>Sean
>
>> 
>> Best regards,
>> Krzysztof
>> 
>

-- 
With best wishes
Dmitry