Re: [uml-devel] Re: [patch 1/3] uml-ubd-no-empty-queue

[BlaisorBlade <blaisorblade_spam@yahoo.it>]

On Tuesday 07 September 2004 11:35, Jens Axboe wrote:
> On Mon, Sep 06 2004, blaisorblade_spam@yahoo.it wrote:

> Patch is correct.
Ok, thanks. Do you see anything else that needs fixing? The code the patch 
below removes has hidden this bug, so there could be other serious bugs (and 
by serious I mean Oopses or data loss).

Known issues:
- need to port to BIOs (a bit hard because I need first to understand the 
request mangling - see cowify_req. The code idea is to redirect each sector 
independently either to the UBD backing file or to the COW file. COW stand 
for Copy On Write: it allows to have the UBD read only and a COW file 
containing just the changes).

- Uml SMP support does not compile from sometimes so spinlocking is broken in 
ubd_finish (only in some cases - when called by do_ubd_request and thread_fd 
== -1). To see this, add ubd=sync on command line and turn spinlock debugging 
on.

--- uml-linux-2.6.8.1/arch/um/drivers/ubd_kern.c~uml-ubd-any-elevator   
2004-08-29 14:40:53.731043416 +0200
+++ uml-linux-2.6.8.1-paolo/arch/um/drivers/ubd_kern.c  2004-08-29 
14:40:53.733043112 +0200
@@ -749,8 +749,6 @@ int ubd_init(void)
                return -1;
        }
                
-       elevator_init(ubd_queue, &elevator_noop);
-
        if (fake_major != MAJOR_NR) {
                char name[sizeof("ubd_nnn\0")];
 
-- 
Paolo Giarrusso, aka Blaisorblade
Linux registered user n. 292729


-------------------------------------------------------
This SF.Net email is sponsored by BEA Weblogic Workshop
FREE Java Enterprise J2EE developer tools!
Get your free copy of BEA WebLogic Workshop 8.1 today.
http://ads.osdn.com/?ad_id=5047&alloc_id=10808&op=click
_______________________________________________
User-mode-linux-devel mailing list
User-mode-linux-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/user-mode-linux-devel