Received: by 2002:a25:8b91:0:0:0:0:0 with SMTP id j17csp515184ybl; Wed, 4 Dec 2019 06:34:08 -0800 (PST) X-Google-Smtp-Source: APXvYqwLEMBpMqfxJdM+9umCZlsxrpnMGyOluZVWY1HLH38KcHWJmNzDxCn9A350F2M7s5oXkxNS X-Received: by 2002:a05:6830:20cf:: with SMTP id z15mr2672186otq.277.1575470048342; Wed, 04 Dec 2019 06:34:08 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1575470048; cv=none; d=google.com; s=arc-20160816; b=CMs80QPImh06yj86kZ1byq8+0RIpnkWAxMz8X3StQ/TCq2/5vSPovcuDyecyMaM3aC 2lgEEgVPKUEyzmgcoFVatr616rc1kCQoY1DTBq13mILngk75cSnV01Vp2qKePixUElGW Y1zJBQugn7WcUWq+WQs8DVzgLLR4lE3IXjBt15fDNrx2Sq7NbVWsg8uLxuTgSkHdH6Je PdpG85Kk0Lm71BR5k/FxF5Dx5Zf9CAxhgftjPWutaipkdn0ISXr204hKOVpM/wcFeydp gVcl2LaJKq64YmFTVkYdqe9/jSgQ5WZM02HcSoFebHAcPkUt+Cqk130raaxcrDyF2z9w 76lg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=ubTKfAixT/2nX53+w+qQ52ePsbTSwSW+RExGVkaZZ/k=; b=yX/vRkNQQNXN9KKnBNxCXcd3xQ0yNkV4Y1TEex+0IC/tiWDl3F5h65FPTEkzVSWKOs UGBU2FJxfgo3u04MAnxGAGnNqIcSOH1rANuyKg89vOJ07bVgwwuqCbswbOe/546y+pWk H4M7Gi0FgxanoO596xjUH96fIBtQw7kKvTV8HlPwjFamcRwSaXAxnZVmlICxEs2LdPS8 jowUJJfbdtPSab8Oa6E5+1ALSktliH/x5ky0n+JC/4Q2ybX7kWoINUvDCOVZSE5ke077 D6tktNFnXzsaUTA/01Qbx0c/3cy1qUuGePqRuKMNCYzVuHqTh2+aywLjG17ogQQ5Hhon ykrg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id i140si3438183oib.90.2019.12.04.06.33.55; Wed, 04 Dec 2019 06:34:08 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=intel.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728010AbfLDOdQ (ORCPT + 99 others); Wed, 4 Dec 2019 09:33:16 -0500 Received: from mga03.intel.com ([134.134.136.65]:24233 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727828AbfLDOdP (ORCPT ); Wed, 4 Dec 2019 09:33:15 -0500 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from orsmga007.jf.intel.com ([10.7.209.58]) by orsmga103.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 04 Dec 2019 06:33:15 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.69,277,1571727600"; d="scan'208";a="201418921" Received: from sjchrist-coffee.jf.intel.com (HELO linux.intel.com) ([10.54.74.41]) by orsmga007.jf.intel.com with ESMTP; 04 Dec 2019 06:33:15 -0800 Date: Wed, 4 Dec 2019 06:33:14 -0800 From: Sean Christopherson To: Paolo Bonzini Cc: Peter Xu , linux-kernel@vger.kernel.org, kvm@vger.kernel.org, "Dr . David Alan Gilbert" , Vitaly Kuznetsov Subject: Re: [PATCH RFC 04/15] KVM: Implement ring-based dirty memory tracking Message-ID: <20191204143314.GA6323@linux.intel.com> References: <20191129213505.18472-1-peterx@redhat.com> <20191129213505.18472-5-peterx@redhat.com> <20191203191328.GD19877@linux.intel.com> <24cf519e-5efa-85a7-9bc0-9be15957eb0a@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <24cf519e-5efa-85a7-9bc0-9be15957eb0a@redhat.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Dec 04, 2019 at 11:14:19AM +0100, Paolo Bonzini wrote: > On 03/12/19 20:13, Sean Christopherson wrote: > > The setting of as_id is wrong, both with and without a vCPU. as_id should > > come from slot->as_id. > > Which doesn't exist, but is an excellent suggestion nevertheless. Huh, I explicitly looked at the code to make sure as_id existed before making this suggestion. No idea what code I actually pulled up.