On Mon, 27 Jun 2022, Xenia Ragiadakou wrote: > The function vm_event_wake() is referenced only in vm_event.c. > Change the linkage of the function from external to internal by adding > the storage-class specifier static to the function definition. > > This patch aims to resolve indirectly a MISRA C 2012 Rule 8.4 violation > warning. > > Signed-off-by: Xenia Ragiadakou <[email protected]>
Reviewed-by: Stefano Stabellini <[email protected]> > --- > xen/common/vm_event.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/xen/common/vm_event.c b/xen/common/vm_event.c > index 0b99a6ea72..ecf49c38a9 100644 > --- a/xen/common/vm_event.c > +++ b/xen/common/vm_event.c > @@ -173,7 +173,7 @@ static void vm_event_wake_queued(struct domain *d, struct > vm_event_domain *ved) > * call vm_event_wake() again, ensuring that any blocked vCPUs will get > * unpaused once all the queued vCPUs have made it through. > */ > -void vm_event_wake(struct domain *d, struct vm_event_domain *ved) > +static void vm_event_wake(struct domain *d, struct vm_event_domain *ved) > { > if ( !list_empty(&ved->wq.list) ) > vm_event_wake_queued(d, ved); > -- > 2.34.1 > >
