On Wed, 10 Mar 2010, Juan Quintela wrote:
> malc <av1...@comtv.ru> wrote:
> > On Wed, 10 Mar 2010, Juan Quintela wrote:
>
> >> - for(ioh = first_io_handler; ioh != NULL; ioh = ioh->next) {
> >> + QTAILQ_FOREACH(ioh, &io_handlers, next) {
> >> if (ioh->fd == fd)
> >> goto found;
> >> }
> >> ioh = qemu_mallocz(sizeof(IOHandlerRecord));
> >> - ioh->next = first_io_handler;
> >> - first_io_handler = ioh;
> >> + QTAILQ_INSERT_TAIL(&io_handlers, ioh, next);
> >
> > The old code inserted at the head, didn't it?
>
> Sorry, you are right, it shouldn't matter too much, but it is a change.
If it did, why queue instead of list?
>
> Later, Juan.
>
--
mailto:av1...@comtv.ru
