From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:35954) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1boagq-0008Cl-QD for qemu-devel@nongnu.org; Mon, 26 Sep 2016 14:31:45 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1boagm-0002xS-IT for qemu-devel@nongnu.org; Mon, 26 Sep 2016 14:31:43 -0400 Received: from mail-lf0-f66.google.com ([209.85.215.66]:34168) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1boagm-0002wv-As for qemu-devel@nongnu.org; Mon, 26 Sep 2016 14:31:40 -0400 Received: by mail-lf0-f66.google.com with SMTP id b71so9440124lfg.1 for ; Mon, 26 Sep 2016 11:31:39 -0700 (PDT) Date: Mon, 26 Sep 2016 20:30:37 +0200 From: "Edgar E. Iglesias" Message-ID: <20160926183037.GD5931@toto> References: <20160822161740.4252-1-prem.mallappa@broadcom.com> <20160822161740.4252-2-prem.mallappa@broadcom.com> <3c8edc1c-fd1a-7da6-9274-ad4191eddbdc@redhat.com> <20160925145853.GB5931@toto> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [PATCH v2 1/9] log: Add new IOMMU type List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Auger Eric Cc: Prem Mallappa , Peter Maydell , QEMU Developers On Mon, Sep 26, 2016 at 08:54:52AM +0200, Auger Eric wrote: > Hi Edgar, Prem, > > On 25/09/2016 16:58, Edgar E. Iglesias wrote: > > On Tue, Sep 13, 2016 at 01:53:39AM +0530, Prem Mallappa wrote: > >> On Fri, Sep 9, 2016 at 9:06 PM, Auger Eric wrote: > >> > >>> Hi Prem, > >>> > >>> Missing commit message > >>> > >>>> Signed-off-by: Prem Mallappa > >>>> --- > >>>> include/qemu/log.h | 1 + > >>>> util/log.c | 2 ++ > >>>> 2 files changed, 3 insertions(+) > >>>> > >>>> diff --git a/include/qemu/log.h b/include/qemu/log.h > >>>> index 234fa81..3dd2131 100644 > >>>> --- a/include/qemu/log.h > >>>> +++ b/include/qemu/log.h > >>>> @@ -42,6 +42,7 @@ static inline bool qemu_log_separate(void) > >>>> #define CPU_LOG_TB_NOCHAIN (1 << 13) > >>>> #define CPU_LOG_PAGE (1 << 14) > >>>> #define LOG_TRACE (1 << 15) > >>>> +#define CPU_LOG_IOMMU (1 << 16) > >>> why is it prefixed with CPU_ ? > >>> besides all arm gic devices seem to use LOG_GUEST_ERROR. what is the > >>> rationale behind introducing a new enum? > >>> > >> > >> Will change this to LOG_GUEST_ERROR, if others on the list are okay. > > > > Hi, > > > > LOG_GUEST_ERROR is used for cases when the guest programs things in bad > > way. E.g sets up a register in an invalid manner or writes to regs that > > don't exist. > > > > In this case we're logging information for valid translation steps, I > > would prefer if we could use something else than LOG_GUEST_ERROR. > > An IOMMU logging class sounds good to me. > > Thank you for the clarification; so indeed LOG_GUEST_ERROR which was my > suggestion is not a good idea. With respect to that patch I was also > wondering whether the CPU_ prefix was relevant. Right, the CPU_ prefix should probably be dropped if we use a new IOMMU log mask. Cheers, Edgar