the modification of the interrupt flag should be ignored

stlankes · stlankes · commit ea02893e48d3 · 2021-11-10T15:36:37.000+01:00
=&gt; using preserves_flags to announce that nothing changed
diff --git a/src/arch/x86_64/kernel/irq.rs b/src/arch/x86_64/kernel/irq.rs
@@ -24,12 +24,12 @@ const KERNEL_CODE_SELECTOR: SegmentSelector = SegmentSelector::new(1, Ring::Ring
 
 /// Enable Interrupts
 pub fn irq_enable() {
-	unsafe { asm!("sti", options(nomem, nostack)) };
+	unsafe { asm!("sti", options(preserves_flags, nomem, nostack)) };
 }
 
 /// Disable Interrupts
 pub fn irq_disable() {
-	unsafe { asm!("cli", options(nomem, nostack)) };
+	unsafe { asm!("cli", options(preserves_flags, nomem, nostack)) };
 }
 
 /// Determines, if the interrupt flags (IF) is set

Original file line number	Diff line number	Diff line change
`@@ -24,12 +24,12 @@ const KERNEL_CODE_SELECTOR: SegmentSelector = SegmentSelector::new(1, Ring::Ring`
`24`	`24`
`25`	`25`	`/// Enable Interrupts`
`26`	`26`	`pub fn irq_enable() {`
`27`		`- unsafe { asm!("sti", options(nomem, nostack)) };`
	`27`	`+ unsafe { asm!("sti", options(preserves_flags, nomem, nostack)) };`
`28`	`28`	`}`
`29`	`29`
`30`	`30`	`/// Disable Interrupts`
`31`	`31`	`pub fn irq_disable() {`
`32`		`- unsafe { asm!("cli", options(nomem, nostack)) };`
	`32`	`+ unsafe { asm!("cli", options(preserves_flags, nomem, nostack)) };`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`/// Determines, if the interrupt flags (IF) is set`