[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH RFC V2 01/45] xen/sched: add inline wrappers for calling per-scheduler functions



>>> On 06.05.19 at 08:56, <jgross@xxxxxxxx> wrote:
> @@ -207,6 +250,141 @@ static inline void sched_free_domdata(const struct 
> scheduler *s,
>          ASSERT(!data);
>  }
>  
> +static inline void *sched_alloc_pdata(const struct scheduler *s, int cpu)
> +{
> +    if ( s->alloc_pdata )
> +        return s->alloc_pdata(s, cpu);
> +    else
> +        return NULL;
> +}

In cases like this one I'd like to ask that either ?: be used, or the pointless
"else" be dropped.

Jan



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.