Virtualization - Sep 2022 - [PATCH v2] jailhouse: Hold reference returned from of_find_xxx API

[ Adding author and reviewers of commit 63338a38db95 again ]

On 9/16/22 2:00 AM, Liang He wrote:
> In jailhouse_paravirt(), we should hold the reference returned from
> of_find_compatible_node() which has increased the refcount and then
> call of_node_put() with it when done.
> 
> Fixes: 63338a38db95 ("jailhouse: Provide detection for non-x86
systems")
> Signed-off-by: Liang He <windhl at 126.com>
> Co-developed-by: Kelin Wang <wangkelin2023 at 163.com>
> Signed-off-by: Kelin Wang <wangkelin2023 at 163.com>
Reviewed-by: Srivatsa S. Bhat (VMware) <srivatsa at csail.mit.edu>
> ---
> 
>  v2: use proper return type not the 'np' pointer
> 
>  include/linux/hypervisor.h | 6 +++++-
>  1 file changed, 5 insertions(+), 1 deletion(-)
> 
> diff --git a/include/linux/hypervisor.h b/include/linux/hypervisor.h
> index 9efbc54e35e5..f11eec57ea63 100644
> --- a/include/linux/hypervisor.h
> +++ b/include/linux/hypervisor.h
> @@ -27,7 +27,11 @@ static inline void hypervisor_pin_vcpu(int cpu)
>  
>  static inline bool jailhouse_paravirt(void)
>  {
> -	return of_find_compatible_node(NULL, NULL, "jailhouse,cell");
> +	struct device_node *np = of_find_compatible_node(NULL, NULL,
"jailhouse,cell");
> +
> +	of_node_put(np);
> +
> +	return np ? true : false;
>  }
>  
>  #endif /* !CONFIG_X86 */
>
Regards,
Srivatsa
VMware Photon OS

On Fri, Sep 16, 2022 at 10:25:31PM -0700, Srivatsa S. Bhat
wrote:
> [ Adding author and reviewers of commit 63338a38db95 again ]
> 
> On 9/16/22 2:00 AM, Liang He wrote:
> > In jailhouse_paravirt(), we should hold the reference returned from
> > of_find_compatible_node() which has increased the refcount and then
> > call of_node_put() with it when done.
> > 
> > Fixes: 63338a38db95 ("jailhouse: Provide detection for non-x86
systems")
> > Signed-off-by: Liang He <windhl at 126.com>
> > Co-developed-by: Kelin Wang <wangkelin2023 at 163.com>
> > Signed-off-by: Kelin Wang <wangkelin2023 at 163.com>
> 
> Reviewed-by: Srivatsa S. Bhat (VMware) <srivatsa at csail.mit.edu>
The message to which you are responding didn't make it to the mailing
list, so it's unlikely that anybody will pick it up.  See the archive:
https://lore.kernel.org/all/0069849b-e6c7-5c9b-4b52-5aa6e4a328e4 at
csail.mit.edu/

Maybe it was a multipart message or was HTML, which the mailing lists
reject: http://vger.kernel.org/majordomo-info.html

Bjorn