Page MenuHomeFreeBSD

Paged/pageable data is not always paged.
ClosedPublic

Authored by tsoome on Feb 26 2016, 11:36 AM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 7 2024, 10:35 AM
Unknown Object (File)
Apr 7 2024, 9:48 AM
Unknown Object (File)
Apr 7 2024, 7:51 AM
Unknown Object (File)
Apr 7 2024, 7:00 AM
Unknown Object (File)
Mar 27 2024, 4:56 PM
Unknown Object (File)
Mar 17 2024, 11:01 AM
Unknown Object (File)
Mar 17 2024, 3:55 AM
Unknown Object (File)
Mar 9 2024, 8:11 AM

Details

Summary

This is another bit of work I have actually done on loader port on illumos
and pushing to fbsd.

The loader is using pager api to represent some data, enabling users
actually to view information, before it will be scrolled out of the screen.
This update adds more pager support for disk devices and implements
pager setup for few commands, which are missing this feature; notably
memmap/nvram for loader.efi. Note the EFI variables will actually need
more work to present the values, I think that would be outside the scope
of this update, however.

Test Plan

For simple test, verify the output of the loader commands.
For better test, one would need more elaborate environment, with more
disk devices, more partitions and more/complex zfs pools for lsdev -v
to verify the screen updates.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 3313
Build 3347: arc lint + arc unit

Event Timeline

tsoome retitled this revision from to Paged/pageable data in not always paged..
tsoome updated this object.
tsoome edited the test plan for this revision. (Show Details)

nvram commad was still buggy, wrong arguments used to initiate
the enumeration.

allanjude added inline comments.
sys/boot/common/dev_net.c
351–356

in order to avoid overflowing 80 character screen width, would it be better to use 8 spaces here rather than a tab?

update against revision 296323.

Updated to revision 296425.

Updated to revision 298256.

Updated to revision 298624.

Updated to revision 298901.

Updated to revision 301442.

tsoome retitled this revision from Paged/pageable data in not always paged. to Paged/pageable data is not always paged..Jun 6 2016, 11:08 AM
allanjude added a reviewer: allanjude.

/media/svn/tsoome/head/sys/boot/efi/libefi/efinet.c:354:12: error: unused variable 'ret' [-Werror,-Wunused-variable]

int unit, ret;
          ^

/media/svn/tsoome/head/sys/boot/efi/libefi/efinet.c:369:1: error: control reaches end of non-void function [-Werror,-Wreturn-type]

This problem is present in a large number of the modified functions

sys/boot/efi/libefi/efinet.c
340

this function has no return value, and ret is unused

sys/boot/efi/libefi/efipart.c
201

no return, ret unused

sys/boot/i386/libi386/bioscd.c
194

no return, ret unused

This revision now requires changes to proceed.Jul 30 2016, 6:19 PM

I deleted the nvram command since it was replaced by useful in scripting commands that also worked in interactive mode. Why is it back?

sys/boot/efi/loader/main.c
683

I thought I wrote commands that make this command no longer necessary and deleted it.

sys/boot/common/part.c
862–864

ret is unused here

In D5461#153258, @imp wrote:

I deleted the nvram command since it was replaced by useful in scripting commands that also worked in interactive mode. Why is it back?

yep, this diff is just that old, I need to review it and clean up properly - just wanted to get few others our of the queue first:)

tsoome edited edge metadata.

svn update, resync with updates since initial version.

/media/svn/tsoome/head/sys/boot/efi/libefi/efinet.c:354:12: error: unused variable 'ret' [-Werror,-Wunused-variable]

int unit, ret;
          ^

/media/svn/tsoome/head/sys/boot/efi/libefi/efinet.c:369:1: error: control reaches end of non-void function [-Werror,-Wreturn-type]

This problem is present in a large number of the modified functions

/media/svn/tsoome/head/sys/boot/efi/libefi/efinet.c:354:12: error: unused variable 'ret' [-Werror,-Wunused-variable]

int unit, ret;
          ^

/media/svn/tsoome/head/sys/boot/efi/libefi/efinet.c:369:1: error: control reaches end of non-void function [-Werror,-Wreturn-type]

This problem is present in a large number of the modified functions

yes, it was issue popped up after svn update, the merges were gone bad. I think I have resolved those now.

tsoome edited edge metadata.

Updated to revision 308195.

/media/svn/tsoome/head/sys/boot/efi/libefi/efinet.c:354:12: error: unused variable 'ret' [-Werror,-Wunused-variable]

int unit, ret;
          ^

/media/svn/tsoome/head/sys/boot/efi/libefi/efinet.c:369:1: error: control reaches end of non-void function [-Werror,-Wreturn-type]

This problem is present in a large number of the modified functions

/media/svn/tsoome/head/sys/boot/efi/libefi/efinet.c:354:12: error: unused variable 'ret' [-Werror,-Wunused-variable]

int unit, ret;
          ^

/media/svn/tsoome/head/sys/boot/efi/libefi/efinet.c:369:1: error: control reaches end of non-void function [-Werror,-Wreturn-type]

This problem is present in a large number of the modified functions

yes, it was issue popped up after svn update, the merges were gone bad. I think I have resolved those now.

sys/boot/common/dev_net.c
351–356

But tab is expanded to 8 spaces anyhow on console emulator. Or did I misunderstood you there?

allanjude edited edge metadata.

You have tested this? if so: Approved for commit

sys/boot/common/dev_net.c
351–356

You can ignore this comment. I figured the \t was only counted as a single character.

This revision is now accepted and ready to land.Nov 7 2016, 8:38 PM
This revision was automatically updated to reflect the committed changes.