From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-15.2 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id EB633C432BE for ; Thu, 26 Aug 2021 09:04:49 +0000 (UTC) Received: from lists.gnu.org (lists.gnu.org [209.51.188.17]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 6F4B7610A4 for ; Thu, 26 Aug 2021 09:04:49 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 6F4B7610A4 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=intel.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=nongnu.org Received: from localhost ([::1]:44780 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1mJBJY-0007ea-65 for qemu-devel@archiver.kernel.org; Thu, 26 Aug 2021 05:04:48 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:50796) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mJBIY-0006xb-9m for qemu-devel@nongnu.org; Thu, 26 Aug 2021 05:03:46 -0400 Received: from mga14.intel.com ([192.55.52.115]:20206) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1mJBIS-0005VG-NW for qemu-devel@nongnu.org; Thu, 26 Aug 2021 05:03:45 -0400 X-IronPort-AV: E=McAfee;i="6200,9189,10087"; a="217423230" X-IronPort-AV: E=Sophos;i="5.84,353,1620716400"; d="scan'208";a="217423230" Received: from fmsmga008.fm.intel.com ([10.253.24.58]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Aug 2021 02:03:32 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.84,353,1620716400"; d="scan'208";a="494992954" Received: from michael-optiplex-9020.sh.intel.com (HELO localhost) ([10.239.159.182]) by fmsmga008.fm.intel.com with ESMTP; 26 Aug 2021 02:03:31 -0700 Date: Thu, 26 Aug 2021 17:19:08 +0800 From: Yang Weijiang To: "pbonzini@redhat.com" , Eduardo Habkost , "qemu-devel@nongnu.org" , "kvm@vger.kernel.org" Subject: Re: [PATCH v4 1/2] qdev-properties: Add a new macro with bitmask check for uint64_t property Message-ID: <20210826091908.GA23724@intel.com> References: <1624156957-7223-1-git-send-email-weijiang.yang@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1624156957-7223-1-git-send-email-weijiang.yang@intel.com> User-Agent: Mutt/1.5.24 (2015-08-30) Received-SPF: pass client-ip=192.55.52.115; envelope-from=weijiang.yang@intel.com; helo=mga14.intel.com X-Spam_score_int: -68 X-Spam_score: -6.9 X-Spam_bar: ------ X-Spam_report: (-6.9 / 5.0 requ) BAYES_00=-1.9, RCVD_IN_DNSWL_HI=-5, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=ham autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "Yang, Weijiang" Errors-To: qemu-devel-bounces+qemu-devel=archiver.kernel.org@nongnu.org Sender: "Qemu-devel" On Sun, Jun 20, 2021 at 10:42:36AM +0800, Yang, Weijiang wrote: Hi, Paolo and Eduardo, Legacy Arch LBR patches have been merged in 5.12 kernel tree, these patches are corresponding change from QEMU side, without these patches,legacy Arch LBR cannot work, could you review them at your convenience? Thanks a lot! > The DEFINE_PROP_UINT64_CHECKMASK maro applies certain mask check agaist > user-supplied property value, reject the value if it violates the bitmask. > > Co-developed-by: Like Xu > Signed-off-by: Like Xu > Signed-off-by: Yang Weijiang > --- > hw/core/qdev-properties.c | 19 +++++++++++++++++++ > include/hw/qdev-properties.h | 12 ++++++++++++ > 2 files changed, 31 insertions(+) > > diff --git a/hw/core/qdev-properties.c b/hw/core/qdev-properties.c > index 50f40949f5..343a200784 100644 > --- a/hw/core/qdev-properties.c > +++ b/hw/core/qdev-properties.c > @@ -428,6 +428,25 @@ const PropertyInfo qdev_prop_int64 = { > .set_default_value = qdev_propinfo_set_default_value_int, > }; > > +static void set_uint64_checkmask(Object *obj, Visitor *v, const char *name, > + void *opaque, Error **errp) > +{ > + Property *prop = opaque; > + uint64_t *ptr = object_field_prop_ptr(obj, prop); > + > + visit_type_uint64(v, name, ptr, errp); > + if (*ptr & ~prop->bitmask) { > + error_setg(errp, "Property value for '%s' violates bitmask '0x%lx'", > + name, prop->bitmask); > + } > +} > + > +const PropertyInfo qdev_prop_uint64_checkmask = { > + .name = "uint64", > + .get = get_uint64, > + .set = set_uint64_checkmask, > +}; > + > /* --- string --- */ > > static void release_string(Object *obj, const char *name, void *opaque) > diff --git a/include/hw/qdev-properties.h b/include/hw/qdev-properties.h > index 0ef97d60ce..075882e8c1 100644 > --- a/include/hw/qdev-properties.h > +++ b/include/hw/qdev-properties.h > @@ -17,6 +17,7 @@ struct Property { > const PropertyInfo *info; > ptrdiff_t offset; > uint8_t bitnr; > + uint64_t bitmask; > bool set_default; > union { > int64_t i; > @@ -53,6 +54,7 @@ extern const PropertyInfo qdev_prop_uint16; > extern const PropertyInfo qdev_prop_uint32; > extern const PropertyInfo qdev_prop_int32; > extern const PropertyInfo qdev_prop_uint64; > +extern const PropertyInfo qdev_prop_uint64_checkmask; > extern const PropertyInfo qdev_prop_int64; > extern const PropertyInfo qdev_prop_size; > extern const PropertyInfo qdev_prop_string; > @@ -102,6 +104,16 @@ extern const PropertyInfo qdev_prop_link; > .set_default = true, \ > .defval.u = (bool)_defval) > > +/** > + * The DEFINE_PROP_UINT64_CHECKMASK macro checks a user-supplied value > + * against corresponding bitmask, rejects the value if it violates. > + * The default value is set in instance_init(). > + */ > +#define DEFINE_PROP_UINT64_CHECKMASK(_name, _state, _field, _bitmask) \ > + DEFINE_PROP(_name, _state, _field, qdev_prop_uint64_checkmask, uint64_t, \ > + .bitmask = (_bitmask), \ > + .set_default = false) > + > #define PROP_ARRAY_LEN_PREFIX "len-" > > /** > -- > 2.21.1