pci: configuration: Upgrade log level of PCI BAR reprogramming message

This message only occurs sporadically and so it should be included at
info!() level. Enhance the output to also include the BAR number.

Signed-off-by: Rob Bradford <robert.bradford@intel.com>
This commit is contained in:
Rob Bradford 2021-09-03 10:28:33 +01:00 committed by Bo Chen
parent 968902dfec
commit 64e217cf39

View File

@ -794,9 +794,9 @@ impl PciConfiguration {
return None; return None;
} }
debug!( info!(
"DETECT BAR REPROG: current 0x{:x}, new 0x{:x}", "Detected BAR reprogramming: (BAR {}) 0x{:x}->0x{:x}",
self.registers[reg_idx], value reg_idx, self.registers[reg_idx], value
); );
let old_base = u64::from(self.bars[bar_idx].addr & mask); let old_base = u64::from(self.bars[bar_idx].addr & mask);
let new_base = u64::from(value & mask); let new_base = u64::from(value & mask);
@ -820,9 +820,9 @@ impl PciConfiguration {
!= (self.bars[bar_idx - 1].addr & self.writable_bits[reg_idx - 1]) != (self.bars[bar_idx - 1].addr & self.writable_bits[reg_idx - 1])
|| (value & mask) != (self.bars[bar_idx].addr & mask)) || (value & mask) != (self.bars[bar_idx].addr & mask))
{ {
debug!( info!(
"DETECT BAR REPROG: current 0x{:x}, new 0x{:x}", "Detected BAR reprogramming: (BAR {}) 0x{:x}->0x{:x}",
self.registers[reg_idx], value reg_idx, self.registers[reg_idx], value
); );
let old_base = u64::from(self.bars[bar_idx].addr & mask) << 32 let old_base = u64::from(self.bars[bar_idx].addr & mask) << 32
| u64::from(self.bars[bar_idx - 1].addr & self.writable_bits[reg_idx - 1]); | u64::from(self.bars[bar_idx - 1].addr & self.writable_bits[reg_idx - 1]);
@ -850,9 +850,9 @@ impl PciConfiguration {
return None; return None;
} }
debug!( info!(
"DETECT ROM BAR REPROG: current 0x{:x}, new 0x{:x}", "Detected ROM BAR reprogramming: (BAR {}) 0x{:x}->0x{:x}",
self.registers[reg_idx], value reg_idx, self.registers[reg_idx], value
); );
let old_base = u64::from(self.rom_bar_addr & mask); let old_base = u64::from(self.rom_bar_addr & mask);
let new_base = u64::from(value & mask); let new_base = u64::from(value & mask);