Project

General

Profile

Actions

Bug #1267

closed

Off by one bug in pci_intrcfg

Added by sepherosa almost 16 years ago. Updated almost 16 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
-
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Hi all,

I want following patch in 2.2 release (i.e. I need the permission to
push it to master):
http://leaf.dragonflybsd.org/~sephe/0001-pci-intr-config-Fix-an-off-by-one-bug.patch

Please review it.

Best Regards,
sephe

Actions #1

Updated by corecode almost 16 years ago

Sure, I don't think you have to ask for permission. If you think the fix
needs to go in, it needs to go in :)

Is that a source of interrupt routing problems? Do you know when this bug
would happen and which effects it would have?

cheers
simon

Actions #2

Updated by sepherosa almost 16 years ago

On Wed, Feb 4, 2009 at 7:43 PM, Simon 'corecode' Schubert
<> wrote:

Sepherosa Ziehau wrote:

Hi all,

I want following patch in 2.2 release (i.e. I need the permission to
push it to master):

http://leaf.dragonflybsd.org/~sephe/0001-pci-intr-config-Fix-an-off-by-one-bug.patch

Please review it.

Sure, I don't think you have to ask for permission. If you think the fix
needs to go in, it needs to go in :)

Is that a source of interrupt routing problems? Do you know when this bug
would happen and which effects it would have?

Can't say it will be root cause, however, it could cause potential problems.

Best Regards,
sephe

Actions #3

Updated by dillon almost 16 years ago

I don't even understand what that loop is doing :-) matchpin is
supposed to start at 1?

-Matt
Actions #4

Updated by sepherosa almost 16 years ago

On Thu, Feb 5, 2009 at 12:18 PM, Matthew Dillon
<> wrote:

I don't even understand what that loop is doing :-) matchpin is

It tries to find the device using the same pin and has been "correctly" routed.

Yep, it starts from 1.

Best Regards,
sephe

Actions #5

Updated by sepherosa almost 16 years ago

OK to "push"?

Here is another trivial fix:
http://leaf.dragonflybsd.org/~sephe/0001-pci-intr-config-j-is-the-pin-index-we-want-to-ski.patch

Please review it. I want to push the above patch too.

Best Regards,
sephe

Actions #6

Updated by dillon almost 16 years ago

:> Yep, it starts from 1.
:>
:
:OK to "push"?
:
:Here is another trivial fix:
:http://leaf.dragonflybsd.org/~sephe/0001-pci-intr-config-j-is-the-pin-index-we-want-to-ski.patch
:
:Please review it. I want to push the above patch too.
:
:Best Regards,
:sephe
:
:--
:Live Free or Die

Yah, push'm both.
-Matt
Matthew Dillon
&lt;&gt;
Actions #7

Updated by corecode almost 16 years ago

committed

Actions

Also available in: Atom PDF