On Tuesday, 21 August 2007 21:30, Rafael J. Wysocki wrote: > On Tuesday, 21 August 2007 16:37, Rafael J. Wysocki wrote: > > On Tuesday, 21 August 2007 09:57, Pavel Machek wrote: [--snip--] > > > > > > > > - movl $24, %eax > > > > - movl %eax, %ds > > > > - > > > > - /* We don't restore %rax, it must be 0 anyway */ > > > > + /* restore GPRs (we don't restore %rax, it must be 0 anyway) */ > > > > movq $saved_context, %rax > > > > movq pt_regs_rsp(%rax), %rsp > > > > movq pt_regs_rbp(%rax), %rbp > > > > > > Hmm, in the old code, we knew we don't have to restore %ds, because it > > > is constant for one kernel. Now, we rely on %ds being constant accross > > > kernels. Not nice, and should be at least documented. > > > > Well, in fact we rely on it all the time (eg. the > > "movq mmu_cr4_features(%rip), %rax" above wouldn't work if that's not true), > > but I can keep the old code here just fine. ;-) > > Well, I was wrong. The new code doesn't work without this change and I'm not > exactly sure why. Actually, it works if the boot kernel is the same as the image kernel and it doesn't work if the boot kernel is different. I think what happens is that by loading %ds with a new selector we cause the descriptor to be loaded from the descriptor table pointed to by the boot kernel's GDT and this need not be in the same area as the image kernel's global descriptor table. If that is the case, an exception will trigger, so we should not load %ds until the GDTR value is restored. Moreover, according to the AMD tech docs, in 64-bit mode the value of %ds is ignored, so we don't need to bother loading it anyway. Greetings, Rafael _______________________________________________ linux-pm mailing list linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/linux-pm
