[Qemu-devel] AIO ide patch

[Qemu-devel] AIO ide patch

[Vladimir N. Oleynik]

[-- Attachment #1: Type: text/plain, Size: 206 bytes --]

Hi Fabrice!

I have tried to make a variant with asynchronous input/output
for ide. First worked patch attached.
(this variant haven`t AIO for ATAPI, only ATA).
May be not full right, but ...



--w
vodz



[-- Attachment #2: qemu_aio.patch.gz --]
[-- Type: application/x-tar, Size: 1697 bytes --]

[Qemu-devel] Re: AIO ide patch

[Vladimir N. Oleynik]

I wrote:

> Hi Fabrice!
> 
> I have tried to make a variant with asynchronous input/output
> for ide. First worked patch attached.
> (this variant haven`t AIO for ATAPI, only ATA).
> May be not full right, but ...

Good news: with this second variant patch installation w2k is full true
and don`t make disk full!
 From second variant I moved set_irq() to thread.
But, I don`t know: this save or not, but all my test cleans.


--w
vodz

Re: [Qemu-devel] Re: AIO ide patch

[Hetz Ben Hamo]

> Good news: with this second variant patch installation w2k is full true
> and don`t make disk full!
>  From second variant I moved set_irq() to thread.
> But, I don`t know: this save or not, but all my test cleans.

And where is the patch? :)

Thanks,
Hetz

Re: [Qemu-devel] Re: AIO ide patch

[Vladimir N. Oleynik]

Hetz,

>> Good news: with this second variant patch installation w2k is full true
>> and don`t make disk full!
>>  From second variant I moved set_irq() to thread.
>> But, I don`t know: this save or not, but all my test cleans.

> And where is the patch? :)

I want see Fabrice a reaction. I don`t know of safe my idea.
I tested 3-th variant with AIO for ATAPI. My tests is works.


ftp://ftp.simtreas.ru/pub/my/qemu/qemu_aio.patch3.gz


--w
vodz

Re: [Qemu-devel] Re: AIO ide patch

[Hetz Ben Hamo]

> I want see Fabrice a reaction. I don`t know of safe my idea.
> I tested 3-th variant with AIO for ATAPI. My tests is works.
>
>
> ftp://ftp.simtreas.ru/pub/my/qemu/qemu_aio.patch3.gz

Well, safe  - it's not :(

It seems that with your patch, Windows XP upon booting complains about wrong 
HAL, and doesn't boot. Windows 98 complains about corrupt hard drive and 
doesn't boot either.

With your patch applied - do I need to re-install the guest OS?

Thanks,
Hetz

Re: [Qemu-devel] Re: AIO ide patch

[Vladimir N. Oleynik]

Hetz,

>>I want see Fabrice a reaction. I don`t know of safe my idea.
>>I tested 3-th variant with AIO for ATAPI. My tests is works.
>>
>>
>>ftp://ftp.simtreas.ru/pub/my/qemu/qemu_aio.patch3.gz

> Well, safe  - it's not :(

May be. Qemu have comment for demontsrate this ;-)

/* XXX: tb_flush is currently not thread safe */
....
     if (cpu_single_env)
         env = cpu_single_env; /* XXX: find a correct solution for multithread */

And many other...

Current cpu_single_env=global_env and cpu_loop() have save context by localy,
not to env. io_port make irq from signle_env, mail_loop() make timer irq from
global_env, network fd_read make irq from mail_loop() with single_env...
I confused this. The current design may be ready not thread safe, but have
working progress for this.

> It seems that with your patch, Windows XP upon booting complains about wrong 
> HAL, and doesn't boot. Windows 98 complains about corrupt hard drive and 
> doesn't boot either.

Heh. My WinXP installation with AIO_patch4v is ok.
4v have async call all set_irq with usage sched_yield().

--w
vodz

[Qemu-devel] Re: AIO ide patch

[Fabrice Bellard]

Vladimir N. Oleynik wrote:
> I wrote:
> 
>> Hi Fabrice!
>>
>> I have tried to make a variant with asynchronous input/output
>> for ide. First worked patch attached.
>> (this variant haven`t AIO for ATAPI, only ATA).
>> May be not full right, but ...
> 
> 
> Good news: with this second variant patch installation w2k is full true
> and don`t make disk full!
>  From second variant I moved set_irq() to thread.
> But, I don`t know: this save or not, but all my test cleans.

The disk dull problem is timing dependent. Your patches change the 
timings, but there is no proof they really fixed the bug. Of course it 
is possible that Win2K has a bug which makes the IDE driver timing 
dependent...

Fabrice.

[Qemu-devel] noSDL "GUI" for Qemu

[Vladimir N. Oleynik]

Hi all.

I have made a no SDL variant "GUI" with use Xlib only for Qemu:
ftp://ftp.simtreas.ru/pub/my/qemu/nosdl.c.gz
This is small, simple and have not problem
with sigacton/threads.
Its first version, without fullscreen support and may be
have problem with keyboard (hmm, PC/AT translation?)


--w
vodz

Re: [Qemu-devel] noSDL "GUI" for Qemu

[Jim C. Brown]

On Mon, Jul 19, 2004 at 11:59:45AM +0400, Vladimir N. Oleynik wrote:
> Hi all.
> 
> I have made a no SDL variant "GUI" with use Xlib only for Qemu:
> ftp://ftp.simtreas.ru/pub/my/qemu/nosdl.c.gz
> This is small, simple and have not problem
> with sigacton/threads.
> Its first version, without fullscreen support and may be
> have problem with keyboard (hmm, PC/AT translation?)
> 
> 
> --w
> vodz
> 
> 

I haven't tried it yet, will do so later this afternoon.

But may I be the first to say .. Thank You!

SDL doesn't have the ability to support the touchpad. It can't tell us the
"absolute" position of the pointer [the position relative to the bottom-left corner
of the window], only the relative mouse movement and the absolute mouse position
relative to the screen itself. The 2nd type can't be used because one can not
get the position of the SDL window (I asked in #SDL and was basicly told "you
shouldn't need to do that") so there's no way to calculate what you need.

Of course, even if the X11 driver is adopted, touchpad support will be some
time away as I need to figure out how to convert pixels to mm.

> _______________________________________________
> Qemu-devel mailing list
> Qemu-devel@nongnu.org
> http://lists.nongnu.org/mailman/listinfo/qemu-devel
> 

-- 
Infinite complexity begets infinite beauty.
Infinite precision begets infinite perfection.

Re: [Qemu-devel] noSDL "GUI" for Qemu

[Richard Zidlicky]

On Mon, Jul 19, 2004 at 11:59:45AM +0400, Vladimir N. Oleynik wrote:
> Hi all.
> 
> I have made a no SDL variant "GUI" with use Xlib only for Qemu:
> ftp://ftp.simtreas.ru/pub/my/qemu/nosdl.c.gz
> This is small, simple and have not problem
> with sigacton/threads.

great thanks, I will try it. SDL often drives me crazy.

Richard

Re: [Qemu-devel] noSDL "GUI" for Qemu

[Darrin Ritter]

sorry for the silly question but as i am not a programmer how di I use 
the nosdl.c.gz file?

DarR!N

Richard Zidlicky wrote:

>On Mon, Jul 19, 2004 at 11:59:45AM +0400, Vladimir N. Oleynik wrote:
>  
>
>>Hi all.
>>
>>I have made a no SDL variant "GUI" with use Xlib only for Qemu:
>>ftp://ftp.simtreas.ru/pub/my/qemu/nosdl.c.gz
>>This is small, simple and have not problem
>>with sigacton/threads.
>>    
>>
>
>great thanks, I will try it. SDL often drives me crazy.
>
>Richard
>
>
>_______________________________________________
>Qemu-devel mailing list
>Qemu-devel@nongnu.org
>http://lists.nongnu.org/mailman/listinfo/qemu-devel
>
>  
>

Re: [Qemu-devel] patches in dad-answers.com

[Vladimir N. Oleynik]

Hetz,

 > 1. NO_SDL - removes the SDL requirement and replaces it with a native Xlib
 > support (I hardly think Fabrice will ever commit this). Anyone can write a
 > small README how to add and compile this thing?

Hmm... You could ask me and in the first person, not in the third.

Its trivial:

$ mv sdl.c sdl_orig.c
$ ln -s nosdl.c sdl.c
$ patch Makefile.target

  $(QEMU_SYSTEM): $(VL_OBJS) libqemu.a
-       $(CC) $(VL_LDFLAGS) -o $@ $^ $(LIBS) $(VL_LIBS) $(SDL_LIBS)
+       $(CC) $(VL_LDFLAGS) -o $@ $^ $(LIBS) $(VL_LIBS) -L/usr/X11R6/lib -lX11


PS: If and to correct this file, it is possible to delete at the same time from 
there option programs slowing down operation: -g,-malign-functions=0, 
-mpreferred-stack-boundary=2 ;-)


--w
vodz

Re: [Qemu-devel] patches in dad-answers.com

[Brad Watson]

--- "Vladimir N. Oleynik" <dzo@simtreas.ru> wrote:
> Hetz,
> 
>  > 1. NO_SDL - removes the SDL requirement and
> replaces it with a native Xlib
>  > support (I hardly think Fabrice will ever commit
> this). Anyone can write a
>  > small README how to add and compile this thing?
> 
> Hmm... You could ask me and in the first person, not
> in the third.
> 
> Its trivial:
> 
> $ mv sdl.c sdl_orig.c
> $ ln -s nosdl.c sdl.c
> $ patch Makefile.target
> 
>   $(QEMU_SYSTEM): $(VL_OBJS) libqemu.a
> -       $(CC) $(VL_LDFLAGS) -o $@ $^ $(LIBS)
> $(VL_LIBS) $(SDL_LIBS)
> +       $(CC) $(VL_LDFLAGS) -o $@ $^ $(LIBS)
> $(VL_LIBS) -L/usr/X11R6/lib -lX11
> 
> 
> PS: If and to correct this file, it is possible to
> delete at the same time from 
> there option programs slowing down operation:
> -g,-malign-functions=0, 
> -mpreferred-stack-boundary=2 ;-)
> 
> 
> --w
> vodz
> 
> 
> _______________________________________________
> Qemu-devel mailing list
> Qemu-devel@nongnu.org
> http://lists.nongnu.org/mailman/listinfo/qemu-devel
> 


Hi All,

The following worked for me:

>$ mv sdl.c sdl_orig.c
>$ ln -s nosdl.c sdl.c

> $ patch Makefile.target
>
   $(QEMU_SYSTEM): $(VL_OBJS) libqemu.a
 -       $(CC) $(VL_LDFLAGS) -o $@ $^ $(LIBS)
 $(VL_LIBS) $(SDL_LIBS)
 +       $(CC) $(VL_LDFLAGS) -o $@ $^ $(LIBS)
 $(VL_LIBS) -L/usr/X11R6/lib -lX11 -lpthread

Regards,

Brad Watson


__________________________________________________
Do You Yahoo!?
Tired of spam?  Yahoo! Mail has the best spam protection around 
http://mail.yahoo.com 

[Qemu-devel] No usermode hack

[Vladimir N. Oleynik]

Hi.

I have tried to make a hack to remove support USER_MODE.
Result: +10% speed up.
 From "checkit" test (REMOVED_USER_MODE/Original Qemu)
CPU:        1000825/98222
FPU:        46374.5/43146.3
VGA BIOS:     25273/24590
VGA direct:  922850/901061
HD direct:  89902.8/68978.7

The method is: removed all "env" usage.
ftp://ftp.simtreas.ru/pub/my/qemu/
have nouserhack.sh for demonstrate changes
to I386-I386-softmmu qemu.

Also. I have placed "checkit" to ftp.
Its a check program show problem with test cirrus-VGA for
some old modes, original VGA emulator give more true.

(Warning, AIO_patch from this URL very outdate. Don`t use,
for demo only!)


--w
vodz

[Qemu-devel] Re: AIO ide patch

[Vladimir N. Oleynik]

Fabrice,

 >> I have tried to make a variant with asynchronous input/output
 >> for ide. First worked patch attached.
 >> (this variant haven`t AIO for ATAPI, only ATA).
 >> May be not full right, but ...

 >> Good news: with this second variant patch installation w2k is full true
 >> and don`t make disk full!
 >> From second variant I moved set_irq() to thread.
 >> But, I don`t know: this save or not, but all my test cleans.

 > The disk dull problem is timing dependent.

Yes.

 > Your patches change
 > the timings, but there is no proof they really fixed the bug.

I can`t patch W2k ;-)

 > Of course it is possible that Win2K has a bug which makes
 > the IDE driver timing dependent...

My idea is not patch timing. My idea is async IO for make speed up
(guest CPU continued after host`s disk/network/sound make IO processing).

If set_pic_irq() is tread safe, I can generate more other async IO
(floppy, DMA, sound, network fd poll)...


--w
vodz