Re: [Qemu-devel] [PATCH 1/2] Move some declarations around in the QEMU CharDriver code

[Anthony Liguori]

Blue Swirl wrote:
> On 10/23/08, Anthony Liguori <address@hidden> wrote:
>   
> > The goal of this series is to move the CharDriverState code out of vl.c and
> >  into its own file, qemu-char.c.  This patch moves around some declarations so
> >  the next patch can be pure code motion.
> >     
>
> Good idea. Do you plan to do this for other things, for example network code?
>   

Yes, I have patches in queue to move out the networking code, the timer 
code, and the main loop.  I have two goals here.  A lot of the vl.c code 
is target independent so I'd like to move a lot of that code to be 
compiled once for all targets.  I also want to be able to reuse the main 
loop in things like qemu-nbd so that it can be fully asychronous without 
needing to rewrite all of that code.

I'll go ahead and commit the CharDriverState movement and post the next 
movement patch in a couple days.

>   
> >  -static inline int send_all(int fd, const uint8_t *buf, int len1)
> >  +inline int send_all(int fd, const uint8_t *buf, int len1)
> >     
>
> The 'inline' looks a bit odd without 'static'.
>   

Yeah, I'll fix that.  Didn't notice that for whatever reason.

Regards,

Anthony Liguori