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

Re: [Xen-devel] [PATCH 21/27 v11] xen/arm: vpl011: Add support for multiple consoles in xenconsole



>  
> @@ -1223,13 +1316,13 @@ void handle_io(void)
>               /* Re-calculate any event counter allowances & unblock
>                  domains with new allowance */
>               for (d = dom_head; d; d = d->next) {
> -                     struct console *con = &d->console;
>  
> -                     console_evtchn_unmask(con, (void *)now);
> +                     console_iter_void_arg2(d, console_evtchn_unmask, (void 
> *)now);

This (together with patch 15's "long long now = (long long)data" cast)
generates fatal warning when building for 32-bit.

The warning can probably be eliminated by casting to uintptr_t (or some
such) but my question is whether 'now' should be a 'long long' --- isn't
time_t a 4-byte type on 32-bit? And if it's not then casting will lose
the upper word.

-boris


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel

 


Rackspace

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