[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/7] spapr, xive: Turn "nr-ends" property into "nr-servers" p
From: |
Greg Kurz |
Subject: |
Re: [PATCH 2/7] spapr, xive: Turn "nr-ends" property into "nr-servers" property |
Date: |
Fri, 4 Oct 2019 08:52:57 +0200 |
On Fri, 4 Oct 2019 07:53:13 +0200
Cédric Le Goater <address@hidden> wrote:
> >> @@ -283,11 +292,13 @@ static void spapr_xive_realize(DeviceState *dev,
> >> Error **errp)
> >> return;
> >> }
> >>
> >> - if (!xive->nr_ends) {
> >> - error_setg(errp, "Number of interrupt needs to be greater 0");
> >> + if (!xive->nr_servers) {
> >> + error_setg(errp, "Number of interrupt servers must be greater
> >> than 0");
> >> return;
> >> }
> >>
> >> + xive->nr_ends = spapr_xive_vcpu_id_to_end_idx(xive->nr_servers);
> >
> > I'd prefer not to store both nr_servers and nr_servers * 8 in the
> > structure. I think you just want xive->nr_servers, then derive it any
> > any places that current look at xive->nr_ends.
>
> Yes I agree. This is a small change.
>
I'm afraid it isn't that simple (see my other mail).
> C.
- Re: [PATCH 1/7] spapr, xics: Get number of servers with a XICSFabricClass method, (continued)
- Re: [PATCH 1/7] spapr, xics: Get number of servers with a XICSFabricClass method, Greg Kurz, 2019/10/03
- Re: [PATCH 1/7] spapr, xics: Get number of servers with a XICSFabricClass method, Cédric Le Goater, 2019/10/03
- Re: [PATCH 1/7] spapr, xics: Get number of servers with a XICSFabricClass method, Greg Kurz, 2019/10/03
- Re: [PATCH 1/7] spapr, xics: Get number of servers with a XICSFabricClass method, Cédric Le Goater, 2019/10/03
- Re: [PATCH 1/7] spapr, xics: Get number of servers with a XICSFabricClass method, Greg Kurz, 2019/10/03
[PATCH 2/7] spapr, xive: Turn "nr-ends" property into "nr-servers" property, Greg Kurz, 2019/10/03
Re: [PATCH 2/7] spapr, xive: Turn "nr-ends" property into "nr-servers" property, Greg Kurz, 2019/10/04
Re: [PATCH 2/7] spapr, xive: Turn "nr-ends" property into "nr-servers" property, David Gibson, 2019/10/05
[PATCH 3/7] spapr, xics, xive: Drop nr_servers argument in DT-related functions, Greg Kurz, 2019/10/03
[PATCH RFC 4/7] linux-headers: Update against 5.3-rc2, Greg Kurz, 2019/10/03
[PATCH 5/7] spapr/xics: Configure number of servers in KVM, Greg Kurz, 2019/10/03