qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [Qemu-devel] AIO ide patch
@ 2004-05-25 16:32 Vladimir N. Oleynik
  2004-05-26 11:03 ` [Qemu-devel] " Vladimir N. Oleynik
                   ` (2 more replies)
  0 siblings, 3 replies; 15+ messages in thread
From: Vladimir N. Oleynik @ 2004-05-25 16:32 UTC (permalink / raw)
  To: qemu-devel, fabrice

[-- 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 --]

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Qemu-devel] Re: AIO ide patch
  2004-05-25 16:32 [Qemu-devel] AIO ide patch Vladimir N. Oleynik
@ 2004-05-26 11:03 ` Vladimir N. Oleynik
  2004-05-26 14:39   ` Hetz Ben Hamo
  2004-05-26 20:03   ` Fabrice Bellard
  2004-07-19  7:59 ` [Qemu-devel] noSDL "GUI" for Qemu Vladimir N. Oleynik
  2004-09-16 11:57 ` [Qemu-devel] No usermode hack Vladimir N. Oleynik
  2 siblings, 2 replies; 15+ messages in thread
From: Vladimir N. Oleynik @ 2004-05-26 11:03 UTC (permalink / raw)
  To: Vladimir N. Oleynik; +Cc: qemu-devel

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [Qemu-devel] Re: AIO ide patch
  2004-05-26 14:39   ` Hetz Ben Hamo
@ 2004-05-26 14:09     ` Vladimir N. Oleynik
  2004-05-30 14:30       ` Hetz Ben Hamo
  0 siblings, 1 reply; 15+ messages in thread
From: Vladimir N. Oleynik @ 2004-05-26 14:09 UTC (permalink / raw)
  To: Hetz Ben Hamo; +Cc: qemu-devel

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [Qemu-devel] Re: AIO ide patch
  2004-05-26 11:03 ` [Qemu-devel] " Vladimir N. Oleynik
@ 2004-05-26 14:39   ` Hetz Ben Hamo
  2004-05-26 14:09     ` Vladimir N. Oleynik
  2004-05-26 20:03   ` Fabrice Bellard
  1 sibling, 1 reply; 15+ messages in thread
From: Hetz Ben Hamo @ 2004-05-26 14:39 UTC (permalink / raw)
  To: qemu-devel; +Cc: Vladimir N. Oleynik

> 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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Qemu-devel] Re: AIO ide patch
  2004-05-26 11:03 ` [Qemu-devel] " Vladimir N. Oleynik
  2004-05-26 14:39   ` Hetz Ben Hamo
@ 2004-05-26 20:03   ` Fabrice Bellard
  1 sibling, 0 replies; 15+ messages in thread
From: Fabrice Bellard @ 2004-05-26 20:03 UTC (permalink / raw)
  To: Vladimir N. Oleynik; +Cc: qemu-devel

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.

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Qemu-devel] Re: AIO ide patch
@ 2004-05-28 16:10 Vladimir N. Oleynik
  0 siblings, 0 replies; 15+ messages in thread
From: Vladimir N. Oleynik @ 2004-05-28 16:10 UTC (permalink / raw)
  To: fabrice, qemu-devel

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [Qemu-devel] Re: AIO ide patch
  2004-05-26 14:09     ` Vladimir N. Oleynik
@ 2004-05-30 14:30       ` Hetz Ben Hamo
  2004-05-31  6:39         ` Vladimir N. Oleynik
  0 siblings, 1 reply; 15+ messages in thread
From: Hetz Ben Hamo @ 2004-05-30 14:30 UTC (permalink / raw)
  To: Vladimir N. Oleynik; +Cc: qemu-devel

> 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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [Qemu-devel] Re: AIO ide patch
  2004-05-30 14:30       ` Hetz Ben Hamo
@ 2004-05-31  6:39         ` Vladimir N. Oleynik
  0 siblings, 0 replies; 15+ messages in thread
From: Vladimir N. Oleynik @ 2004-05-31  6:39 UTC (permalink / raw)
  To: hetz; +Cc: qemu-devel

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Qemu-devel] noSDL "GUI" for Qemu
  2004-05-25 16:32 [Qemu-devel] AIO ide patch Vladimir N. Oleynik
  2004-05-26 11:03 ` [Qemu-devel] " Vladimir N. Oleynik
@ 2004-07-19  7:59 ` Vladimir N. Oleynik
  2004-07-20  0:16   ` Jim C. Brown
                     ` (2 more replies)
  2004-09-16 11:57 ` [Qemu-devel] No usermode hack Vladimir N. Oleynik
  2 siblings, 3 replies; 15+ messages in thread
From: Vladimir N. Oleynik @ 2004-07-19  7:59 UTC (permalink / raw)
  Cc: qemu-devel

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [Qemu-devel] noSDL "GUI" for Qemu
  2004-07-19  7:59 ` [Qemu-devel] noSDL "GUI" for Qemu Vladimir N. Oleynik
@ 2004-07-20  0:16   ` Jim C. Brown
  2004-07-20 11:52   ` Richard Zidlicky
  2004-07-20 12:32   ` [Qemu-devel] patches in dad-answers.com Vladimir N. Oleynik
  2 siblings, 0 replies; 15+ messages in thread
From: Jim C. Brown @ 2004-07-20  0:16 UTC (permalink / raw)
  To: qemu-devel

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.

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [Qemu-devel] noSDL "GUI" for Qemu
  2004-07-19  7:59 ` [Qemu-devel] noSDL "GUI" for Qemu Vladimir N. Oleynik
  2004-07-20  0:16   ` Jim C. Brown
@ 2004-07-20 11:52   ` Richard Zidlicky
  2004-07-24  3:21     ` Darrin Ritter
  2004-07-20 12:32   ` [Qemu-devel] patches in dad-answers.com Vladimir N. Oleynik
  2 siblings, 1 reply; 15+ messages in thread
From: Richard Zidlicky @ 2004-07-20 11:52 UTC (permalink / raw)
  To: qemu-devel

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [Qemu-devel] patches in dad-answers.com
  2004-07-19  7:59 ` [Qemu-devel] noSDL "GUI" for Qemu Vladimir N. Oleynik
  2004-07-20  0:16   ` Jim C. Brown
  2004-07-20 11:52   ` Richard Zidlicky
@ 2004-07-20 12:32   ` Vladimir N. Oleynik
  2004-07-20 14:16     ` Brad Watson
  2 siblings, 1 reply; 15+ messages in thread
From: Vladimir N. Oleynik @ 2004-07-20 12:32 UTC (permalink / raw)
  To: Hetz Ben Hamo, qemu-devel

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [Qemu-devel] patches in dad-answers.com
  2004-07-20 12:32   ` [Qemu-devel] patches in dad-answers.com Vladimir N. Oleynik
@ 2004-07-20 14:16     ` Brad Watson
  0 siblings, 0 replies; 15+ messages in thread
From: Brad Watson @ 2004-07-20 14:16 UTC (permalink / raw)
  To: qemu-devel

--- "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 

^ permalink raw reply	[flat|nested] 15+ messages in thread

* Re: [Qemu-devel] noSDL "GUI" for Qemu
  2004-07-20 11:52   ` Richard Zidlicky
@ 2004-07-24  3:21     ` Darrin Ritter
  0 siblings, 0 replies; 15+ messages in thread
From: Darrin Ritter @ 2004-07-24  3:21 UTC (permalink / raw)
  To: qemu-devel

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
>
>  
>

^ permalink raw reply	[flat|nested] 15+ messages in thread

* [Qemu-devel] No usermode hack
  2004-05-25 16:32 [Qemu-devel] AIO ide patch Vladimir N. Oleynik
  2004-05-26 11:03 ` [Qemu-devel] " Vladimir N. Oleynik
  2004-07-19  7:59 ` [Qemu-devel] noSDL "GUI" for Qemu Vladimir N. Oleynik
@ 2004-09-16 11:57 ` Vladimir N. Oleynik
  2 siblings, 0 replies; 15+ messages in thread
From: Vladimir N. Oleynik @ 2004-09-16 11:57 UTC (permalink / raw)
  Cc: qemu-devel

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

^ permalink raw reply	[flat|nested] 15+ messages in thread

end of thread, other threads:[~2004-09-16 12:03 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-05-25 16:32 [Qemu-devel] AIO ide patch Vladimir N. Oleynik
2004-05-26 11:03 ` [Qemu-devel] " Vladimir N. Oleynik
2004-05-26 14:39   ` Hetz Ben Hamo
2004-05-26 14:09     ` Vladimir N. Oleynik
2004-05-30 14:30       ` Hetz Ben Hamo
2004-05-31  6:39         ` Vladimir N. Oleynik
2004-05-26 20:03   ` Fabrice Bellard
2004-07-19  7:59 ` [Qemu-devel] noSDL "GUI" for Qemu Vladimir N. Oleynik
2004-07-20  0:16   ` Jim C. Brown
2004-07-20 11:52   ` Richard Zidlicky
2004-07-24  3:21     ` Darrin Ritter
2004-07-20 12:32   ` [Qemu-devel] patches in dad-answers.com Vladimir N. Oleynik
2004-07-20 14:16     ` Brad Watson
2004-09-16 11:57 ` [Qemu-devel] No usermode hack Vladimir N. Oleynik
  -- strict thread matches above, loose matches on Subject: below --
2004-05-28 16:10 [Qemu-devel] Re: AIO ide patch Vladimir N. Oleynik

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).