Re: amdgpu: Enable full DCN support on POWER

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 2019-12-05 6:02 p.m., Liu, Zhan wrote:
> 
> 
>> -----Original Message-----
>> From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of
>> Timothy Pearson
>> Sent: 2019/December/05, Thursday 4:58 PM
>> To: amd-gfx <amd-gfx@xxxxxxxxxxxxxxxxxxxxx>
>> Subject: [PATCH] [RFC v2] amdgpu: Enable full DCN support on POWER
>>
>> DCN requires floating point support to operate.  Add the appropriate
>> x86/ppc64 guards and FPU / AltiVec / VSX context switches to DCN.
>>
>> Signed-off-by: Timothy Pearson <tpearson@xxxxxxxxxxxxxxxxxxxxx>
>> ---
>>  drivers/gpu/drm/amd/display/Kconfig           |   8 +-
>>  drivers/gpu/drm/amd/display/dc/Makefile       |   1 +
>>  drivers/gpu/drm/amd/display/dc/calcs/Makefile |   8 +
>>  .../gpu/drm/amd/display/dc/calcs/dcn_calcs.c  | 157 ++++++++++++++++++
>>  drivers/gpu/drm/amd/display/dc/dcn20/Makefile |   8 +
>>  .../drm/amd/display/dc/dcn20/dcn20_resource.c |  27 +++
>>  drivers/gpu/drm/amd/display/dc/dcn21/Makefile |   8 +
>>  .../drm/amd/display/dc/dcn21/dcn21_resource.c |  27 +++
>>  drivers/gpu/drm/amd/display/dc/dml/Makefile   |   9 +
>>  drivers/gpu/drm/amd/display/dc/dsc/Makefile   |   8 +
>>  drivers/gpu/drm/amd/display/dc/os_types.h     |   6 +
>>  11 files changed, 263 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/gpu/drm/amd/display/Kconfig
>> b/drivers/gpu/drm/amd/display/Kconfig
>> index 313183b80032..c73a63f3e245 100644
>> --- a/drivers/gpu/drm/amd/display/Kconfig
>> +++ b/drivers/gpu/drm/amd/display/Kconfig
>> @@ -6,7 +6,7 @@ config DRM_AMD_DC
>>  	bool "AMD DC - Enable new display engine"
>>  	default y
>>  	select SND_HDA_COMPONENT if SND_HDA_CORE
>> -	select DRM_AMD_DC_DCN1_0 if X86 && !(KCOV_INSTRUMENT_ALL
>> && KCOV_ENABLE_COMPARISONS)
>> +	select DRM_AMD_DC_DCN1_0 if (X86 || PPC64)
>> && !(KCOV_INSTRUMENT_ALL &&
>> +KCOV_ENABLE_COMPARISONS)
>>  	help
>>  	  Choose this option if you want to use the new display engine
>>  	  support for AMDGPU. This adds required support for Vega and @@
>> -20,7 +20,7 @@ config DRM_AMD_DC_DCN1_0  config
>> DRM_AMD_DC_DCN2_0
>>  	bool "DCN 2.0 family"
>>  	default y
>> -	depends on DRM_AMD_DC && X86
>> +	depends on DRM_AMD_DC && (X86 || PPC64)
>>  	depends on DRM_AMD_DC_DCN1_0
>>  	help
>>  	  Choose this option if you want to have @@ -28,7 +28,7 @@ config
>> DRM_AMD_DC_DCN2_0
>>
>>  config DRM_AMD_DC_DCN2_1
>>  	bool "DCN 2.1 family"
>> -	depends on DRM_AMD_DC && X86
>> +	depends on DRM_AMD_DC && (X86 || PPC64)
>>  	depends on DRM_AMD_DC_DCN2_0
>>  	help
>>  	  Choose this option if you want to have @@ -37,7 +37,7 @@ config
>> DRM_AMD_DC_DCN2_1  config DRM_AMD_DC_DSC_SUPPORT
>>  	bool "DSC support"
>>  	default y
>> -	depends on DRM_AMD_DC && X86
>> +	depends on DRM_AMD_DC && (X86 || PPC64)
>>  	depends on DRM_AMD_DC_DCN1_0
>>  	depends on DRM_AMD_DC_DCN2_0
>>  	help
>> diff --git a/drivers/gpu/drm/amd/display/dc/Makefile
>> b/drivers/gpu/drm/amd/display/dc/Makefile
>> index a160512a2f04..3e026a969386 100644
>> --- a/drivers/gpu/drm/amd/display/dc/Makefile
>> +++ b/drivers/gpu/drm/amd/display/dc/Makefile
>> @@ -1,5 +1,6 @@
>>  #
>>  # Copyright 2017 Advanced Micro Devices, Inc.
>> +# Copyright 2019 Raptor Engineering, LLC
> 
> NAK.
> 
> IANAL, but I don't think you can add your company's name by modifying part of the code. The copyright notice shows the authors of the original work.
> 
> When modifying the code, you are required to agree with that copyright notice. That's the purpose of that copyright notice piece.
> 

I always thought these copyright notices are nearly meaningless.

That said, this patch doesn't have any change in this file. I don't
think it warrants an additional copyright notice.

Harry

>>  #
>>  # Permission is hereby granted, free of charge, to any person obtaining a  #
>> copy of this software and associated documentation files (the "Software"),
>> diff --git a/drivers/gpu/drm/amd/display/dc/calcs/Makefile
>> b/drivers/gpu/drm/amd/display/dc/calcs/Makefile
>> index 26c6d735cdc7..20c88aff930a 100644
>> --- a/drivers/gpu/drm/amd/display/dc/calcs/Makefile
>> +++ b/drivers/gpu/drm/amd/display/dc/calcs/Makefile
>> @@ -24,7 +24,13 @@
>>  # It calculates Bandwidth and Watermarks values for HW programming  #
>>
>> +ifdef CONFIG_X86_64
>>  calcs_ccflags := -mhard-float -msse
>> +endif
>> +
>> +ifdef CONFIG_PPC64
>> +calcs_ccflags := -mhard-float -maltivec endif
>>
>>  ifdef CONFIG_CC_IS_GCC
>>  ifeq ($(call cc-ifversion, -lt, 0701, y), y) @@ -32,6 +38,7 @@ IS_OLD_GCC =
>> 1  endif  endif
>>
>> +ifdef CONFIG_X86_64
>>  ifdef IS_OLD_GCC
>>  # Stack alignment mismatch, proceed with caution.
>>  # GCC < 7.1 cannot compile code using `double` and -mpreferred-stack-
>> boundary=3 @@ -40,6 +47,7 @@ calcs_ccflags += -mpreferred-stack-
>> boundary=4  else  calcs_ccflags += -msse2  endif
>> +endif
>>
>>  CFLAGS_$(AMDDALPATH)/dc/calcs/dcn_calcs.o := $(calcs_ccflags)
>> CFLAGS_$(AMDDALPATH)/dc/calcs/dcn_calc_auto.o := $(calcs_ccflags) diff -
>> -git a/drivers/gpu/drm/amd/display/dc/calcs/dcn_calcs.c
>> b/drivers/gpu/drm/amd/display/dc/calcs/dcn_calcs.c
>> index 9b2cb57bf2ba..236e852ea60b 100644
>> --- a/drivers/gpu/drm/amd/display/dc/calcs/dcn_calcs.c
>> +++ b/drivers/gpu/drm/amd/display/dc/calcs/dcn_calcs.c
>> @@ -1,5 +1,6 @@
>>  /*
>>   * Copyright 2017 Advanced Micro Devices, Inc.
>> + * Copyright 2019 Raptor Engineering, LLC
>>   *
>>   * Permission is hereby granted, free of charge, to any person obtaining a
>>   * copy of this software and associated documentation files (the "Software"),
>> @@ -626,7 +627,20 @@ static bool dcn_bw_apply_registry_override(struct
>> dc *dc)  {
>>  	bool updated = false;
>>
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_begin();
>> +#elif defined(CONFIG_PPC64)
>> +	if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +		preempt_disable();
>> +		enable_kernel_vsx();
>> +	} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +		preempt_disable();
>> +		enable_kernel_altivec();
>> +	} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +		preempt_disable();
>> +		enable_kernel_fp();
>> +	}
>> +#endif
>>  	if ((int)(dc->dcn_soc->sr_exit_time * 1000) != dc-
>>> debug.sr_exit_time_ns
>>  			&& dc->debug.sr_exit_time_ns) {
>>  		updated = true;
>> @@ -662,7 +676,20 @@ static bool dcn_bw_apply_registry_override(struct
>> dc *dc)
>>  		dc->dcn_soc->dram_clock_change_latency =
>>  				dc->debug.dram_clock_change_latency_ns /
>> 1000.0;
>>  	}
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_end();
>> +#elif defined(CONFIG_PPC64)
>> +	if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +		disable_kernel_vsx();
>> +		preempt_enable();
>> +	} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +		disable_kernel_altivec();
>> +		preempt_enable();
>> +	} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +		disable_kernel_fp();
>> +		preempt_enable();
>> +	}
>> +#endif
>>
>>  	return updated;
>>  }
>> @@ -742,7 +769,20 @@ bool dcn_validate_bandwidth(
>>  		dcn_bw_sync_calcs_and_dml(dc);
>>
>>  	memset(v, 0, sizeof(*v));
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_begin();
>> +#elif defined(CONFIG_PPC64)
>> +	if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +		preempt_disable();
>> +		enable_kernel_vsx();
>> +	} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +		preempt_disable();
>> +		enable_kernel_altivec();
>> +	} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +		preempt_disable();
>> +		enable_kernel_fp();
>> +	}
>> +#endif
>>
>>  	v->sr_exit_time = dc->dcn_soc->sr_exit_time;
>>  	v->sr_enter_plus_exit_time = dc->dcn_soc->sr_enter_plus_exit_time;
>> @@ -1275,7 +1315,20 @@ bool dcn_validate_bandwidth(
>>  	bw_limit = dc->dcn_soc->percent_disp_bw_limit * v-
>>> fabric_and_dram_bandwidth_vmax0p9;
>>  	bw_limit_pass = (v->total_data_read_bandwidth / 1000.0) < bw_limit;
>>
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_end();
>> +#elif defined(CONFIG_PPC64)
>> +	if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +		disable_kernel_vsx();
>> +		preempt_enable();
>> +	} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +		disable_kernel_altivec();
>> +		preempt_enable();
>> +	} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +		disable_kernel_fp();
>> +		preempt_enable();
>> +	}
>> +#endif
>>
>>  	PERFORMANCE_TRACE_END();
>>  	BW_VAL_TRACE_FINISH();
>> @@ -1443,7 +1496,20 @@ void dcn_bw_update_from_pplib(struct dc *dc)
>>  	res = dm_pp_get_clock_levels_by_type_with_voltage(
>>  			ctx, DM_PP_CLOCK_TYPE_FCLK, &fclks);
>>
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_begin();
>> +#elif defined(CONFIG_PPC64)
>> +	if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +		preempt_disable();
>> +		enable_kernel_vsx();
>> +	} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +		preempt_disable();
>> +		enable_kernel_altivec();
>> +	} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +		preempt_disable();
>> +		enable_kernel_fp();
>> +	}
>> +#endif
>>
>>  	if (res)
>>  		res = verify_clock_values(&fclks);
>> @@ -1463,12 +1529,38 @@ void dcn_bw_update_from_pplib(struct dc *dc)
>>  	} else
>>  		BREAK_TO_DEBUGGER();
>>
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_end();
>> +#elif defined(CONFIG_PPC64)
>> +	if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +		disable_kernel_vsx();
>> +		preempt_enable();
>> +	} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +		disable_kernel_altivec();
>> +		preempt_enable();
>> +	} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +		disable_kernel_fp();
>> +		preempt_enable();
>> +	}
>> +#endif
>>
>>  	res = dm_pp_get_clock_levels_by_type_with_voltage(
>>  			ctx, DM_PP_CLOCK_TYPE_DCFCLK, &dcfclks);
>>
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_begin();
>> +#elif defined(CONFIG_PPC64)
>> +	if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +		preempt_disable();
>> +		enable_kernel_vsx();
>> +	} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +		preempt_disable();
>> +		enable_kernel_altivec();
>> +	} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +		preempt_disable();
>> +		enable_kernel_fp();
>> +	}
>> +#endif
>>
>>  	if (res)
>>  		res = verify_clock_values(&dcfclks);
>> @@ -1481,7 +1573,20 @@ void dcn_bw_update_from_pplib(struct dc *dc)
>>  	} else
>>  		BREAK_TO_DEBUGGER();
>>
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_end();
>> +#elif defined(CONFIG_PPC64)
>> +	if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +		disable_kernel_vsx();
>> +		preempt_enable();
>> +	} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +		disable_kernel_altivec();
>> +		preempt_enable();
>> +	} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +		disable_kernel_fp();
>> +		preempt_enable();
>> +	}
>> +#endif
>>  }
>>
>>  void dcn_bw_notify_pplib_of_wm_ranges(struct dc *dc) @@ -1496,11
>> +1601,37 @@ void dcn_bw_notify_pplib_of_wm_ranges(struct dc *dc)
>>  	if (!pp || !pp->set_wm_ranges)
>>  		return;
>>
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_begin();
>> +#elif defined(CONFIG_PPC64)
>> +	if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +		preempt_disable();
>> +		enable_kernel_vsx();
>> +	} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +		preempt_disable();
>> +		enable_kernel_altivec();
>> +	} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +		preempt_disable();
>> +		enable_kernel_fp();
>> +	}
>> +#endif
>>  	min_fclk_khz = dc->dcn_soc-
>>> fabric_and_dram_bandwidth_vmin0p65 * 1000000 / 32;
>>  	min_dcfclk_khz = dc->dcn_soc->dcfclkv_min0p65 * 1000;
>>  	socclk_khz = dc->dcn_soc->socclk * 1000;
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_end();
>> +#elif defined(CONFIG_PPC64)
>> +	if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +		disable_kernel_vsx();
>> +		preempt_enable();
>> +	} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +		disable_kernel_altivec();
>> +		preempt_enable();
>> +	} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +		disable_kernel_fp();
>> +		preempt_enable();
>> +	}
>> +#endif
>>
>>  	/* Now notify PPLib/SMU about which Watermarks sets they should
>> select
>>  	 * depending on DPM state they are in. And update BW MGR GFX
>> Engine and @@ -1551,7 +1682,20 @@ void
>> dcn_bw_notify_pplib_of_wm_ranges(struct dc *dc)
>>
>>  void dcn_bw_sync_calcs_and_dml(struct dc *dc)  {
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_begin();
>> +#elif defined(CONFIG_PPC64)
>> +	if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +		preempt_disable();
>> +		enable_kernel_vsx();
>> +	} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +		preempt_disable();
>> +		enable_kernel_altivec();
>> +	} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +		preempt_disable();
>> +		enable_kernel_fp();
>> +	}
>> +#endif
>>  	DC_LOG_BANDWIDTH_CALCS("sr_exit_time: %f ns\n"
>>  			"sr_enter_plus_exit_time: %f ns\n"
>>  			"urgent_latency: %f ns\n"
>> @@ -1740,5 +1884,18 @@ void dcn_bw_sync_calcs_and_dml(struct dc *dc)
>>  	dc->dml.ip.bug_forcing_LC_req_same_size_fixed =
>>  		dc->dcn_ip-
>>> bug_forcing_luma_and_chroma_request_to_same_size_fixed ==
>> dcn_bw_yes;
>>  	dc->dml.ip.dcfclk_cstate_latency = dc->dcn_ip-
>>> dcfclk_cstate_latency;
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_end();
>> +#elif defined(CONFIG_PPC64)
>> +	if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +		disable_kernel_vsx();
>> +		preempt_enable();
>> +	} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +		disable_kernel_altivec();
>> +		preempt_enable();
>> +	} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +		disable_kernel_fp();
>> +		preempt_enable();
>> +	}
>> +#endif
>>  }
>> diff --git a/drivers/gpu/drm/amd/display/dc/dcn20/Makefile
>> b/drivers/gpu/drm/amd/display/dc/dcn20/Makefile
>> index 63f3bddba7da..6a872b7a58bd 100644
>> --- a/drivers/gpu/drm/amd/display/dc/dcn20/Makefile
>> +++ b/drivers/gpu/drm/amd/display/dc/dcn20/Makefile
>> @@ -10,7 +10,13 @@ ifdef CONFIG_DRM_AMD_DC_DSC_SUPPORT
>>  DCN20 += dcn20_dsc.o
>>  endif
>>
>> +ifdef CONFIG_X86_64
>>  CFLAGS_$(AMDDALPATH)/dc/dcn20/dcn20_resource.o := -mhard-float -
>> msse
>> +endif
>> +
>> +ifdef CONFIG_PPC64
>> +CFLAGS_$(AMDDALPATH)/dc/dcn20/dcn20_resource.o := -mhard-float
>> +-maltivec endif
>>
>>  ifdef CONFIG_CC_IS_GCC
>>  ifeq ($(call cc-ifversion, -lt, 0701, y), y) @@ -18,6 +24,7 @@ IS_OLD_GCC =
>> 1  endif  endif
>>
>> +ifdef CONFIG_X86_64
>>  ifdef IS_OLD_GCC
>>  # Stack alignment mismatch, proceed with caution.
>>  # GCC < 7.1 cannot compile code using `double` and -mpreferred-stack-
>> boundary=3 @@ -26,6 +33,7 @@
>> CFLAGS_$(AMDDALPATH)/dc/dcn20/dcn20_resource.o += -mpreferred-
>> stack-boundary=4  else
>> CFLAGS_$(AMDDALPATH)/dc/dcn20/dcn20_resource.o += -msse2  endif
>> +endif
>>
>>  AMD_DAL_DCN20 = $(addprefix $(AMDDALPATH)/dc/dcn20/,$(DCN20))
>>
>> 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 bbd1c98564be..7e1da6b3ca36 100644
>> --- a/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c
>> +++ b/drivers/gpu/drm/amd/display/dc/dcn20/dcn20_resource.c
>> @@ -1,5 +1,6 @@
>>  /*
>>  * Copyright 2016 Advanced Micro Devices, Inc.
>> + * Copyright 2019 Raptor Engineering, LLC
>>   *
>>   * Permission is hereby granted, free of charge, to any person obtaining a
>>   * copy of this software and associated documentation files (the "Software"),
>> @@ -3173,7 +3174,20 @@ void dcn20_update_bounding_box(struct dc *dc,
>> struct _vcs_dpi_soc_bounding_box_s
>>
>>  void dcn20_patch_bounding_box(struct dc *dc, struct
>> _vcs_dpi_soc_bounding_box_st *bb)  {
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_begin();
>> +#elif defined(CONFIG_PPC64)
>> +		if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +			preempt_disable();
>> +			enable_kernel_vsx();
>> +		} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +			preempt_disable();
>> +			enable_kernel_altivec();
>> +		} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +			preempt_disable();
>> +			enable_kernel_fp();
>> +		}
>> +#endif
>>  	if ((int)(bb->sr_exit_time_us * 1000) != dc-
>>> bb_overrides.sr_exit_time_ns
>>  			&& dc->bb_overrides.sr_exit_time_ns) {
>>  		bb->sr_exit_time_us = dc->bb_overrides.sr_exit_time_ns /
>> 1000.0; @@ -3197,7 +3211,20 @@ void dcn20_patch_bounding_box(struct
>> dc *dc, struct _vcs_dpi_soc_bounding_box_st
>>  		bb->dram_clock_change_latency_us =
>>  				dc-
>>> bb_overrides.dram_clock_change_latency_ns / 1000.0;
>>  	}
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_end();
>> +#elif defined(CONFIG_PPC64)
>> +		if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +			disable_kernel_vsx();
>> +			preempt_enable();
>> +		} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +			disable_kernel_altivec();
>> +			preempt_enable();
>> +		} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +			disable_kernel_fp();
>> +			preempt_enable();
>> +		}
>> +#endif
>>  }
>>
>>  static struct _vcs_dpi_soc_bounding_box_st *get_asic_rev_soc_bb( diff --git
>> a/drivers/gpu/drm/amd/display/dc/dcn21/Makefile
>> b/drivers/gpu/drm/amd/display/dc/dcn21/Makefile
>> index 14113ccf498d..79bb926a6890 100644
>> --- a/drivers/gpu/drm/amd/display/dc/dcn21/Makefile
>> +++ b/drivers/gpu/drm/amd/display/dc/dcn21/Makefile
>> @@ -3,7 +3,13 @@
>>
>>  DCN21 = dcn21_hubp.o dcn21_hubbub.o dcn21_resource.o dcn21_hwseq.o
>> dcn21_link_encoder.o
>>
>> +ifdef CONFIG_X86_64
>>  CFLAGS_$(AMDDALPATH)/dc/dcn21/dcn21_resource.o := -mhard-float -
>> msse
>> +endif
>> +
>> +ifdef CONFIG_PPC64
>> +CFLAGS_$(AMDDALPATH)/dc/dcn21/dcn21_resource.o := -mhard-float
>> +-maltivec endif
>>
>>  ifdef CONFIG_CC_IS_GCC
>>  ifeq ($(call cc-ifversion, -lt, 0701, y), y) @@ -11,6 +17,7 @@ IS_OLD_GCC =
>> 1  endif  endif
>>
>> +ifdef CONFIG_X86_64
>>  ifdef IS_OLD_GCC
>>  # Stack alignment mismatch, proceed with caution.
>>  # GCC < 7.1 cannot compile code using `double` and -mpreferred-stack-
>> boundary=3 @@ -19,6 +26,7 @@
>> CFLAGS_$(AMDDALPATH)/dc/dcn21/dcn21_resource.o += -mpreferred-
>> stack-boundary=4  else
>> CFLAGS_$(AMDDALPATH)/dc/dcn21/dcn21_resource.o += -msse2  endif
>> +endif
>>
>>  AMD_DAL_DCN21 = $(addprefix $(AMDDALPATH)/dc/dcn21/,$(DCN21))
>>
>> 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 459bd9a5caed..8dcd871deceb 100644
>> --- a/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c
>> +++ b/drivers/gpu/drm/amd/display/dc/dcn21/dcn21_resource.c
>> @@ -1,5 +1,6 @@
>>  /*
>>  * Copyright 2018 Advanced Micro Devices, Inc.
>> + * Copyright 2019 Raptor Engineering, LLC
>>   *
>>   * Permission is hereby granted, free of charge, to any person obtaining a
>>   * copy of this software and associated documentation files (the "Software"),
>> @@ -1017,7 +1018,20 @@ static void calculate_wm_set_for_vlevel(
>>
>>  static void patch_bounding_box(struct dc *dc, struct
>> _vcs_dpi_soc_bounding_box_st *bb)  {
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_begin();
>> +#elif defined(CONFIG_PPC64)
>> +		if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +			preempt_disable();
>> +			enable_kernel_vsx();
>> +		} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +			preempt_disable();
>> +			enable_kernel_altivec();
>> +		} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +			preempt_disable();
>> +			enable_kernel_fp();
>> +		}
>> +#endif
>>  	if (dc->bb_overrides.sr_exit_time_ns) {
>>  		bb->sr_exit_time_us = dc->bb_overrides.sr_exit_time_ns /
>> 1000.0;
>>  	}
>> @@ -1035,7 +1049,20 @@ static void patch_bounding_box(struct dc *dc,
>> struct _vcs_dpi_soc_bounding_box_s
>>  		bb->dram_clock_change_latency_us =
>>  				dc-
>>> bb_overrides.dram_clock_change_latency_ns / 1000.0;
>>  	}
>> +#if defined(CONFIG_X86_64)
>>  	kernel_fpu_end();
>> +#elif defined(CONFIG_PPC64)
>> +		if (cpu_has_feature(CPU_FTR_VSX_COMP)) {
>> +			disable_kernel_vsx();
>> +			preempt_enable();
>> +		} else if (cpu_has_feature(CPU_FTR_ALTIVEC_COMP)) {
>> +			disable_kernel_altivec();
>> +			preempt_enable();
>> +		} else if (!cpu_has_feature(CPU_FTR_FPU_UNAVAILABLE)) {
>> +			disable_kernel_fp();
>> +			preempt_enable();
>> +		}
>> +#endif
>>  }
>>
>>  void dcn21_calculate_wm(
>> diff --git a/drivers/gpu/drm/amd/display/dc/dml/Makefile
>> b/drivers/gpu/drm/amd/display/dc/dml/Makefile
>> index 8df251626e22..ae49d23386e1 100644
>> --- a/drivers/gpu/drm/amd/display/dc/dml/Makefile
>> +++ b/drivers/gpu/drm/amd/display/dc/dml/Makefile
>> @@ -1,5 +1,6 @@
>>  #
>>  # Copyright 2017 Advanced Micro Devices, Inc.
>> +# Copyright 2019 Raptor Engineering, LLC
>>  #
>>  # Permission is hereby granted, free of charge, to any person obtaining a  #
>> copy of this software and associated documentation files (the "Software"),
>> @@ -24,7 +25,13 @@  # It provides the general basic services required by
>> other DAL  # subcomponents.
>>
>> +ifdef CONFIG_X86_64
>>  dml_ccflags := -mhard-float -msse
>> +endif
>> +
>> +ifdef CONFIG_PPC64
>> +dml_ccflags := -mhard-float -maltivec
>> +endif
>>
>>  ifdef CONFIG_CC_IS_GCC
>>  ifeq ($(call cc-ifversion, -lt, 0701, y), y) @@ -32,6 +39,7 @@ IS_OLD_GCC =
>> 1  endif  endif
>>
>> +ifdef CONFIG_X86_64
>>  ifdef IS_OLD_GCC
>>  # Stack alignment mismatch, proceed with caution.
>>  # GCC < 7.1 cannot compile code using `double` and -mpreferred-stack-
>> boundary=3 @@ -40,6 +48,7 @@ dml_ccflags += -mpreferred-stack-
>> boundary=4  else  dml_ccflags += -msse2  endif
>> +endif
>>
>>  CFLAGS_$(AMDDALPATH)/dc/dml/display_mode_lib.o := $(dml_ccflags)
>>
>> diff --git a/drivers/gpu/drm/amd/display/dc/dsc/Makefile
>> b/drivers/gpu/drm/amd/display/dc/dsc/Makefile
>> index 970737217e53..f73304af4b85 100644
>> --- a/drivers/gpu/drm/amd/display/dc/dsc/Makefile
>> +++ b/drivers/gpu/drm/amd/display/dc/dsc/Makefile
>> @@ -1,7 +1,13 @@
>>  #
>>  # Makefile for the 'dsc' sub-component of DAL.
>>
>> +ifdef CONFIG_X86_64
>>  dsc_ccflags := -mhard-float -msse
>> +endif
>> +
>> +ifdef CONFIG_PPC64
>> +dsc_ccflags := -mhard-float -maltivec
>> +endif
>>
>>  ifdef CONFIG_CC_IS_GCC
>>  ifeq ($(call cc-ifversion, -lt, 0701, y), y) @@ -9,6 +15,7 @@ IS_OLD_GCC = 1
>> endif  endif
>>
>> +ifdef CONFIG_X86_64
>>  ifdef IS_OLD_GCC
>>  # Stack alignment mismatch, proceed with caution.
>>  # GCC < 7.1 cannot compile code using `double` and -mpreferred-stack-
>> boundary=3 @@ -17,6 +24,7 @@ dsc_ccflags += -mpreferred-stack-
>> boundary=4  else  dsc_ccflags += -msse2  endif
>> +endif
>>
>>  CFLAGS_$(AMDDALPATH)/dc/dsc/rc_calc.o := $(dsc_ccflags)
>> CFLAGS_$(AMDDALPATH)/dc/dsc/rc_calc_dpi.o := $(dsc_ccflags) diff --git
>> a/drivers/gpu/drm/amd/display/dc/os_types.h
>> b/drivers/gpu/drm/amd/display/dc/os_types.h
>> index 30ec80ac6fc8..834e69b409d1 100644
>> --- a/drivers/gpu/drm/amd/display/dc/os_types.h
>> +++ b/drivers/gpu/drm/amd/display/dc/os_types.h
>> @@ -1,5 +1,6 @@
>>  /*
>>   * Copyright 2012-16 Advanced Micro Devices, Inc.
>> + * Copyright 2019 Raptor Engineering, LLC
>>   *
>>   * Permission is hereby granted, free of charge, to any person obtaining a
>>   * copy of this software and associated documentation files (the "Software"),
>> @@ -49,7 +50,12 @@  #define dm_error(fmt, ...) DRM_ERROR(fmt,
>> ##__VA_ARGS__)
>>
>>  #if defined(CONFIG_DRM_AMD_DC_DCN1_0)
>> +#if defined(CONFIG_X86_64)
>>  #include <asm/fpu/api.h>
>> +#elif defined(CONFIG_PPC32) || defined(CONFIG_PPC64) #include
>> +<asm/switch_to.h> #include <asm/cputable.h> #endif
>>  #endif
>>
>>  /*
>> --
>> 2.20.1
>> _______________________________________________
>> amd-gfx mailing list
>> amd-gfx@xxxxxxxxxxxxxxxxxxxxx
>> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.
>> freedesktop.org%2Fmailman%2Flistinfo%2Famd-
>> gfx&amp;data=02%7C01%7Czhan.liu%40amd.com%7Ced80e6447d544a463
>> 8d808d779ce451e%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7
>> C637111799221875534&amp;sdata=S4qa%2BDK53ziQ4EWaWCUSC0udcj84q
>> jz6qIzeiVg7el0%3D&amp;reserved=0
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
> 
_______________________________________________
amd-gfx mailing list
amd-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/amd-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux