Page MenuHomeFreeBSD

LinuxKPI: Implement default sysfs kobject attribute operations
ClosedPublic

Authored by wulf on Dec 6 2021, 9:59 AM.
Tags
None
Referenced Files
F166320293: D33292.id101223.diff
Wed, Aug 12, 9:34 PM
F166307622: D33292.id.diff
Wed, Aug 12, 7:34 PM
Unknown Object (File)
Wed, Aug 12, 1:52 AM
Unknown Object (File)
Tue, Aug 11, 6:47 PM
Unknown Object (File)
Tue, Aug 11, 8:01 AM
Unknown Object (File)
Mon, Aug 10, 6:42 AM
Unknown Object (File)
Mon, Aug 10, 6:32 AM
Unknown Object (File)
Sat, Aug 8, 2:28 AM

Diff Detail

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

Event Timeline

wulf requested review of this revision.Dec 6 2021, 9:59 AM
sys/compat/linuxkpi/common/include/linux/kobject.h
71

Should this definition be moved to:

sys/compat/linuxkpi/common/include/linux/sysfs.h
sys/compat/linuxkpi/common/include/linux/kobject.h
71
sys/compat/linuxkpi/common/include/linux/kobject.h
71

OK, I think you need to add:

struct sysfs_ops;

At the beginning of this file to avoid issues with standalone inclusion.

sys/compat/linuxkpi/common/include/linux/kobject.h
71

It looks unnecessary. I just placed

extern struct foo bar;

line in to this file and it still compiles

OK, try -Wall just in case.

This revision is now accepted and ready to land.Dec 6 2021, 11:30 AM