Received-SPF: pass (google.com: domain of linux-kernel+bounces-149419-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Precedence: bulk
MIME-Version: 1.0
References: <20240416035240.2450127-1-wangzhu9@huawei.com> <2024041658-imagines-unlatch-a9b6@gregkh>
 <036c3371d3a64ef8881260197ce37dbc@huawei.com>
In-Reply-To: <036c3371d3a64ef8881260197ce37dbc@huawei.com>
From: Alex Deucher <alexdeucher@gmail.com>
Date: Wed, 17 Apr 2024 21:58:20 -0400
Message-ID: <CADnq5_NML_BiqQx2UmwH86d3qv57D3tFRL--dro1qA99r0Qr5w@mail.gmail.com>
Subject: Re: [PATCH v6.6] drm/amd/display: Wake DMCUB before executing GPINT commands
To: wangzhu <wangzhu9@huawei.com>
Cc: Greg KH <gregkh@linuxfoundation.org>, 
	"harry.wentland@amd.com" <harry.wentland@amd.com>, "sunpeng.li@amd.com" <sunpeng.li@amd.com>, 
	"Rodrigo.Siqueira@amd.com" <Rodrigo.Siqueira@amd.com>, 
	"alexander.deucher@amd.com" <alexander.deucher@amd.com>, 
	"christian.koenig@amd.com" <christian.koenig@amd.com>, "airlied@linux.ie" <airlied@linux.ie>, 
	"daniel@ffwll.ch" <daniel@ffwll.ch>, "qingqing.zhuo@amd.com" <qingqing.zhuo@amd.com>, 
	"stylon.wang@amd.com" <stylon.wang@amd.com>, "Josip.Pavic@amd.com" <Josip.Pavic@amd.com>, 
	"trix@redhat.com" <trix@redhat.com>, "cruise.hung@amd.com" <cruise.hung@amd.com>, 
	"Eric.Yang2@amd.com" <Eric.Yang2@amd.com>, 
	"mario.limonciello@amd.com" <mario.limonciello@amd.com>, "alvin.lee2@amd.com" <alvin.lee2@amd.com>, 
	"jun.lei@amd.com" <jun.lei@amd.com>, "austin.zheng@amd.com" <austin.zheng@amd.com>, 
	"sunglee@amd.com" <sunglee@amd.com>, "paul.hsieh@amd.com" <paul.hsieh@amd.com>, 
	"hanghong.ma@amd.com" <hanghong.ma@amd.com>, "JinZe.Xu@amd.com" <JinZe.Xu@amd.com>, 
	"lewis.huang@amd.com" <lewis.huang@amd.com>, Zhengzengkai <zhengzengkai@huawei.com>, 
	"alex.hung@amd.com" <alex.hung@amd.com>, "syed.hassan@amd.com" <syed.hassan@amd.com>, 
	"wayne.lin@amd.com" <wayne.lin@amd.com>, 
	"nicholas.kazlauskas@amd.com" <nicholas.kazlauskas@amd.com>, 
	"chiahsuan.chung@amd.com" <chiahsuan.chung@amd.com>, 
	"aurabindo.pillai@amd.com" <aurabindo.pillai@amd.com>, "aric.cyr@amd.com" <aric.cyr@amd.com>, 
	"amd-gfx@lists.freedesktop.org" <amd-gfx@lists.freedesktop.org>, 
	"dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>, 
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable

On Wed, Apr 17, 2024 at 9:51=E2=80=AFPM wangzhu <wangzhu9@huawei.com> wrote=
:
>
> Hi Greg, thanks for your reply. Since there is no patch to fix CVE-2023-5=
2624 in linux-5.10, there is a patch in the linux-6.7 branch, its commit is=
 2ef98c6d753a744e333b7e34b9cf687040fba57d ("drm/amd/display: Wake DMCUB bef=
ore executing GPINT commands"). When we apply this patch to linux-5.10, the=
re are lots of conflicts, and we found there are lots of dependent patches,=
 and lots of patches are not proposed to fix the cve, they are presented to=
 add new functions of the kernel.
>

Why is there a CVE?  Have you uncovered some specific issue?

Alex

> My commit comes from nearly 20 patches. For each patch, not all of its co=
ntent is meant to fix the cve, so I just get the part which is helpful to f=
ix. It is why I don't present the patches one by one instead of merging the=
m into one big patch.
>
>
> -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6-----
> =E5=8F=91=E4=BB=B6=E4=BA=BA: Greg KH [mailto:gregkh@linuxfoundation.org]
> =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2024=E5=B9=B44=E6=9C=8816=E6=97=A5 =
12:54
> =E6=94=B6=E4=BB=B6=E4=BA=BA: wangzhu <wangzhu9@huawei.com>
> =E6=8A=84=E9=80=81: harry.wentland@amd.com; sunpeng.li@amd.com; Rodrigo.S=
iqueira@amd.com; alexander.deucher@amd.com; christian.koenig@amd.com; airli=
ed@linux.ie; daniel@ffwll.ch; qingqing.zhuo@amd.com; stylon.wang@amd.com; J=
osip.Pavic@amd.com; trix@redhat.com; cruise.hung@amd.com; Eric.Yang2@amd.co=
m; mario.limonciello@amd.com; alvin.lee2@amd.com; jun.lei@amd.com; austin.z=
heng@amd.com; sunglee@amd.com; paul.hsieh@amd.com; hanghong.ma@amd.com; Jin=
Ze.Xu@amd.com; lewis.huang@amd.com; Zhengzengkai <zhengzengkai@huawei.com>;=
 alex.hung@amd.com; syed.hassan@amd.com; wayne.lin@amd.com; nicholas.kazlau=
skas@amd.com; chiahsuan.chung@amd.com; aurabindo.pillai@amd.com; aric.cyr@a=
md.com; amd-gfx@lists.freedesktop.org; dri-devel@lists.freedesktop.org; lin=
ux-kernel@vger.kernel.org
> =E4=B8=BB=E9=A2=98: Re: [PATCH v6.6] drm/amd/display: Wake DMCUB before e=
xecuting GPINT commands
>
> On Tue, Apr 16, 2024 at 03:52:40AM +0000, Zhu Wang wrote:
> > From: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com>
> >
> > stable inclusion
> > from stable-v6.7.3
> > commit 2ef98c6d753a744e333b7e34b9cf687040fba57d
> > category: bugfix
> > bugzilla: https://gitee.com/src-openeuler/kernel/issues/I9BV4C
> > CVE: CVE-2023-52624
> >
> > Reference:
> > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/com
> > mit/?id=3D2ef98c6d753a744e333b7e34b9cf687040fba57d
> >
> > --------------------------------
> >
> > [ Upstream commit e5ffd1263dd5b44929c676171802e7b6af483f21 ]
> >
> > [Why]
> > DMCUB can be in idle when we attempt to interface with the HW through
> > the GPINT mailbox resulting in a system hang.
> >
> > [How]
> > Add dc_wake_and_execute_gpint() to wrap the wake, execute, sleep
> > sequence.
> >
> > If the GPINT executes successfully then DMCUB will be put back into
> > sleep after the optional response is returned.
> >
> > It functions similar to the inbox command interface.
> >
> > Cc: Mario Limonciello <mario.limonciello@amd.com>
> > Cc: Alex Deucher <alexander.deucher@amd.com>
> > Cc: stable@vger.kernel.org
> > Reviewed-by: Hansen Dsouza <hansen.dsouza@amd.com>
> > Acked-by: Wayne Lin <wayne.lin@amd.com>
> > Signed-off-by: Nicholas Kazlauskas <nicholas.kazlauskas@amd.com>
> > Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
> > Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
> >
> > This commit comes from following commits:
> >
> >  8774029f76b9 ("drm/amd/display: Add DCN35 CLK_MGR")  65138eb72e1f
> > ("drm/amd/display: Add DCN35 DMUB")  dc01c4b79bfe ("drm/amd/display:
> > Update driver and IPS interop")
> >  820c3870c491 ("drm/amd/display: Refactor DMCUB enter/exit idle
> > interface")  2ef98c6d753a ("drm/amd/display: Wake DMCUB before
> > executing GPINT commands")
>
> Why are you putting multiple commits together and not just submitting the=
 individual ones?  And what is this for?
>
> confused,
>
> greg k-h