Re: [dpdk-dev] [PATCH v14 00/12] Arm build options rework
From: Honnappa Nagarahalli <hidden>
Date: 2021-01-27 15:18:35
<snip>
quoted
quoted
quoted
quoted
quoted
quoted
v14: Resent for retesting (because of suspected false negative). Series Acked-by: Jerin Jacob [off-list ref] Series Tested-by:quoted
quoted
quoted
quoted
Jerin Jacob [off-list ref] Series Tested-by: DharmikThakkarquoted
quoted
quoted
quoted
[off-list ref] Series Tested-by: Vimal Chungath [off-list ref]Series Tested-by: Pavan Nikhilesh [off-list ref] Also, I believe we need to take a look at ARCH_SOC_ID supportcomingquoted
quoted
quoted
up in 5.9 kernel. https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F %2 Furldefense.proofpoint.com%2Fv2%2Furl%3Fu%3Dhttps-&data=04%7Cquoted
quoted
quoted
quoted
01%7Chemant.agrawal%40nxp.com%7Cf05a2623ec9347bd254708d8c2d4896quoted
c%quoted
quoted
quoted
quoted
7C686ea1d3bc2b4c6fa92cd99c5c301635%7C0%7C0%7C637473565383043785 %7quoted
quoted
quoted
quoted
CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJ Bquoted
Tiquoted
quoted
quoted
quoted
I6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=o7%2BXNileFgI2By08IHvquoted
EYquoted
quoted
quoted
quoted
PWrDjRikXlfFtal1MCdyCM%3D&reserved=03A__lore.kernel.org_linux-2Darm-2Dkernel_20200506164411.3284- 2D1- 2D&d=DwIGaQ&c=nKjWec2b6R0mOyPaz7xtfQ&r=1cjuAHrGh745jHNmj 2fD85sUMIJ2IPIDsIJzo6FN6Z0&m=A7kOn5DYMe_WZiykt8BVATgv7- qTVhfo4iSgHBg- BTo&s=dslFI0W1qZdDxkbXDMTzPSUzlKMFfMUsE6tllPaF9Oc&e=quoted
quoted
quoted
sudeep.holla@arm.com/Agree, it can come after this series is merged. Note that it requires firmware with SMCCC v1.2 support in the underlying platform.I have a few questions to everyone: Is there any comprehensive documentation about how to use this?Thingsquoted
like where in Linux can I get the ARCH_SOC_ID and how are the ID'smappedquoted
to socs?Kernel patch is at [1] - merged in Kernel 5.9 Makes use of SMCCC v1.2, SMCCC_ARCH_SOC_ID call. SMCCC v1.2 spec is at [2]. The SoC ID becomes available at: /sys/devices/socX/family /sys/devices/socX/soc_id /sys/devices/socX/revision SOC_ID makes use of JEP-106 code [3] for the SiP which makes it unique. [1] https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Furl dequoted
fense.proofpoint.com%2Fv2%2Furl%3Fu%3Dhttps-&data=04%7C01%7Chemant. agrawal%40nxp.com%7Cf05a2623ec9347bd254708d8c2d4896c%7C686ea1d 3bc2b4c6fquoted
a92cd99c5c301635%7C0%7C0%7C637473565383043785%7CUnknown%7CTWFpbGZsb3d8e yJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3 D%7C100 0&sdata=o7%2BXNileFgI2By08IHvEYPWrDjRikXlfFtal1MCdyCM%3D&a mp;reservquoted
ed=03A__lkml.org_lkml_2020_6_25_208&d=DwIGaQ&c=nKjWec2b6R0mOyquoted
quoted
Paz7xtfQ&r=1cjuAHrGh745jHNmj2fD85sUMIJ2IPIDsIJzo6FN6Z0&m=A7k On5DYMe_WZiykt8BVATgv7-qTVhfo4iSgHBg- BTo&s=5xXdzD7DpbcDeG81JVGZf396EFNbV0rSl00hRLuZJBc&e= [2] https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Furl dequoted
fense.proofpoint.com%2Fv2%2Furl%3Fu%3Dhttps-&data=04%7C01%7Chemant. agrawal%40nxp.com%7Cf05a2623ec9347bd254708d8c2d4896c%7C686ea1d 3bc2b4c6fquoted
a92cd99c5c301635%7C0%7C0%7C637473565383043785%7CUnknown%7CTWFpbGZsb3d8e yJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3 D%7C100 0&sdata=o7%2BXNileFgI2By08IHvEYPWrDjRikXlfFtal1MCdyCM%3D&a mp;reservquoted
ed=03A__developer.arm.com_documentation_den0028_c&d=DwIGaQ&c=nquoted
KjWec2b6R0mOyPaz7xtfQ&r=1cjuAHrGh745jHNmj2fD85sUMIJ2IPIDsIJz o6FN6Z0&m=A7kOn5DYMe_WZiykt8BVATgv7-qTVhfo4iSgHBg- BTo&s=DFG7a9oFDFczCcODfR0wVGjkLQXwMU19q_ogaFVw90I&e= [3] https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Furl dequoted
fense.proofpoint.com%2Fv2%2Furl%3Fu%3Dhttps-&data=04%7C01%7Chemant. agrawal%40nxp.com%7Cf05a2623ec9347bd254708d8c2d4896c%7C686ea1d 3bc2b4c6fquoted
a92cd99c5c301635%7C0%7C0%7C637473565383043785%7CUnknown%7CTWFpbGZsb3d8e yJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3 D%7C100 0&sdata=o7%2BXNileFgI2By08IHvEYPWrDjRikXlfFtal1MCdyCM%3D&a mp;reservquoted
ed=0 3A__developer.arm.com_documentation_ka001301_1- 2D0&d=DwIGaQ&c=nKjWec2b6R0mOyPaz7xtfQ&r=1cjuAHrGh745jHNm j2fD85sUMIJ2IPIDsIJzo6FN6Z0&m=A7kOn5DYMe_WZiykt8BVATgv7- qTVhfo4iSgHBg- BTo&s=GQsJJuft9k1iooO8VNNIM8ZUUgGjCEPB6eRxN3qm62U&e=quoted
If the ID mappings are not public then the plaform owners would needtoquoted
add the support, right?I agree here. There are firmware changes required and we do not know the IDs as well. It will be good for Marvell/NXP/NVIDIA to do this. Pavan, is this ok for you?Yeah, I believe we need fallback to the current mechanisms if SOC_ID support is not present or platform owners have not updated them.[Hemant] +1
Agree. Just to be clear, my ask is, can any of you take this work as Arm will not have all the parts to test this?