From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vitaly Kuznetsov Subject: [PATCH v11 05/11] xen: grant_table: implement grant_table_warn_active_grants() Date: Fri, 4 Sep 2015 15:39:45 +0200 Message-ID: <1441373991-18332-6-git-send-email-vkuznets@redhat.com> References: <1441373991-18332-1-git-send-email-vkuznets@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail6.bemta14.messagelabs.com ([193.109.254.103]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1ZXrE0-0003bc-SK for xen-devel@lists.xenproject.org; Fri, 04 Sep 2015 13:40:16 +0000 In-Reply-To: <1441373991-18332-1-git-send-email-vkuznets@redhat.com> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: xen-devel@lists.xenproject.org Cc: Andrew Jones , Julien Grall , Keir Fraser , Ian Campbell , Stefano Stabellini , Andrew Cooper , Ian Jackson , Olaf Hering , Tim Deegan , David Vrabel , Jan Beulich , Wei Liu , Daniel De Graaf List-Id: xen-devel@lists.xenproject.org Log first 10 active grants for a domain. This function is going to be used for soft reset, active grants on this path usually mean misbehaving backends refusing to release their mappings on shutdown. We need that in addition to the already existent 'g' keyhandler as such misbehaving backends can cause a domain to crash right after the soft reset operation and 'g' option won't be available in this case. Signed-off-by: Vitaly Kuznetsov Reviewed-by: Konrad Rzeszutek Wilk Acked-by: Jan Beulich --- Changes since v10: - None. --- xen/common/grant_table.c | 35 +++++++++++++++++++++++++++++++++++ xen/include/xen/grant_table.h | 5 +++++ 2 files changed, 40 insertions(+) diff --git a/xen/common/grant_table.c b/xen/common/grant_table.c index 2b449d5..23185e7 100644 --- a/xen/common/grant_table.c +++ b/xen/common/grant_table.c @@ -3352,6 +3352,41 @@ gnttab_release_mappings( } } +void grant_table_warn_active_grants(struct domain *d) +{ + struct grant_table *gt = d->grant_table; + struct active_grant_entry *act; + grant_ref_t ref; + unsigned int nr_active = 0; + +#define WARN_GRANT_MAX 10 + + read_lock(>->lock); + + for ( ref = 0; ref != nr_grant_entries(gt); ref++ ) + { + act = active_entry_acquire(gt, ref); + if ( !act->pin ) + { + active_entry_release(act); + continue; + } + + nr_active++; + if ( nr_active <= WARN_GRANT_MAX ) + printk(XENLOG_G_DEBUG "Dom%d has an active grant: GFN: %lx (MFN: %lx)\n", + d->domain_id, act->gfn, act->frame); + active_entry_release(act); + } + + if ( nr_active > WARN_GRANT_MAX ) + printk(XENLOG_G_DEBUG "Dom%d has too many (%d) active grants to report\n", + d->domain_id, nr_active); + + read_unlock(>->lock); + +#undef WARN_GRANT_MAX +} void grant_table_destroy( diff --git a/xen/include/xen/grant_table.h b/xen/include/xen/grant_table.h index 5263fd6..1c29cee 100644 --- a/xen/include/xen/grant_table.h +++ b/xen/include/xen/grant_table.h @@ -89,6 +89,11 @@ void grant_table_destroy( struct domain *d); void grant_table_init_vcpu(struct vcpu *v); +/* + * Check if domain has active grants and log first 10 of them. + */ +void grant_table_warn_active_grants(struct domain *d); + /* Domain death release of granted mappings of other domains' memory. */ void gnttab_release_mappings( -- 2.4.3