[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] xen-unstable warn_on in msi.c:636



On Thu, Jun 30, 2011 at 1:17 PM, Jan Beulich <JBeulich@xxxxxxxxxx> wrote:
>>>> On 30.06.11 at 14:03, George Dunlap <George.Dunlap@xxxxxxxxxxxxx> wrote:
>> Olaf / Jan / all, what do you think of the attached patch, which
>> teaches the MSI code how to deal with 0 pbas?
>
> No, definitely not an option. They simply must not be zero. The
> problem just is that the BARs for virtual functions all read as zero
> (and the real value must be calculated from information from the
> PF's config space).

But at the moment it IS zero, and this "real value" is simply not
being calculated.

I realize this needs to be fixed the Right Way eventually, but at the
moment anyone who uses SRIOV will have a bunch of scary warnings which
they can't do anything about except ignore; and it's never a good idea
to condition people to ignore this kind of warning.

If the current situation is safe enough that fixing it is a low
priority, then it's safe enough to remove the warnings for the time
being.  If it's not safe enough to remove the warnings, then it's not
safe enough to leave and a fix needs to be a priority.

 -George

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.