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] [day] [month] [year] [list]
Message-ID: <e4de1a96-d669-1923-8677-026da2dc5585@amd.com>
Date:   Tue, 7 Mar 2023 17:13:29 -0700
From:   Rodrigo Siqueira Jordao <Rodrigo.Siqueira@....com>
To:     David Tadokoro <davidbtadokoro@....br>, harry.wentland@....com,
        sunpeng.li@....com, alexander.deucher@....com,
        christian.koenig@....com, Xinhui.Pan@....com, airlied@...il.com,
        daniel@...ll.ch
Cc:     amd-gfx@...ts.freedesktop.org, dri-devel@...ts.freedesktop.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] drm/amd/display: remove legacy fields of dc_plane_cap
 struct



On 3/7/23 15:53, David Tadokoro wrote:
> The fields blends_with_above and blends_with_below of struct
> dc_plane_cap (defined in dc/dc.h) are boolean and set to true by
> default. All instances of a dc_plane_cap maintain the default values of
> both. Also, there is only one if statement that checks those fields and
> there would be the same effect if it was deleted (assuming that those
> fields are always going to be true).
> 
> For this reason, considering both fields as legacy ones, this commit
> removes them and the aforementioned if statement.
> 
> Signed-off-by: David Tadokoro <davidbtadokoro@....br>
> ---
>   drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c       | 3 ---
>   drivers/gpu/drm/amd/display/dc/dc.h                     | 2 --
>   drivers/gpu/drm/amd/display/dc/dce110/dce110_resource.c | 3 ---
>   drivers/gpu/drm/amd/display/dc/dcn10/dcn10_resource.c   | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c   | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn201/dcn201_resource.c | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c   | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn30/dcn30_resource.c   | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn301/dcn301_resource.c | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn302/dcn302_resource.c | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn303/dcn303_resource.c | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn31/dcn31_resource.c   | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn314/dcn314_resource.c | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn315/dcn315_resource.c | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn316/dcn316_resource.c | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource.c   | 2 --
>   drivers/gpu/drm/amd/display/dc/dcn321/dcn321_resource.c | 2 --
>   17 files changed, 36 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> index b472931cb7ca..fdcb375e908a 100644
> --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
> @@ -4354,9 +4354,6 @@ static int amdgpu_dm_initialize_drm_device(struct amdgpu_device *adev)
>   		if (plane->type != DC_PLANE_TYPE_DCN_UNIVERSAL)
>   			continue;
>   
> -		if (!plane->blends_with_above || !plane->blends_with_below)
> -			continue;
> -
>   		if (!plane->pixel_format_support.argb8888)
>   			continue;
>   
> diff --git a/drivers/gpu/drm/amd/display/dc/dc.h b/drivers/gpu/drm/amd/display/dc/dc.h
> index f0a1934ebf8c..ccc27d482640 100644
> --- a/drivers/gpu/drm/amd/display/dc/dc.h
> +++ b/drivers/gpu/drm/amd/display/dc/dc.h
> @@ -82,8 +82,6 @@ enum det_size {
>   
>   struct dc_plane_cap {
>   	enum dc_plane_type type;
> -	uint32_t blends_with_above : 1;
> -	uint32_t blends_with_below : 1;
>   	uint32_t per_pixel_alpha : 1;
>   	struct {
>   		uint32_t argb8888 : 1;
> diff --git a/drivers/gpu/drm/amd/display/dc/dce110/dce110_resource.c b/drivers/gpu/drm/amd/display/dc/dce110/dce110_resource.c
> index f808315b2835..a4a45a6ce61e 100644
> --- a/drivers/gpu/drm/amd/display/dc/dce110/dce110_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dce110/dce110_resource.c
> @@ -401,8 +401,6 @@ static const struct resource_caps stoney_resource_cap = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   		.type = DC_PLANE_TYPE_DCE_RGB,
> -		.blends_with_below = true,
> -		.blends_with_above = true,
>   		.per_pixel_alpha = 1,
>   
>   		.pixel_format_support = {
> @@ -428,7 +426,6 @@ static const struct dc_plane_cap plane_cap = {
>   
>   static const struct dc_plane_cap underlay_plane_cap = {
>   		.type = DC_PLANE_TYPE_DCE_UNDERLAY,
> -		.blends_with_above = true,
>   		.per_pixel_alpha = 1,
>   
>   		.pixel_format_support = {
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_resource.c b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_resource.c
> index 6bfac8088ab0..2bb8e11f26e0 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn10/dcn10_resource.c
> @@ -504,8 +504,6 @@ static const struct resource_caps rv2_res_cap = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   	.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -	.blends_with_above = true,
> -	.blends_with_below = true,
>   	.per_pixel_alpha = true,
>   
>   	.pixel_format_support = {
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c
> index 3af24ef9cb2d..00668df0938e 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c
> @@ -670,8 +670,6 @@ static const struct resource_caps res_cap_nv10 = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   	.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -	.blends_with_above = true,
> -	.blends_with_below = true,
>   	.per_pixel_alpha = true,
>   
>   	.pixel_format_support = {
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn201/dcn201_resource.c b/drivers/gpu/drm/amd/display/dc/dcn201/dcn201_resource.c
> index cd46701398d9..6ea70da28aaa 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn201/dcn201_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn201/dcn201_resource.c
> @@ -571,8 +571,6 @@ static const struct resource_caps res_cap_dnc201 = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   	.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -	.blends_with_above = true,
> -	.blends_with_below = true,
>   	.per_pixel_alpha = true,
>   
>   	.pixel_format_support = {
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c b/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c
> index 8f9244fe5c86..3ac8c0282589 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c
> @@ -609,8 +609,6 @@ static const struct resource_caps res_cap_rn_FPGA_2pipe_dsc = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   	.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -	.blends_with_above = true,
> -	.blends_with_below = true,
>   	.per_pixel_alpha = true,
>   
>   	.pixel_format_support = {
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_resource.c b/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_resource.c
> index b5b5320c7bef..d60c17d5a0d8 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn30/dcn30_resource.c
> @@ -680,8 +680,6 @@ static const struct resource_caps res_cap_dcn3 = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   	.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -	.blends_with_above = true,
> -	.blends_with_below = true,
>   	.per_pixel_alpha = true,
>   
>   	.pixel_format_support = {
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn301/dcn301_resource.c b/drivers/gpu/drm/amd/display/dc/dcn301/dcn301_resource.c
> index ee62ae3eb98f..b93b4498dba4 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn301/dcn301_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn301/dcn301_resource.c
> @@ -651,8 +651,6 @@ static struct resource_caps res_cap_dcn301 = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   	.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -	.blends_with_above = true,
> -	.blends_with_below = true,
>   	.per_pixel_alpha = true,
>   
>   	.pixel_format_support = {
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn302/dcn302_resource.c b/drivers/gpu/drm/amd/display/dc/dcn302/dcn302_resource.c
> index 03ddf4f5f065..6ccad53f1e49 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn302/dcn302_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn302/dcn302_resource.c
> @@ -147,8 +147,6 @@ static const struct resource_caps res_cap_dcn302 = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   		.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -		.blends_with_above = true,
> -		.blends_with_below = true,
>   		.per_pixel_alpha = true,
>   		.pixel_format_support = {
>   				.argb8888 = true,
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn303/dcn303_resource.c b/drivers/gpu/drm/amd/display/dc/dcn303/dcn303_resource.c
> index 727f458f6ee9..5c28f7151d13 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn303/dcn303_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn303/dcn303_resource.c
> @@ -126,8 +126,6 @@ static const struct resource_caps res_cap_dcn303 = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   		.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -		.blends_with_above = true,
> -		.blends_with_below = true,
>   		.per_pixel_alpha = true,
>   		.pixel_format_support = {
>   				.argb8888 = true,
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_resource.c b/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_resource.c
> index d3918a10773a..eaaa2e01f6d0 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn31/dcn31_resource.c
> @@ -827,8 +827,6 @@ static const struct resource_caps res_cap_dcn31 = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   	.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -	.blends_with_above = true,
> -	.blends_with_below = true,
>   	.per_pixel_alpha = true,
>   
>   	.pixel_format_support = {
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn314/dcn314_resource.c b/drivers/gpu/drm/amd/display/dc/dcn314/dcn314_resource.c
> index f9dfbc7407ee..50ed7e09d5ba 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn314/dcn314_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn314/dcn314_resource.c
> @@ -855,8 +855,6 @@ static const struct resource_caps res_cap_dcn314 = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   	.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -	.blends_with_above = true,
> -	.blends_with_below = true,
>   	.per_pixel_alpha = true,
>   
>   	.pixel_format_support = {
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn315/dcn315_resource.c b/drivers/gpu/drm/amd/display/dc/dcn315/dcn315_resource.c
> index 7887078c5f64..41c972c8eb19 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn315/dcn315_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn315/dcn315_resource.c
> @@ -824,8 +824,6 @@ static const struct resource_caps res_cap_dcn31 = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   	.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -	.blends_with_above = true,
> -	.blends_with_below = true,
>   	.per_pixel_alpha = true,
>   
>   	.pixel_format_support = {
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn316/dcn316_resource.c b/drivers/gpu/drm/amd/display/dc/dcn316/dcn316_resource.c
> index dc0b49506275..9ead347a33e9 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn316/dcn316_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn316/dcn316_resource.c
> @@ -824,8 +824,6 @@ static const struct resource_caps res_cap_dcn31 = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   	.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -	.blends_with_above = true,
> -	.blends_with_below = true,
>   	.per_pixel_alpha = true,
>   
>   	.pixel_format_support = {
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource.c b/drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource.c
> index 87f7669e81d7..100b6df33b33 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn32/dcn32_resource.c
> @@ -657,8 +657,6 @@ static const struct resource_caps res_cap_dcn32 = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   	.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -	.blends_with_above = true,
> -	.blends_with_below = true,
>   	.per_pixel_alpha = true,
>   
>   	.pixel_format_support = {
> diff --git a/drivers/gpu/drm/amd/display/dc/dcn321/dcn321_resource.c b/drivers/gpu/drm/amd/display/dc/dcn321/dcn321_resource.c
> index deaa4769be10..0f477d50e935 100644
> --- a/drivers/gpu/drm/amd/display/dc/dcn321/dcn321_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/dcn321/dcn321_resource.c
> @@ -655,8 +655,6 @@ static const struct resource_caps res_cap_dcn321 = {
>   
>   static const struct dc_plane_cap plane_cap = {
>   	.type = DC_PLANE_TYPE_DCN_UNIVERSAL,
> -	.blends_with_above = true,
> -	.blends_with_below = true,
>   	.per_pixel_alpha = true,
>   
>   	.pixel_format_support = {

Reviewed-by: Rodrigo Siqueira <Rodrigo.Siqueira@....com>

I also applied your change to amd-staging-drm-next.

Thanks!
Siqueira

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ