Revert "OvmfPkg/QemuVideoDxe: Helper functions for unaligned port I/O."

This reverts commit 05a5379458.

The VMWare SVGA display device implemented by Qemu (-vga vmware) uses
an I/O-type BAR which is laid out such that some register offsets are
not aligned to the read/write width with which they are expected to be
accessed. However, we reverted the initialization of VMWare SVGA device,
we don't need such unaligned I/O.

Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: yuchenlin <yuchenlin@synology.com>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Regression-tested-by: Laszlo Ersek <lersek@redhat.com>
(cherry picked from commit 438ada5aa5)
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
This commit is contained in:
yuchenlin 2018-11-02 11:24:00 +08:00 committed by Laszlo Ersek
parent 1358ecb77f
commit 9442266c70
6 changed files with 0 additions and 359 deletions

View File

@ -41,15 +41,9 @@
Qemu.h Qemu.h
[Sources.Ia32, Sources.X64] [Sources.Ia32, Sources.X64]
UnalignedIoGcc.c | GCC
UnalignedIoIcc.c | INTEL
UnalignedIoMsc.c | MSFT
VbeShim.c VbeShim.c
VbeShim.h VbeShim.h
[Sources.EBC]
UnalignedIoUnsupported.c
[Packages] [Packages]
MdePkg/MdePkg.dec MdePkg/MdePkg.dec
MdeModulePkg/MdeModulePkg.dec MdeModulePkg/MdeModulePkg.dec

View File

@ -1,70 +0,0 @@
/** @file
Unaligned Port I/O. This file has compiler specifics for GCC as there is no
ANSI C standard for doing IO.
Based on IoLibGcc.c.
Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials are licensed and made available
under the terms and conditions of the BSD License which accompanies this
distribution. The full text of the license may be found at
http://opensource.org/licenses/bsd-license.php.
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
**/
#include "UnalignedIoInternal.h"
/**
Performs a 32-bit write to the specified, possibly unaligned I/O-type
address.
Writes the 32-bit I/O port specified by Port with the value specified by
Value and returns Value. This function must guarantee that all I/O read and
write operations are serialized.
If 32-bit unaligned I/O port operations are not supported, then ASSERT().
@param[in] Port I/O port address
@param[in] Value 32-bit word to write
@return The value written to the I/O port.
**/
UINT32
UnalignedIoWrite32 (
IN UINTN Port,
IN UINT32 Value
)
{
__asm__ __volatile__ ( "outl %0, %1" : : "a" (Value), "d" ((UINT16)Port) );
return Value;
}
/**
Reads a 32-bit word from the specified, possibly unaligned I/O-type address.
Reads the 32-bit I/O port specified by Port. The 32-bit read value is
returned. This function must guarantee that all I/O read and write operations
are serialized.
If 32-bit unaligned I/O port operations are not supported, then ASSERT().
@param[in] Port The I/O port to read.
@return The value read.
**/
UINT32
UnalignedIoRead32 (
IN UINTN Port
)
{
UINT32 Data;
__asm__ __volatile__ ( "inl %1, %0" : "=a" (Data) : "d" ((UINT16)Port) );
return Data;
}

View File

@ -1,80 +0,0 @@
/** @file
Unaligned port I/O. This file has compiler specifics for ICC as there
is no ANSI C standard for doing IO.
Based on IoLibIcc.c.
Copyright (c) 2006 - 2008, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials are licensed and made available
under the terms and conditions of the BSD License which accompanies this
distribution. The full text of the license may be found at
http://opensource.org/licenses/bsd-license.php.
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
**/
#include "UnalignedIoInternal.h"
/**
Performs a 32-bit write to the specified, possibly unaligned I/O-type
address.
Writes the 32-bit I/O port specified by Port with the value specified by
Value and returns Value. This function must guarantee that all I/O read and
write operations are serialized.
If 32-bit unaligned I/O port operations are not supported, then ASSERT().
@param[in] Port I/O port address
@param[in] Value 32-bit word to write
@return The value written to the I/O port.
**/
UINT32
UnalignedIoWrite32 (
IN UINTN Port,
IN UINT32 Value
)
{
__asm {
mov eax, dword ptr [Value]
mov dx, word ptr [Port]
out dx, eax
}
return Value;
}
/**
Reads a 32-bit word from the specified, possibly unaligned I/O-type address.
Reads the 32-bit I/O port specified by Port. The 32-bit read value is
returned. This function must guarantee that all I/O read and write operations
are serialized.
If 32-bit unaligned I/O port operations are not supported, then ASSERT().
@param[in] Port The I/O port to read.
@return The value read.
**/
UINT32
UnalignedIoRead32 (
IN UINTN Port
)
{
UINT32 Data;
__asm {
mov dx, word ptr [Port]
in eax, dx
mov dword ptr [Data], eax
}
return Data;
}

View File

@ -1,59 +0,0 @@
/** @file
Unaligned port I/O, with implementations for various x86 compilers and a
dummy for platforms which do not support unaligned port I/O.
Copyright (c) 2017, Phil Dennis-Jordan.<BR>
This program and the accompanying materials are licensed and made available
under the terms and conditions of the BSD License which accompanies this
distribution. The full text of the license may be found at
http://opensource.org/licenses/bsd-license.php.
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
**/
#ifndef _UNALIGNED_IO_INTERNAL_H_
#define _UNALIGNED_IO_INTERNAL_H_
/**
Performs a 32-bit write to the specified, possibly unaligned I/O-type address.
Writes the 32-bit I/O port specified by Port with the value specified by Value
and returns Value. This function must guarantee that all I/O read and write
operations are serialized.
If 32-bit unaligned I/O port operations are not supported, then ASSERT().
@param[in] Port I/O port address
@param[in] Value 32-bit word to write
@return The value written to the I/O port.
**/
UINT32
UnalignedIoWrite32 (
IN UINTN Port,
IN UINT32 Value
);
/**
Reads a 32-bit word from the specified, possibly unaligned I/O-type address.
Reads the 32-bit I/O port specified by Port. The 32-bit read value is
returned. This function must guarantee that all I/O read and write operations
are serialized.
If 32-bit unaligned I/O port operations are not supported, then ASSERT().
@param[in] Port The I/O port to read.
@return The value read.
**/
UINT32
UnalignedIoRead32 (
IN UINTN Port
);
#endif

View File

@ -1,78 +0,0 @@
/** @file
Unaligned port I/O. This file has compiler specifics for Microsoft C as there
is no ANSI C standard for doing IO.
Based on IoLibMsc.c
Copyright (c) 2006 - 2010, Intel Corporation. All rights reserved.<BR>
This program and the accompanying materials are licensed and made available
under the terms and conditions of the BSD License which accompanies this
distribution. The full text of the license may be found at
http://opensource.org/licenses/bsd-license.php.
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
**/
#include "UnalignedIoInternal.h"
unsigned long _inpd (unsigned short port);
unsigned long _outpd (unsigned short port, unsigned long dataword );
void _ReadWriteBarrier (void);
/**
Performs a 32-bit write to the specified, possibly unaligned I/O-type
address.
Writes the 32-bit I/O port specified by Port with the value specified by
Value and returns Value. This function must guarantee that all I/O read and
write operations are serialized.
If 32-bit unaligned I/O port operations are not supported, then ASSERT().
@param[in] Port I/O port address
@param[in] Value 32-bit word to write
@return The value written to the I/O port.
**/
UINT32
UnalignedIoWrite32 (
IN UINTN Port,
IN UINT32 Value
)
{
_ReadWriteBarrier ();
_outpd ((UINT16)Port, Value);
_ReadWriteBarrier ();
return Value;
}
/**
Reads a 32-bit word from the specified, possibly unaligned I/O-type address.
Reads the 32-bit I/O port specified by Port. The 32-bit read value is
returned. This function must guarantee that all I/O read and write operations
are serialized.
If 32-bit unaligned I/O port operations are not supported, then ASSERT().
@param[in] Port The I/O port to read.
@return The value read.
**/
UINT32
UnalignedIoRead32 (
IN UINTN Port
)
{
UINT32 Value;
_ReadWriteBarrier ();
Value = _inpd ((UINT16)Port);
_ReadWriteBarrier ();
return Value;
}

View File

@ -1,66 +0,0 @@
/** @file
Unaligned port I/O dummy implementation for platforms which do not support it.
Copyright (c) 2017, Phil Dennis-Jordan.<BR>
This program and the accompanying materials are licensed and made available
under the terms and conditions of the BSD License which accompanies this
distribution. The full text of the license may be found at
http://opensource.org/licenses/bsd-license.php.
THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS,
WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED.
**/
#include <Library/DebugLib.h>
#include "UnalignedIoInternal.h"
/**
Performs a 32-bit write to the specified, possibly unaligned I/O-type
address.
Writes the 32-bit I/O port specified by Port with the value specified by
Value and returns Value. This function must guarantee that all I/O read and
write operations are serialized.
If 32-bit unaligned I/O port operations are not supported, then ASSERT().
@param[in] Port I/O port address
@param[in] Value 32-bit word to write
@return The value written to the I/O port.
**/
UINT32
UnalignedIoWrite32 (
IN UINTN Port,
IN UINT32 Value
)
{
ASSERT (FALSE);
return Value;
}
/**
Reads a 32-bit word from the specified, possibly unaligned I/O-type address.
Reads the 32-bit I/O port specified by Port. The 32-bit read value is
returned. This function must guarantee that all I/O read and write operations
are serialized.
If 32-bit unaligned I/O port operations are not supported, then ASSERT().
@param[in] Port The I/O port to read.
@return The value read.
**/
UINT32
UnalignedIoRead32 (
IN UINTN Port
)
{
ASSERT (FALSE);
return 0;
}