This patch improperly assumes that GRUB is the only thing in EFI that
transmits. Your patch surely fixed your machine but likely breaks some
other machines. Could you instead make an explicit check for (void *)1 and
add a comment on which machine it's necessary?
Le 5 août 2015 10:28 PM, "Josef Bacik" <jba...@fb.com> a écrit :

> On 08/05/2015 04:04 PM, Andrei Borzenkov wrote:
>
>> В Wed, 5 Aug 2015 14:36:37 -0400
>> Josef Bacik <jba...@fb.com> пишет:
>>
>> The EFI SNP documentation isn't super clear on the value that is returned
>>> in
>>> txbuf when calling into GetStatus.  The documentation says its the
>>> pointer to
>>> the recycle buffer, but the documentation for Transmit() says that it
>>> should be
>>> the pointer to the buffer that we transmitted.
>>>
>>
>> Actually it says "Recycled transmit buffer address" and further
>> "GetStatus() until the transmitted buffer shows up in the recycled
>> transmit buffer" so it looks reasonably clear to me.
>>
>>                                                On the boxes I'm using
>>> it's just
>>> a random pointer (usually 0x1).  It is definitely transmitting stuff,
>>> but the
>>> get_status call is not returning the pointer to the txbuf we passed in.
>>>
>>
>> Which sounds like firmware bug. To be sure - you observe it also using
>> current GIT master?
>>
>>
> This is on git master as of last week, so I have your latest patch
>
> efinet: enable hardware filters when opening interface
>
> and it was still happening.  I know what Transmit() says, but GetStatus()
> says it'll just be a pointer to the recycled transmit buffer address, which
> to me could mean the pointer to whatever the internal queue was being used
> by UEFI.  Anyway that's not important, what is important is that the
> current code doesn't work with hardware that exists in the wild.  If it's a
> firmware bug then fine, what do users do if they have buggy firmware that
> isn't being updated anymore?  I think making grub more tolerant to crappy
> firmware is a good thing.  Thanks,
>
> Josef
>
> _______________________________________________
> Grub-devel mailing list
> Grub-devel@gnu.org
> https://lists.gnu.org/mailman/listinfo/grub-devel
>
_______________________________________________
Grub-devel mailing list
Grub-devel@gnu.org
https://lists.gnu.org/mailman/listinfo/grub-devel

Reply via email to