All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Cédric Le Goater" <clg@kaod.org>
To: David Gibson <david@gibson.dropbear.id.au>
Cc: "Philippe Mathieu-Daudé" <philmd@redhat.com>,
	gkurz@kaod.org, qemu-ppc@nongnu.org, qemu-devel@nongnu.org
Subject: Re: [PATCH 4/4] xics: Merge TYPE_ICS_BASE and TYPE_ICS_SIMPLE classes
Date: Wed, 25 Sep 2019 08:04:30 +0200	[thread overview]
Message-ID: <13b43747-1082-c803-a998-5b3d49418dc3@kaod.org> (raw)
In-Reply-To: <20190925014653.GK17405@umbus>

On 25/09/2019 03:46, David Gibson wrote:
> On Tue, Sep 24, 2019 at 04:06:02PM +0200, Cédric Le Goater wrote:
>> On 24/09/2019 13:41, David Gibson wrote:
>>> On Tue, Sep 24, 2019 at 07:31:44AM +0200, Cédric Le Goater wrote:
>>>> On 24/09/2019 06:59, David Gibson wrote:
>>>>> TYPE_ICS_SIMPLE is the only subtype of TYPE_ICS_BASE that's ever
>>>>> instantiated, and the only one we're ever likely to want.  The
>>>>> existence of different classes is just a hang over from when we
>>>>> (misguidedly) had separate subtypes for the KVM and non-KVM version of
>>>>> the device.
>>>>>
>>>>> So, collapse the two classes together into just TYPE_ICS.
>>>>
>>>>
>>>> Well, I have been maintaining another subclass for the PHB3 MSI 
>>>> but it has never been merged and it will require some rework.
>>>
>>> Well, if you did do this again, is there an actual need for it to be a
>>> subclass of ICS_BASE, and not ICS_SIMPLE?  AFAICT the merged ICS class
>>> should be fine for pnv as well.
>>
>> the reject resend handlers might be an issue. Anyhow, let's merge this 
>> cleanup. PHB3 has been out of tree for too long.
> 
> Hrm, are you sure.  I remember thinking the other day "whatever
> happened to that PHB3 patchset?".  Is it actually broken,

It is not broken. 

PowerNV machines can boot rather complex PCI layouts on P8 (XICS) 
and P9 (XIVE). See the complex configuration examples here :

  https://github.com/legoater/qemu/wiki/PowerNV

> or has it just been a long time since it was posted, and therefore 
> been forgotten by me.

It hasn't been posted in a long time (+1 year). Here are the latest
exchanges we had in November: 

  http://patchwork.ozlabs.org/patch/951227/

Nothing really worrying : 

  - some XICS infrastructure are needed (currently being removed)
  - check if we need a separate source model for the MSI/LSI 
  - rework the PBCQ modeling to some extent.
  - misc cleanups.

I have been quite busy this last year. I would appreciate if someone 
could take ownership of the PHB part. 

C.


  reply	other threads:[~2019-09-25  6:06 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-24  4:59 [PATCH 0/4] xics: Eliminate unnecessary class David Gibson
2019-09-24  4:59 ` [PATCH 1/4] xics: Eliminate 'reject', 'resend' and 'eoi' class hooks David Gibson
2019-09-24  5:23   ` Cédric Le Goater
2019-09-24  7:28   ` Greg Kurz
2019-09-24  4:59 ` [PATCH 2/4] xics: Merge reset and realize hooks David Gibson
2019-09-24  5:26   ` Cédric Le Goater
2019-09-24  7:36   ` Greg Kurz
2019-09-24  9:44   ` Philippe Mathieu-Daudé
2019-09-24 11:40     ` David Gibson
2019-09-24  4:59 ` [PATCH 3/4] xics: Rename misleading ics_simple_*() functions David Gibson
2019-09-24  5:26   ` Cédric Le Goater
2019-09-24  7:38   ` Greg Kurz
2019-09-24  4:59 ` [PATCH 4/4] xics: Merge TYPE_ICS_BASE and TYPE_ICS_SIMPLE classes David Gibson
2019-09-24  5:31   ` Cédric Le Goater
2019-09-24 11:41     ` David Gibson
2019-09-24 14:06       ` Cédric Le Goater
2019-09-25  1:46         ` David Gibson
2019-09-25  6:04           ` Cédric Le Goater [this message]
2019-10-03 17:53             ` Cédric Le Goater
2019-09-24  7:40   ` Greg Kurz
2019-09-24  9:46   ` Philippe Mathieu-Daudé
2019-09-24  5:22 ` [PATCH 0/4] xics: Eliminate unnecessary class Cédric Le Goater
2019-09-24  7:52   ` Greg Kurz
2019-09-24  9:55     ` Cédric Le Goater
2019-09-24 10:04       ` Philippe Mathieu-Daudé
2019-09-24 11:00         ` Cédric Le Goater
2019-09-26  1:28           ` David Gibson
2019-09-24  9:47 ` Philippe Mathieu-Daudé
2019-09-24 10:06   ` Greg Kurz
2019-09-24 10:22     ` Philippe Mathieu-Daudé

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=13b43747-1082-c803-a998-5b3d49418dc3@kaod.org \
    --to=clg@kaod.org \
    --cc=david@gibson.dropbear.id.au \
    --cc=gkurz@kaod.org \
    --cc=philmd@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=qemu-ppc@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.