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: <Y8q0H7SiDkdfmyXP@art_vandelay>
Date:   Fri, 20 Jan 2023 15:32:47 +0000
From:   Sean Paul <sean@...rly.run>
To:     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,
        dmitry.baryshkov@...aro.org, 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 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?

Thanks again,

Sean

> 
> Best regards,
> Krzysztof
> 

-- 
Sean Paul, Software Engineer, Google / Chromium OS