OvmfPkg/VirtioFsDxe: implement EFI_FILE_PROTOCOL.Write()

Using the functions introduced previously, we can now implement
VirtioFsSimpleFileWrite().

Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>
Cc: Jordan Justen <jordan.l.justen@intel.com>
Cc: Philippe Mathieu-Daudé <philmd@redhat.com>
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3097
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Message-Id: <20201216211125.19496-40-lersek@redhat.com>
Acked-by: Ard Biesheuvel <ard.biesheuvel@arm.com>
This commit is contained in:
Laszlo Ersek 2020-12-16 22:11:16 +01:00 committed by mergify[bot]
parent 6f7bc7196f
commit 44f43f94ce
1 changed files with 62 additions and 1 deletions

View File

@ -16,5 +16,66 @@ VirtioFsSimpleFileWrite (
IN VOID *Buffer IN VOID *Buffer
) )
{ {
return EFI_NO_MEDIA; VIRTIO_FS_FILE *VirtioFsFile;
VIRTIO_FS *VirtioFs;
EFI_STATUS Status;
UINTN Transferred;
UINTN Left;
VirtioFsFile = VIRTIO_FS_FILE_FROM_SIMPLE_FILE (This);
VirtioFs = VirtioFsFile->OwnerFs;
if (VirtioFsFile->IsDirectory) {
return EFI_UNSUPPORTED;
}
if (!VirtioFsFile->IsOpenForWriting) {
return EFI_ACCESS_DENIED;
}
Status = EFI_SUCCESS;
Transferred = 0;
Left = *BufferSize;
while (Left > 0) {
UINT32 WriteSize;
//
// Honor the write buffer size limit.
//
WriteSize = (UINT32)MIN ((UINTN)VirtioFs->MaxWrite, Left);
Status = VirtioFsFuseWrite (
VirtioFs,
VirtioFsFile->NodeId,
VirtioFsFile->FuseHandle,
VirtioFsFile->FilePosition + Transferred,
&WriteSize,
(UINT8 *)Buffer + Transferred
);
if (!EFI_ERROR (Status) && WriteSize == 0) {
//
// Progress should have been made.
//
Status = EFI_DEVICE_ERROR;
}
if (EFI_ERROR (Status)) {
break;
}
Transferred += WriteSize;
Left -= WriteSize;
}
*BufferSize = Transferred;
VirtioFsFile->FilePosition += Transferred;
//
// According to the UEFI spec,
//
// - 'Partial writes only occur when there has been a data error during the
// write attempt (such as "file space full")', and
//
// - (as an example) EFI_VOLUME_FULL is returned when 'The volume is full'.
//
// These together imply that after a partial write, we have to return an
// error. In other words, (Transferred > 0) is inconsequential for the return
// value.
//
return Status;
} }