Message-ID: <0086fb07-c5d3-9600-4b62-d7497b192461@birger-koblitz.de>
Date:   Tue, 28 Dec 2021 09:09:24 +0100
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101
 Thunderbird/91.4.0
Subject: Re: [RFC PATCH v2 0/5] Rework realtek-rtl IRQ driver
Content-Language: en-US
To:     Sander Vanheule <sander@svanheule.net>, tglx@linutronix.de,
        maz@kernel.org, robh+dt@kernel.org, devicetree@vger.kernel.org
Cc:     linux-kernel@vger.kernel.org, bert@biot.com, john@phrozen.org
References: <cover.1640548009.git.sander@svanheule.net>
 <90a3ce57-875f-d34a-0714-f815acae12d4@birger-koblitz.de>
 <bvv98e.r4rs9b.2rw3hx-qmf@polaris.svanheule.net>
From:   Birger Koblitz <mail@birger-koblitz.de>
In-Reply-To: <bvv98e.r4rs9b.2rw3hx-qmf@polaris.svanheule.net>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk

Hi Sander,
> I haven't tested this with VSMP, because it is out of scope for this series. For the binding, I expect that would only require N register ranges instead of one; one per CPU. I think the driver should then be able to perform the IRQ balancing based on that information alone, given that the parent IRQs are available at each CPU.

whether this is out of the scope of this series is not the point. In my experience you only see issues with locking and race conditions with the IRQ driver if you test with VSMP enabled,
because only with VSMP you can be in the IRQ code multiple times at the same time. Since you want to change routing logic and hierarchies I would believe it to be a very good idea
to test that. The present code passes that test.

Cheers,
   Birger