On Tue, 16 Feb 2016 18:59:05 -0200
Eduardo Habkost <[email protected]> wrote:

> There's no need to use g_malloc0() to allocate the channel_subsys
> struct, just use a static variable.

The original intention was to model that dynamically somehow, but I
don't see that happening anytime soon, so we can just keep it simple.

> 
> Cc: Cornelia Huck <[email protected]>
> Cc: Christian Borntraeger <[email protected]>
> Cc: Richard Henderson <[email protected]>
> Cc: Alexander Graf <[email protected]>
> Signed-off-by: Eduardo Habkost <[email protected]>
> ---
>  hw/s390x/css.c | 177 
> ++++++++++++++++++++++++++++-----------------------------
>  1 file changed, 88 insertions(+), 89 deletions(-)

Seems fine.

Should this go through the s390x tree, or will it go through another
tree together with the other patches? In any case, have a

Reviewed-by: Cornelia Huck <[email protected]>


Reply via email to