From: Paolo M. <pao...@gm...> - 2008-05-21 12:05:59
|
Hi Henry today for the first time I try your RC4 under XP. I have modified the console size X and Y. I have do it several times, so I think to do it in a correct way. This time I have change a little my patch. I have added two #define in include/linux/cooperative.h #define CONSOLE_SIZE_X 120 #define CONSOLE_SIZE_Y 60 and all (kernel cocon.c, daemons and ntdriver) use #include <linux/cooperative.h> and use CONSOLE_SIZE_X and CONSOLE_SIZE_Y. These are my changes : build/linux-2.6.22.18-source/drivers/video/console/cocon.c: c->vc_cols = CONSOLE_SIZE_X; // 80; BY PAOLO build/linux-2.6.22.18-source/drivers/video/console/cocon.c: c->vc_rows = CONSOLE_SIZE_Y; // 25; BY PAOLO build/linux-2.6.22.18-source/include/linux/cooperative.h:#define CONSOLE_SIZE_X 120 // BY PAOLO build/linux-2.6.22.18-source/include/linux/cooperative.h:#define CONSOLE_SIZE_Y 60 // BY PAOLO stable-colinux-20080512/src/colinux/kernel/monitor.c:#include <linux/cooperative.h> // BY PAOLO stable-colinux-20080512/src/colinux/kernel/monitor.c: rc = co_console_create(CONSOLE_SIZE_X, CONSOLE_SIZE_Y, 0, &cmon->console); // 80,25 BY PAOLO stable-colinux-20080512/src/colinux/os/winnt/user/console-nt/widget.cpp:#include <linux/cooperative.h> // BY PAOLO stable-colinux-20080512/src/colinux/os/winnt/user/console-nt/widget.cpp: r.bottom = fs.Y * CONSOLE_SIZE_Y; // 25 BY PAOLO stable-colinux-20080512/src/colinux/os/winnt/user/console-nt/widget.cpp: r.right = fs.X * CONSOLE_SIZE_X; // 80 BY PAOLO stable-colinux-20080512/src/colinux/os/winnt/user/console-nt/widget.cpp: size.X = CONSOLE_SIZE_X; // 80 BY PAOLO stable-colinux-20080512/src/colinux/os/winnt/user/console-nt/widget.cpp: size.Y = CONSOLE_SIZE_Y; // 25 BY PAOLO stable-colinux-20080512/src/colinux/os/winnt/user/console-nt/widget.cpp: region.Right = CONSOLE_SIZE_X-1; // 79 BY PAOLO stable-colinux-20080512/src/colinux/os/winnt/user/console-nt/widget.cpp: region.Bottom = CONSOLE_SIZE_Y-1; // 24 BY PAOLO As you can see I have inserted a comment "BY PAOLO" at every change. All is compiled OK (under vmware, slackware 11.0). After that I use the old colinux-daemon --remove-driver to remove the driver. Then I have installed the new package and install the new driver with the new colinux-daemon --install-driver I have removed and installed the ntdriver WITHOUT rebooting. I have worked for some hours, than I have tried to compile qemu under colinux. After a while I have had internal OOP, like when under a real linux system I make an error coding a module. I saw several line and at the end of them a line with <cocon> and an address. I have thought I made an error in patching the size of console, so I didn't take note about the fault. I could using other windows opened, but I could not close the console-nt window. I could not close windowXP in a clean way. Using task manager I had the colinux-daemon at 99% of the CPU. Now I have checked my changes and I think all was OK. Before trying the RC4 I used the original 0.7.2 with the endless patch. My console was 100 x 50. (not 120 x 60, like the new one). A question : *************** Is it secure remove the ntdriver and install a new ntdriver without rebooting ? Under linux it is possibile and it is secure, under XP I don't know. Now I have recompiled all (with my patch) and it is working OK. I have recompiled qemu under colinux and all is OK. I underline that I have used > make clean colinux kernel to rebuild my version, because there is a little bug in colinux buildind process. Using make clean I am sure the building is OK. I realize that it is difficult trace an error with this information. But I have decided to post the error. Regards Paolo |
From: Henry N. <Hen...@Ar...> - 2008-05-21 20:27:34
|
Hello Paolo, Paolo Minazzi wrote: > today for the first time I try your RC4 under XP. > I have modified the console size X and Y. > > I have do it several times, so I think to do it in a correct way. > > This time I have change a little my patch. > I have added two #define in include/linux/cooperative.h > #define CONSOLE_SIZE_X 120 > #define CONSOLE_SIZE_Y 60 > > and all (kernel cocon.c, daemons and ntdriver) use > #include <linux/cooperative.h> > > and use CONSOLE_SIZE_X and CONSOLE_SIZE_Y. > [...] > After that I use the old colinux-daemon --remove-driver to remove the driver. > Then I have installed the new package and install the new driver with > the new colinux-daemon --install-driver > I have removed and installed the ntdriver WITHOUT rebooting. Have you replaced the file "vmlinux"? You should! > I have worked for some hours, than I have tried to compile qemu under colinux. > After a while I have had internal OOP, like when under a real linux > system I make an error coding a module. > I saw several line and at the end of them a line with <cocon> and an address. > I have thought I made an error in patching the size of console, so I > didn't take note about the fault. > I could using other windows opened, but I could not close the console-nt window. > I could not close windowXP in a clean way. > Using task manager I had the colinux-daemon at 99% of the CPU. > > Now I have checked my changes and I think all was OK. Yes, I saw are all changes, I know. Think, you have changed correctly. > Before trying the RC4 I used the original 0.7.2 with the endless > patch. My console was 100 x 50. (not 120 x 60, like the new one). > > A question : > *************** > Is it secure remove the ntdriver and install a new ntdriver without rebooting ? > Under linux it is possibile and it is secure, under XP I don't know. Yes, it's save, if you have verifyed, that the driver was unloaded. Under Windows the driver sometimes can not unload, if any daemon is active. So, I remember, you don't get an error. After --remove-driver you should check, that the driver is true unloaded. Only if no driver is installed, the next --install-driver is working. > Now I have recompiled all (with my patch) and it is working OK. I have > recompiled qemu under colinux and all is OK. > > I underline that I have used >> make clean colinux kernel > to rebuild my version, because there is a little bug in colinux > buildind process. > Using make clean I am sure the building is OK. You have cleaned the Linux kernel? The source has no dependency to "include/linux/cooperative.h". You needs to remove all objects (*.o) and archives (*.a), which can have the console size compiled in. Your patch needs to rebuild all. All means coLinux ntkernel (linux.sys), daemons (colinux-console-nt) and the Linux kernel (vmlinux). I suggest you to use these patches: console-size-40x120-colinux-0.7.3.patch console-size-40x120-linux-2.6.22.18-co-0.7.3.patch There don't exist build time depency for console size between Linux kernel and ntkernel. Only rebuild the coLinux driver and daemons ("make clean && make colinux"), than reinstall the driver and use the new daemons to have the new console size. You not need to rebuild Linux kernel, after you have ones compiled with the runtime size set (CO_OPERATION_CONSOLE_SIZES). This patch for console sizes defines exist in 0.8.0 now, since yesterday ;-) Checkout the last source from SVN or from autobuild. I have also uploaded console with 40x120 for 0.8.0: http://www.henrynestler.com/colinux/testing/devel-0.8.0/20080520-con-40x120/ The same as 0.7.3: http://www.henrynestler.com/colinux/testing/stable-0.7.3/20080521-con-40x120/ -- Henry N. |
From: Paolo M. <pao...@gm...> - 2008-05-22 07:14:17
|
>> A question : >> *************** >> Is it secure remove the ntdriver and install a new ntdriver without rebooting ? >> Under linux it is possibile and it is secure, under XP I don't know. > > Yes, it's save, if you have verifyed, that the driver was unloaded. > Under Windows the driver sometimes can not unload, if any daemon is > active. So, I remember, you don't get an error. After --remove-driver > you should check, that the driver is true unloaded. Only if no driver is > installed, the next --install-driver is working. OK, my old driver was unloaded correctly because I could install the new driver. >> Now I have recompiled all (with my patch) and it is working OK. I have >> recompiled qemu under colinux and all is OK. >> >> I underline that I have used >>> make clean colinux kernel >> to rebuild my version, because there is a little bug in colinux >> buildind process. >> Using make clean I am sure the building is OK. > > You have cleaned the Linux kernel? No, I have not cleaned the Linux kernel. I have only rebuild it using make in the linux-xxx-build directory I thought that the linux building process automatically sees the changes and recompiles the new source. Maybe it was better to make clean and rebuild all the Linux kernel to avoid dependecies problems. . > > The source has no dependency to "include/linux/cooperative.h". You needs > to remove all objects (*.o) and archives (*.a), which can have the > console size compiled in. Ok, is this done by make clean in the linux-xxx-build ? Is it right ? > > Your patch needs to rebuild all. All means coLinux ntkernel (linux.sys), > daemons (colinux-console-nt) and the Linux kernel (vmlinux). > > I suggest you to use these patches: > console-size-40x120-colinux-0.7.3.patch > console-size-40x120-linux-2.6.22.18-co-0.7.3.patch > There don't exist build time depency for console size between Linux > kernel and ntkernel. Only rebuild the coLinux driver and daemons ("make > clean && make colinux"), than reinstall the driver and use the new > daemons to have the new console size. > You not need to rebuild Linux kernel, after you have ones compiled with > the runtime size set (CO_OPERATION_CONSOLE_SIZES). > > This patch for console sizes defines exist in 0.8.0 now, since yesterday > ;-) Checkout the last source from SVN or from autobuild. > > I have also uploaded console with 40x120 for 0.8.0: > http://www.henrynestler.com/colinux/testing/devel-0.8.0/20080520-con-40x120/ > > The same as 0.7.3: > http://www.henrynestler.com/colinux/testing/stable-0.7.3/20080521-con-40x120/ OK, Now I check my changes to understand if I have made some error (maybe some dependencies problem). In the future I will use your patch. I think that the problem is mine (not a colinux problem) because in the OOP trace I have seen the "cocon" string. I will post if I will have the same error or if the error vanishes. Then I small idea .... Don't you think that in the main Makefile a "make clean-kernel" is missing ? The "make clean" only clean colinux daemon and driver, the kernel is not touched. But maybe you prefer do it manually..... Thanks for your time ! Paolo |