From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 30 Apr 2019 11:37:18 +0200 From: Halil Pasic Subject: Re: [PATCH v7 3/4] s390: ap: implement PAPQ AQIC interception in kernel In-Reply-To: References: <1556283688-556-1-git-send-email-pmorel@linux.ibm.com> <1556283688-556-4-git-send-email-pmorel@linux.ibm.com> <20190429185002.6041eecc.pasic@linux.ibm.com> <14453f04-f13f-f63c-fd8a-d9d8834182e0@linux.ibm.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit Message-Id: <20190430113718.426392f0.pasic@linux.ibm.com> Sender: kvm-owner@vger.kernel.org List-Archive: List-Post: To: Pierre Morel Cc: borntraeger@de.ibm.com, alex.williamson@redhat.com, cohuck@redhat.com, linux-kernel@vger.kernel.org, linux-s390@vger.kernel.org, kvm@vger.kernel.org, frankja@linux.ibm.com, akrowiak@linux.ibm.com, david@redhat.com, schwidefsky@de.ibm.com, heiko.carstens@de.ibm.com, freude@linux.ibm.com, mimu@linux.ibm.com List-ID: On Tue, 30 Apr 2019 10:32:52 +0200 Pierre Morel wrote: > >>> +    aqic_gisa.gisa = gisa->next_alert >> 4; > >> > >> Why gisa->next_alert? Isn't this supposed to get set to gisa origin > >> (without some bits on the left)? s/left/right/ > > > > Someone already asked this question. It must have been in some previous iteration... Can you give me a pointer? > > The answer is: look at the ap_qirq_ctrl structure, you will see that the > > gisa field is 27 bits wide. My question was not about the width, but about gisa->next_alert being used. Regards, Halil