[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 01/12] xen/trace: Don't over-read trace objects
- To: "andrew.cooper3@xxxxxxxxxx" <andrew.cooper3@xxxxxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
- From: Dario Faggioli <dfaggioli@xxxxxxxx>
- Date: Fri, 24 Sep 2021 14:51:25 +0000
- Accept-language: en-US
- Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=suse.com; dmarc=pass action=none header.from=suse.com; dkim=pass header.d=suse.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; bh=0ne2rZslJgOXXz3fWsZIIWBdbnwrHNB68NV7aproGxE=; b=BVFuzv/S/Fy20on8iswfiKiVXYv8RAHP1cNqyaXMxpLIEDh4mbAFnghasIzaBxgUgl2Z3kNrjH5w+Ya5pe5DOycuLANPGdD3534nxxJ0eE/F+FBnrffygjCgmrQZW3diGz8xQVI7DHzUBgvaAZyz7cj/g0Mfdt8N91z4O0gqeDep3D/gKMoqT8+7L7LRZl2sfsocjJrW6V97Q/iI4f0K4tPBfRFCk5wLmI2f6yUuhdLpGbhBcbxviaP2vBxTPGHSzZjjbC33G0c2y4X5hIQ2EGMM0htMJOqjFARDu10pTgElFN8hpqnD3sJwHGbLfRZ0O2ya0TiAEemqdUeVg/C+Lg==
- Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=gUsUM9nx+uYoiJn1rLc5AwG2dd01PZ6JldoGGVL2lBP3/ewTz5luAcTwNAlXhjq0jESFC9Ga+npBabDvm/slXeH3JJKV/01RWQx++V1kxDvzZsQSpx6cSes8Q5It39w09LKkIa4iD3Iv6+AqKEWTP/KjaL88ZPlmhl06qQTd6NSZ3rwUIYN9zNkERQoeRARwbqnjFyQsIOvC2wgn2KEVfXXex20CKQTPsCg6o1NxPIrjVsMe5YEJB6pqgEsRN/OoegTW9YVakEWYW837g9uksqONxCFj5QkzJIHNWY9mOq9JlvDHhIOw9GTwC2oPWIJUSc9K+OPH4jXd3xtcCIMAGA==
- Authentication-results: citrix.com; dkim=none (message not signed) header.d=none;citrix.com; dmarc=none action=none header.from=suse.com;
- Cc: "julien@xxxxxxx" <julien@xxxxxxx>, "George.Dunlap@xxxxxxxxxxxxx" <George.Dunlap@xxxxxxxxxxxxx>, "sstabellini@xxxxxxxxxx" <sstabellini@xxxxxxxxxx>, Jan Beulich <JBeulich@xxxxxxxx>, "iwj@xxxxxxxxxxxxxx" <iwj@xxxxxxxxxxxxxx>, "wl@xxxxxxx" <wl@xxxxxxx>, "mengxu@xxxxxxxxxxxxx" <mengxu@xxxxxxxxxxxxx>
- Delivery-date: Fri, 24 Sep 2021 14:51:38 +0000
- List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
- Thread-index: AQHXrkSLbMGUvlBXkkK2FQ1vuFXyFauzSsKA
- Thread-topic: [PATCH v2 01/12] xen/trace: Don't over-read trace objects
On Mon, 2021-09-20 at 18:25 +0100, Andrew Cooper wrote:
> There is one buggy race record, TRC_RTDS_BUDGET_BURN. As it must
> remain
> __packed (as cur_budget is misaligned), change bool has_extratime to
> uint32_t
> to compensate.
>
Mmm... maybe my understanding of data alignment inside structs is a bit
lacking, but what the actual issue here, and what would we need to do
to fix it (where, by fix, I mean us being able to get rid of the
`__packed`)?
If rearranging fields is not enough, we can think about making
priority_level and has_extratime smaller, or even combining them in
just one field and decode the information in xentrace.
Of course, I can send a patch for that myself, even as a followup of
this series when it's in, as soon as we agree about the best way
forward.
Thanks and Regards
--
Dario Faggioli, Ph.D
http://about.me/dario.faggioli
Virtualization Software Engineer
SUSE Labs, SUSE https://www.suse.com/
-------------------------------------------------------------------
<<This happens because _I_ choose it to happen!>> (Raistlin Majere)
Attachment:
signature.asc
Description: This is a digitally signed message part
|