Page MenuHomeFreeBSD

xen: move x86/xen/xenpv.c to dev/xen/bus/xenpv.c
ClosedPublic

Authored by ehem_freebsd_m5p.com on Mar 3 2021, 4:34 PM.
Tags
None
Referenced Files
F166431494: D29042.id92865.diff
Thu, Aug 13, 1:20 PM
F166417280: D29042.id92865.diff
Thu, Aug 13, 12:19 PM
Unknown Object (File)
Sat, Aug 8, 1:26 PM
Unknown Object (File)
Fri, Aug 7, 9:03 AM
Unknown Object (File)
Wed, Aug 5, 6:41 PM
Unknown Object (File)
Wed, Aug 5, 12:51 AM
Unknown Object (File)
Sun, Aug 2, 5:54 AM
Unknown Object (File)
Sun, Aug 2, 4:30 AM

Details

Summary

Minor changes are necessary to make this processor-independent, but
moving the file out of x86 and into common is the first step (so others
don't add /more/ x86-isms).

Adapted-to-HEAD-by: Elliott Mitchell <ehem+freebsd@m5p.com>

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

I would consider placing it in sys/dev/xen/bus/xenpv.c

Yeah, I've got no idea what-so-ever why one might feel xenpv.c should be
in a "bus" directory. No similarity to any type of bus. No devices
attached at all. Certainly not a bus.

Like so many comments, that one is pretty obvious when suggested.

Yeah, I've got no idea what-so-ever why one might feel xenpv.c should be
in a "bus" directory. No similarity to any type of bus. No devices
attached at all. Certainly not a bus.

I'm confused by the text above, do you agree it should be in a bus/ directory or not?

There are Xen devices (xen timer, xenstore) that attach to the xenpv bus.

I'm confused by the text above, do you agree it should be in a bus/ directory or not?

There are Xen devices (xen timer, xenstore) that attach to the xenpv bus.

My actual view should be reflected in the diff update. I went ahead and moved as requested. Felt like a 20/20 hindsight comment, it does look like a bus since it has devices attached and as such the idea is entirely reasonable.

ehem_freebsd_m5p.com retitled this revision from xen: move x86/xen/xenpv.c to dev/xen/xenpv.c to xen: move x86/xen/xenpv.c to dev/xen/bus/xenpv.c.Mar 4 2021, 9:48 PM
This revision is now accepted and ready to land.Mar 25 2021, 4:02 PM
This revision was automatically updated to reflect the committed changes.