[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2 4/4] x86/dpci: remove the dpci EOI timer

  • To: <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Roger Pau Monné <roger.pau@xxxxxxxxxx>
  • Date: Fri, 15 Jan 2021 15:56:00 +0100
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=iJmeT+Ap4ZSgbXteOfx49/tvSjzdiM8VxppfM9rWvck=; b=LiPI0DMYko7tVBUHgjyyp+Uc6IHbUJiTwdzpKSDXM4nb/11nLTAben+LB9NhP5B5PYpBBZ5KW3UY9SedZfoQHZfT6SEFO3+56eMInwxzoTn2pHiX8XSQ3CJLYfNyRk1K3Dgja55Dfa38jQkq3UPUETJUVJ32W6EYNiPnowQk/ms0gadQjsPNyVjgytCDYyV3fRJojhLOtjb5rDjS6gSEDY4AhoxtuwJJBnME+S8Zcpk8gZQhjUi0butw81OWRJ+cN+3O7/J12cDgiUr0i3PLW1Shw8R5gf6oDkwXNbJ7ALNcujGhF1N4RxOF7p/1XVqaUGS/vq+4k6derLc3tt62mg==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=PxoZx2f2mujmyAHV9upgGYCnBjo52W4NMDK37V2qySJ6E1I1sIsTzw2LoxWWeRVXvS81TbpzpDVy9526xfrhaz7JoJSY0Aw76t9onm4gms6fv9ZSxtiiMGK1UuWMqTwJgIY+w4HIkQsMlGgMNSDzk7slaQdgPmBd29CLyUV5tzQuexbDkkXb67mmDs7U8GfIxRLtOS6RW2c8EcNK4HJPl1J9XOJ771l5hWQVJtuh1gBTs26Ncd02dplgYcxK98t2yynrehww4TTsylScV35ZTGhMbgi2u3FQMdwUBZ2GcEjd6rPaX5wKm51rb54voFyMn1X1Guz3PWbVkj/Et/n4fg==
  • Authentication-results: esa4.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: Kevin Tian <kevin.tian@xxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, "Paul Durrant" <paul@xxxxxxx>, Andrew Cooper <andrew.cooper3@xxxxxxxxxx>, Wei Liu <wl@xxxxxxx>
  • Delivery-date: Fri, 15 Jan 2021 14:56:14 +0000
  • Ironport-sdr: QLq5yvN17Y2ThrRn0OEDr0pagdJG99dlcBU2PY+hOhMjxmct46v2kY512puW1MPIpjmr1feHJ8 xngdUMmAodfTMuVsqNoXQtN9l6gCpkWnmNQR01e/WrjIqjpi42iGaU8jgbIw5nzJcOgdOscd8J durur57dglqB9E8pK2UnY80vTt2w0Om3SIIwxnVZkiUUFww30VJ5E6XvkxCHWU4dubAnWmGBvo rP8+9UPmVM9MZ18zkC1g8ciPRmXnQNxSB/Uwp51BOjwAgf8h3cdEBsl0rnhFE35M/abWXK0zYB vlI=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>

On Fri, Jan 15, 2021 at 03:28:20PM +0100, Roger Pau Monne wrote:
> Current interrupt pass though code will setup a timer for each
> interrupt injected to the guest that requires an EOI from the guest.
> Such timer would perform two actions if the guest doesn't EOI the
> interrupt before a given period of time. The first one is deasserting
> the virtual line, the second is perform an EOI of the physical
> interrupt source if it requires such.
> The deasserting of the guest virtual line is wrong, since it messes
> with the interrupt status of the guest. This seems to have been done
> in order to compensate for missing deasserts when certain interrupt
> controller actions are performed. The original motivation of the
> introduction of the timer was to fix issues when a GSI was shared
> between different guests. We believe that other changes in the
> interrupt handling code (ie: proper propagation of EOI related actions
> to dpci) will have fixed such errors now.
> Performing an EOI of the physical interrupt source is redundant, since
> there's already a timer that takes care of this for all interrupts,
> not just the HVM dpci ones, see irq_guest_action_t struct eoi_timer
> field.
> Since both of the actions performed by the dpci timer are not
> required, remove it altogether.
> Signed-off-by: Roger Pau Monné <roger.pau@xxxxxxxxxx>
> ---
> Changes since v1:
>  - Add parentheses.

This is a lie because I forgot to refresh the patch before sending.
Will send a new version if required.

Thanks, Roger.



Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.