* Re: [dpdk-ci] [dpdk-dev] [PATCH] crypto/octeontx: enable build on non Linux OS [not found] ` <1815797.mnHlnkzLNo@thomas> @ 2021-07-22 19:08 ` Thomas Monjalon 2021-07-22 20:20 ` Brandon Lo 0 siblings, 1 reply; 3+ messages in thread From: Thomas Monjalon @ 2021-07-22 19:08 UTC (permalink / raw) To: dpdklab, lylavoie, Brandon Lo Cc: Shijith Thotton, Akhil Goyal, david.marchand, dev, aconole, ci 22/07/2021 21:06, Thomas Monjalon: > 22/07/2021 11:17, Akhil Goyal: > > > Enabled build of Octeontx crypto PMD on non linux OS. Other Octeontx > > > PMDs are enabled already. > > > > > > This is to avoid ABI test failure on an OS once we add dependency > > > between a driver which is built to another which is not. > > > > Fixes: 8dc6c2f12ecf ("crypto/octeontx: add crypto adapter framework") > > > > > > > Reported-by: David Marchand <david.marchand@redhat.com> > > > > > Signed-off-by: Shijith Thotton <sthotton@marvell.com> > > > > Acked-by: Akhil Goyal <gakhil@marvell.com> > > > > Thomas/David: please pick this patch directly on main to fix build on CI for FreeBSD. > > Applied, thanks. Please could you re-test the ABI on FreeBSD and re-enable in the CI if the test is passing? Thank you ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [dpdk-ci] [dpdk-dev] [PATCH] crypto/octeontx: enable build on non Linux OS 2021-07-22 19:08 ` [dpdk-ci] [dpdk-dev] [PATCH] crypto/octeontx: enable build on non Linux OS Thomas Monjalon @ 2021-07-22 20:20 ` Brandon Lo 2021-07-22 20:32 ` Thomas Monjalon 0 siblings, 1 reply; 3+ messages in thread From: Brandon Lo @ 2021-07-22 20:20 UTC (permalink / raw) To: Thomas Monjalon Cc: dpdklab, lylavoie, Shijith Thotton, Akhil Goyal, david.marchand, dev, aconole, ci On Thu, Jul 22, 2021 at 3:08 PM Thomas Monjalon <thomas@monjalon.net> wrote: > > 22/07/2021 21:06, Thomas Monjalon: > > 22/07/2021 11:17, Akhil Goyal: > > > > Enabled build of Octeontx crypto PMD on non linux OS. Other Octeontx > > > > PMDs are enabled already. > > > > > > > > This is to avoid ABI test failure on an OS once we add dependency > > > > between a driver which is built to another which is not. > > > > > > Fixes: 8dc6c2f12ecf ("crypto/octeontx: add crypto adapter framework") > > > > > > > > > > Reported-by: David Marchand <david.marchand@redhat.com> > > > > > > > Signed-off-by: Shijith Thotton <sthotton@marvell.com> > > > > > > Acked-by: Akhil Goyal <gakhil@marvell.com> > > > > > > Thomas/David: please pick this patch directly on main to fix build on CI for FreeBSD. > > > > Applied, thanks. > > Please could you re-test the ABI on FreeBSD > and re-enable in the CI if the test is passing? > > Thank you I ran a couple test runs on FreeBSD 13 to ensure that the patch compiles successfully, and I enabled reporting. FreeBSD 13 should start to appear in the ABI test results of newer tarballs with the patch. Thanks, Brandon -- Brandon Lo UNH InterOperability Laboratory 21 Madbury Rd, Suite 100, Durham, NH 03824 blo@iol.unh.edu www.iol.unh.edu ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [dpdk-ci] [dpdk-dev] [PATCH] crypto/octeontx: enable build on non Linux OS 2021-07-22 20:20 ` Brandon Lo @ 2021-07-22 20:32 ` Thomas Monjalon 0 siblings, 0 replies; 3+ messages in thread From: Thomas Monjalon @ 2021-07-22 20:32 UTC (permalink / raw) To: Brandon Lo Cc: dpdklab, lylavoie, Shijith Thotton, Akhil Goyal, david.marchand, dev, aconole, ci 22/07/2021 22:20, Brandon Lo: > On Thu, Jul 22, 2021 at 3:08 PM Thomas Monjalon <thomas@monjalon.net> wrote: > > > > 22/07/2021 21:06, Thomas Monjalon: > > > 22/07/2021 11:17, Akhil Goyal: > > > > > Enabled build of Octeontx crypto PMD on non linux OS. Other Octeontx > > > > > PMDs are enabled already. > > > > > > > > > > This is to avoid ABI test failure on an OS once we add dependency > > > > > between a driver which is built to another which is not. > > > > > > > > Fixes: 8dc6c2f12ecf ("crypto/octeontx: add crypto adapter framework") > > > > > > > > > > > > > Reported-by: David Marchand <david.marchand@redhat.com> > > > > > > > > > Signed-off-by: Shijith Thotton <sthotton@marvell.com> > > > > > > > > Acked-by: Akhil Goyal <gakhil@marvell.com> > > > > > > > > Thomas/David: please pick this patch directly on main to fix build on CI for FreeBSD. > > > > > > Applied, thanks. > > > > Please could you re-test the ABI on FreeBSD > > and re-enable in the CI if the test is passing? > > > > Thank you > > I ran a couple test runs on FreeBSD 13 to ensure that the patch > compiles successfully, and I enabled reporting. > FreeBSD 13 should start to appear in the ABI test results of newer > tarballs with the patch. Thanks a lot Brandon, well managed. ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-07-22 20:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <CO6PR18MB4484E8FF18760E91DD96AFE9D8E49@CO6PR18MB4484.namprd18.prod.outlook.com>
[not found] ` <CO6PR18MB4484AED5BD861E05C8B1CC3BD8E49@CO6PR18MB4484.namprd18.prod.outlook.com>
[not found] ` <1815797.mnHlnkzLNo@thomas>
2021-07-22 19:08 ` [dpdk-ci] [dpdk-dev] [PATCH] crypto/octeontx: enable build on non Linux OS Thomas Monjalon
2021-07-22 20:20 ` Brandon Lo
2021-07-22 20:32 ` Thomas Monjalon
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).