# \[PATCH RFC 30/38\] powerpc/pseries/lparcfg: return h\_get\_ppp()'s status as a long - From: Christian Brauner \ - Date: 2026-11-02 14:28:42 - List: linux-fbdev - Page: https://ratatoskr.run/linux-fbdev/2026/11/17844980 - Thread: https://ratatoskr.run/linux-fsdevel/2026/11/17844895/t.md ```text h_get_ppp() returns the status of the H_GET_PPP hcall as an unsigned int. update_ppp() hands it on as its ssize_t result, and lparcfg_write() turns the hcall statuses it knows into an errno and returns anything else as it is. So when H_GET_PPP fails, say with H_PARAMETER (-4), the status comes back as 4294967292 and a write to /proc/powerpc/lparcfg reports that many bytes written. Return the status as the long it is. lparcfg_write() then treats it like the status of H_SET_PPP and an unknown one is at least negative. Before commit 11529396ea31 ("powerpc/pseries: Split processor entitlement retrieval and gathering to helper routines") a failed H_GET_PPP turned into -EIO. Reported-by: Chris Mason Fixes: 11529396ea31 ("powerpc/pseries: Split processor entitlement retrieval and gathering to helper routines") Cc: Signed-off-by: Christian Brauner (Amutable) --- arch/powerpc/platforms/pseries/lparcfg.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/powerpc/platforms/pseries/lparcfg.c b/arch/powerpc/platforms/pseries/lparcfg.c index 3280b7968cab..aa9b5b141434 100644 --- a/arch/powerpc/platforms/pseries/lparcfg.c +++ b/arch/powerpc/platforms/pseries/lparcfg.c @@ -113,7 +113,7 @@ struct hvcall_ppp_data { * XXXXXX - Entitled procs capacity % available to the * partitions pool. */ -static unsigned int h_get_ppp(struct hvcall_ppp_data *ppp_data) +static long h_get_ppp(struct hvcall_ppp_data *ppp_data) { unsigned long retbuf[PLPAR_HCALL9_BUFSIZE] = {0}; long rc; -- 2.53.0 ```