Page MenuHomeFreeBSD

riscv: bus: provide bus_space_set_{multi,region}_stream definitions
AcceptedPublic

Authored by zishun.yi.dev_gmail.com on Mon, Jun 1, 10:16 AM.
Tags
None
Referenced Files
F159137489: D57378.diff
Wed, Jun 10, 12:50 PM
Unknown Object (File)
Tue, Jun 9, 8:52 PM
Unknown Object (File)
Mon, Jun 8, 11:40 AM
Unknown Object (File)
Mon, Jun 8, 11:31 AM
Unknown Object (File)
Sun, Jun 7, 5:18 PM
Unknown Object (File)
Sun, Jun 7, 5:12 PM
Unknown Object (File)
Sun, Jun 7, 12:00 PM
Unknown Object (File)
Sun, Jun 7, 12:00 PM

Details

Reviewers
mhorne
markj
minsoochoo0122_proton.me
Group Reviewers
riscv
Summary

These definitions are introduced for KASAN convenience.

KASAN intercepts the original defines at compile time, so having original definitions simplifies porting the KASAN.

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 73609
Build 70492: arc lint + arc unit

Event Timeline

I would like to know what this is for. Code looks okay, but I wonder why we need these aliases. Could you elaborate more in the commit message and revision summary?

I would like to know what this is for. Code looks okay, but I wonder why we need these aliases. Could you elaborate more in the commit message and revision summary?

Thanks for taking a look.
This is for KASAN convenience — it intercepts these defines, so the aliases simplify porting.
I'll update the revision summary to explain this clearly.

Both segments are placed in wrong lines. Please refer to sys/arm64/include/bus.h for correct order.

This revision now requires changes to proceed.Mon, Jun 1, 5:33 PM
This revision is now accepted and ready to land.Mon, Jun 1, 6:23 PM