> On 2011-10-10 19:21:39, Gabe Black wrote: > > I'm sorry if I haven't been paying attention, but what do these changes > > have to do with QEMU? > > Nilay Vaish wrote: > I should have been more elaborate. The io ports assigned to Ide Disk are > different from the those assigned in QEMU. Specifically QEMU assigns > IDE BAR 1 to be 3f6 instead of 3f4 as it is in gem5 right now. The size > is 1 in place of 3. > > I would expect these ports to be same for both gem5 and QEMU. > > Nathan Binkert wrote: > I would agree that these should be the same, though the question is, why > did you do this? Was there a bug? I assume that linux didn't work and this > fixed it, but your commit message wasn't exactly clear (it may also mean that > you simply tested it with that kernel version). Finally, you added a dma > controller. What was that for? Overall, the change seems fine and should > probably go in, just a bit confused as to what it is for. > > Nilay Vaish wrote: > I am trying to get a checkpoint obtained from QEMU based virtual > machine to work with gem5. Since it is not working out right now, > I am trying to figure out where the differences are. This is one > of the differences I noticed. > > Linux kernel 2.6.28.4 works with current gem5, so I am not trying > to fix any bug as such. But as you pointed out that the ports used > in QEMU and gem5 should be same and this is exactly what I am aiming > for. The DMA controller exists in QEMU, that's why I added it. > > Nathan Binkert wrote: > All sounds good. These details should go in the commit message. > > Ali Saidi wrote: > Before this goes in it needs to be tested with alpha and arm. Also, > looking at the panics you removed, it seems like they were there because some > functionality isn't imolemnented. Just removing them doesn't implement it. > Finally, to really test this you probably need to do more than just boot > Linux. You need to make sure that some writes hit the disk too. > > Nilay Vaish wrote: > I agree that both alpha and arm should also work correctly. That's > something I am skeptical about and that's why I had mentioned x86 > in my original description. I think QEMU works for alpha and arm, > so things should work out. I will try booting Linux kernel for > these architectures. > > As far as disk writes are concerned, Linux kernel does write some > logs when the kernel boots. I did not check but I am expecting that > those would have happened correctly.
I think what Ali meant was that we have to make sure this doesn't affect ARM and Alpha working properly, independent of if we can restore QEMU checkpoints to them. Generally speaking it's probably handy to be able to use QEMU checkpoints, but I don't want us to be stuck doing things a certain way because QEMU did it that way. In this case it might be fine because this is probably just standard PC stuff, but I'd like a chance to look it over carefully. In cases where the config isn't fixed by how the architecture is defined, it may make sense to have a separate configuration that's specifically set up to be compatible with QEMU. I'll try to look into this properly this weekend. - Gabe ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviews.m5sim.org/r/890/#review1594 ----------------------------------------------------------- On 2011-10-10 17:55:10, Nilay Vaish wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://reviews.m5sim.org/r/890/ > ----------------------------------------------------------- > > (Updated 2011-10-10 17:55:10) > > > Review request for Default, Ali Saidi, Gabe Black, Steve Reinhardt, and > Nathan Binkert. > > > Summary > ------- > > IDE Disk: Bring it inline with QEMU > I have made some changes to the IDE Disk's ports so that it is inline with > QEMU. I am able to boot Linux 2.6.28.4 for x86 with these changes. > > > Diffs > ----- > > src/dev/ide_ctrl.cc 30a97c4198df > src/dev/ide_disk.hh 30a97c4198df > src/dev/ide_disk.cc 30a97c4198df > src/dev/x86/SouthBridge.py 30a97c4198df > > Diff: http://reviews.m5sim.org/r/890/diff > > > Testing > ------- > > > Thanks, > > Nilay > > _______________________________________________ gem5-dev mailing list [email protected] http://m5sim.org/mailman/listinfo/gem5-dev
