Revert "fix: call isPrefetchDisablingRequired directly from release helper"

This reverts commit 4ecff31c26.

Signed-off-by: Kamil Kopryk <kamil.kopryk@intel.com>
This commit is contained in:
Kamil Kopryk
2023-05-25 11:13:16 +00:00
committed by Compute-Runtime-Automation
parent cefef2d404
commit ab1b4681cc
7 changed files with 23 additions and 3 deletions

View File

@@ -116,6 +116,7 @@ class ProductHelper {
virtual bool imagePitchAlignmentWARequired(const HardwareInfo &hwInfo) const = 0;
virtual bool isForceEmuInt32DivRemSPWARequired(const HardwareInfo &hwInfo) const = 0;
virtual bool is3DPipelineSelectWARequired() const = 0;
virtual bool isPrefetchDisablingRequired(const ReleaseHelper *releaseHelper) const = 0;
virtual bool isStorageInfoAdjustmentRequired() const = 0;
virtual bool isBlitterForImagesSupported() const = 0;
virtual bool isPageFaultSupported() const = 0;

View File

@@ -288,6 +288,14 @@ template <PRODUCT_FAMILY gfxProduct>
int ProductHelperHw<gfxProduct>::getProductMaxPreferredSlmSize(const HardwareInfo &hwInfo, int preferredEnumValue) const {
return preferredEnumValue;
}
template <PRODUCT_FAMILY gfxProduct>
bool ProductHelperHw<gfxProduct>::isPrefetchDisablingRequired(const ReleaseHelper *releaseHelper) const {
if (releaseHelper) {
return releaseHelper->isPrefetchDisablingRequired();
}
return false;
}
template <PRODUCT_FAMILY gfxProduct>
bool ProductHelperHw<gfxProduct>::isAssignEngineRoundRobinSupported() const {
return false;

View File

@@ -70,6 +70,7 @@ class ProductHelperHw : public ProductHelper {
bool isForceEmuInt32DivRemSPWARequired(const HardwareInfo &hwInfo) const override;
bool is3DPipelineSelectWARequired() const override;
bool isStorageInfoAdjustmentRequired() const override;
bool isPrefetchDisablingRequired(const ReleaseHelper *releaseHelper) const override;
bool isBlitterForImagesSupported() const override;
bool isPageFaultSupported() const override;
bool isKmdMigrationSupported() const override;

View File

@@ -115,8 +115,9 @@ void EncodeDispatchKernel<Family>::programBarrierEnable(INTERFACE_DESCRIPTOR_DAT
template <>
void EncodeDispatchKernel<Family>::encodeAdditionalWalkerFields(const RootDeviceEnvironment &rootDeviceEnvironment, WALKER_TYPE &walkerCmd, const EncodeWalkerArgs &walkerArgs) {
auto &productHelper = rootDeviceEnvironment.getHelper<ProductHelper>();
auto *releaseHelper = rootDeviceEnvironment.getReleaseHelper();
bool l3PrefetchDisable = releaseHelper->isPrefetchDisablingRequired();
bool l3PrefetchDisable = productHelper.isPrefetchDisablingRequired(releaseHelper);
int32_t overrideL3PrefetchDisable = DebugManager.flags.ForceL3PrefetchForComputeWalker.get();
if (overrideL3PrefetchDisable != -1) {
l3PrefetchDisable = !overrideL3PrefetchDisable;