Skip to content

Conversation

@jinyouzhi
Copy link
Contributor

@jinyouzhi jinyouzhi commented Aug 24, 2023

PR types

Others

PR changes

Others

Description

https://github.com/xianyi/OpenBLAS/blob/394a9fbafe9010b76a2615c562204277a956eb52/Changelog.txt#L54
https://github.com/xianyi/OpenBLAS/blob/394a9fbafe9010b76a2615c562204277a956eb52/Changelog.txt#L17

OpenBLAS 0.3.22 开始支持Raptor Lake,当前 0.3.18 版本不能在 Xeon 4th CPU上编译(当WITH_MKL=OFF时)。

@jinyouzhi
Copy link
Contributor Author

@jiweibo 注意到注释中此前升级曾对IDG业务线有实际影响,能否帮忙确认新版本

@paddle-bot paddle-bot bot added the contributor External developers label Aug 24, 2023
Copy link
Contributor

@risemeup1 risemeup1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个改动没有生效,需要改动third_party目录下的openbals版本

@jinyouzhi
Copy link
Contributor Author

@risemeup1

这个改动没有生效,需要改动third_party目录下的openbals版本

改好了

# OpenBLAS support Raptor Lake from v0.3.22
if(UNIX
AND NOT APPLE
AND NOT WITH_ROCM
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里加的这些判断是为啥?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

参考这个以前 #53748

Copy link
Contributor

@risemeup1 risemeup1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

# OpenBLAS support Raptor Lake from v0.3.22
if(UNIX
AND NOT APPLE
AND NOT WITH_ROCM
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

先确定下在这几个环境下 v0.3.22 能否正常编译,根据情况再添加上吧

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR可以稍后合入哈@risemeup1,我们先验证下新版本下公司内部业务有没有问题~

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

条件有限,我先用CI测一下 #56806

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

可合~

@luotao1 luotao1 merged commit 15728a7 into PaddlePaddle:develop Sep 1, 2023
@GreatV
Copy link
Contributor

GreatV commented Sep 3, 2023

hello, @jinyouzhi @risemeup1 我在本地Raptor Lake (Core i9 13900KF 上)机器上编译会报错。
image

编译选项

cmake .. -DWITH_GPU=OFF -DWITH_MKLDNN=OFF -DWITH_MKL=OFF

编译器
gcc 8.2

@jinyouzhi
Copy link
Contributor Author

hello, @jinyouzhi @risemeup1 我在本地Raptor Lake (Core i9 13900KF 上)机器上编译会报错。 image

编译选项

cmake .. -DWITH_GPU=OFF -DWITH_MKLDNN=OFF -DWITH_MKL=OFF

编译器 gcc 8.2

刚刚在一台 Core i7 13700 上可以编译成功,编译器gcc 11.4,你可以检查一下你的目录Paddle/third-party/openblas以及Paddle/build/third-party/openblas 里面是指定版本吗?

@GreatV
Copy link
Contributor

GreatV commented Sep 3, 2023

@jinyouzhi 我这边 gcc 8.2 不行

@jinyouzhi
Copy link
Contributor Author

@jinyouzhi 我这边 gcc 8.2 不行

我再尝试复现一下吧

@jinyouzhi
Copy link
Contributor Author

@GreatV 我能复现了,thanks

BeingGod pushed a commit to BeingGod/Paddle that referenced this pull request Sep 9, 2023
…lePaddle#56617)

* [3rd-party] update openBLAS to fix compile error on Raptor Lake

* modify comments and remove the gcc version limit

* switch third-party/openblas to v0.3.23

* fix
risemeup1 added a commit that referenced this pull request Oct 13, 2023
risemeup1 added a commit that referenced this pull request Oct 16, 2023
jiahy0825 pushed a commit to jiahy0825/Paddle that referenced this pull request Oct 26, 2023
danleifeng pushed a commit to danleifeng/Paddle that referenced this pull request Nov 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor External developers

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants