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

Re: [Xen-devel] [PATCH] Switch from select() to poll() in xenconsoled's IO loop

On Tue, 2013-01-08 at 14:01 +0000, Mats Petersson wrote:
> >> +struct pollfd *xs_pollfd;
> Why is this not "static"?
> >> +static struct pollfd  *fds;
> >> +static unsigned int current_array_size;
> >> +static unsigned int nr_fds;
> There seems to be no particular reason why these variables are here, and 
> the other ones up the top of the file. For example xce_handle is not 
> used "above" here, but it's declared at the top of the file. I think 
> keeping all variables together makes more sense than scattering them around.

I'd prefer if those which can be were local to the handle_io function
and not global at all.  I think this includes both xs_pollfd and

I think it also includes the existing xce_handle and log_hv_evtchn.


Xen-devel mailing list



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