Re: [PATCH 1/1] x86/iommu: use bit structures for context_entry

From: Li, ZhenHua
Date: Fri Jan 10 2014 - 02:29:24 EST


I have not seen such a bug yet . but obviously a '=' should be used when you want to set a value.

for example, if x != 0,
x=10
and
x|=10
will cause different result.

ZhenHua

On 01/07/2014 10:41 PM, Joerg Roedel wrote:
On Fri, Dec 20, 2013 at 04:45:01PM +0800, Li, Zhen-Hua wrote:
There is a structure named context_entry used by intel iommu, and there
are some bit operations on it. Use bit structure may make these operations
easy.
Also the function context_set_address_root may cause problem because it uses
|= operation, not set the new value. And this patch can fix it.

What is the problem you are trying to fix here? Is it an actual bug?


Joerg



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/