Re: [PATCH] dom0less: Fix LLC coloring when mixing colored and non-colored domains

[Stefano Stabellini <sstabellini@xxxxxxxxxx>]

On Wed, 16 Jul 2025, dmkhn@xxxxxxxxx wrote:
> On Wed, Jul 16, 2025 at 05:35:03PM +0200, Alejandro Vallejo wrote:
> > llc_colors_str being outside the main loop makes domains without
> > explicit coloring inherit the colors of the last colored domain parsed.
> > 
> > Move it inside so that's no longer the case.
> > 
> > Fixes: 5948dc16759c("xen/arm: add support for cache coloring...")
> > Signed-off-by: Alejandro Vallejo <alejandro.garciavallejo@xxxxxxx>
> 
> Reviewed-by: Denis Mukhin <dmukhin@xxxxxxxx> 

Reviewed-by: Stefano Stabellini <sstabellini@xxxxxxxxxx>


> > ---
> > pipeline: 
> > https://gitlab.com/xen-project/people/agvallejo/xen/-/pipelines/1930276681
> > ---
> >  xen/common/device-tree/dom0less-build.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/xen/common/device-tree/dom0less-build.c 
> > b/xen/common/device-tree/dom0less-build.c
> > index 221b875a2f..87828d4d9d 100644
> > --- a/xen/common/device-tree/dom0less-build.c
> > +++ b/xen/common/device-tree/dom0less-build.c
> > @@ -829,11 +829,11 @@ void __init create_domUs(void)
> >      bool iommu = false;
> >      const struct dt_device_node *cpupool_node,
> >                                  *chosen = dt_find_node_by_path("/chosen");
> > -    const char *llc_colors_str = NULL;
> > 
> >      BUG_ON(chosen == NULL);
> >      dt_for_each_child_node(chosen, node)
> >      {
> > +        const char *llc_colors_str = NULL;
> >          struct domain *d;
> >          struct xen_domctl_createdomain d_cfg = {0};
> >          unsigned int flags = 0U;
> > 
> > base-commit: dff462313fe0bfa273c54ffc9bff71f054c365af
> > --
> > 2.43.0
> > 
> > 
>