linux-kernel - Re: [PATCH 4/4] x86_64 ioapic: Improve the heuristics for when check

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <m14pr1i76p.fsf@ebiederm.dsl.xmission.com>
Date:	Mon, 08 Jan 2007 13:39:58 -0700
From:	ebiederm@...ssion.com (Eric W. Biederman)
To:	"Lu, Yinghai" <yinghai.lu@....com>
Cc:	"Linus Torvalds" <torvalds@...l.org>,
	"Tobias Diedrich" <ranma+kernel@...edrich.de>,
	"Andrew Morton" <akpm@...l.org>, "Adrian Bunk" <bunk@...sta.de>,
	"Andi Kleen" <ak@...e.de>,
	"Linux Kernel Mailing List" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 4/4] x86_64 ioapic: Improve the heuristics for when check_timer fails.

"Lu, Yinghai" <yinghai.lu@....com> writes:

>>+static int check_timer_pin(int apic, int pin)
>>+{
>>+	int irq, idx;
>>+	/* 
>>+	 * Test the architecture default i8254 timer pin
>>+	 * of apic 0 pin 2.
>>+	 */
>
> Comments need to be updated.

Probably.   Although this is all I have this doing in my patch.

>>+
>>+
>>+	/* If the apic pin pair is in use by another irq fail */
>>+	irq = irq_from_pin(apic, pin);
             ^^^^^^^^^^^^^^^^^^
>>+	if ((irq != -1) && (irq != 0)) {
>>+		apic_printk(APIC_VERBOSE,KERN_INFO "...apic %d pin % in
> use by irq %d\n",
>>+			apic, pin, irq);
>>+		return 0; 
>>+	}
>>+
>>+	/* Add an entry in mp_irqs for irq 0 */
>>+	idx = update_irq0_entry(apic, pin);
>>+
>>+	/* Add an entry in irq_to_pin */
>>+	add_pin_to_irq(0, apic, pin);
>>+
>>+	/* Now setup the irq */
>>+	setup_IO_APIC_irq(apic, pin, idx, 0);
>>+
>>+	/* And finally check to see if the irq works */
>>+	return do_check_timer_pin(apic, pin);
>>+}
>>+
>
> Did you miss to call irq_from_pin before add_pin_to_irq? You defined
> irq_from_pin in PATCH 2/4.

No see above.

Eric

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/