Re: [PATCH 03/11] Introduce slices

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Sascha,

Am 22.04.20 um 09:54 schrieb Sascha Hauer:
> diff --git a/commands/Kconfig b/commands/Kconfig
> index 7fb47b8fb5..5ff6454427 100644
> --- a/commands/Kconfig
> +++ b/commands/Kconfig
> @@ -269,6 +269,15 @@ config CMD_POLLER
>  	  is_timeout() or one of the various delay functions. The poller command prints
>  	  informations about registered pollers.
>  
> +config CMD_SLICE
> +	tristate
> +	prompt "slice"
> +	depends on SLICE
> +	help
> +	  slices are a way to protect resources from being accessed by pollers. The slice
> +	  command can be used to print informations about slices and also to manipulate
> +	  them on the command line for debugging purposes.
> +
>  # end Information commands
>  endmenu
>  
> diff --git a/common/Kconfig b/common/Kconfig
> index 400c0553cf..bd2aebac75 100644
> --- a/common/Kconfig
> +++ b/common/Kconfig
> @@ -913,6 +913,10 @@ config BAREBOXCRC32_TARGET
>  config POLLER
>  	bool "generic polling infrastructure"
>  
> +config SLICE
> +	depends on POLLER
> +	default y
> +
>  config STATE
>  	bool "generic state infrastructure"
>  	select CRC32
> diff --git a/common/Makefile b/common/Makefile
> index 84463b4d48..16f14db41c 100644
> --- a/common/Makefile
> +++ b/common/Makefile
> @@ -11,6 +11,7 @@ obj-y				+= bootsource.o
>  obj-$(CONFIG_ELF)		+= elf.o
>  obj-y				+= restart.o
>  obj-y				+= poweroff.o
> +obj-y				+= slice.o
>  obj-$(CONFIG_MACHINE_ID)	+= machine_id.o
>  obj-$(CONFIG_AUTO_COMPLETE)	+= complete.o
>  obj-y				+= version.o

the Kconfig logic doesn't make sense to me. slice.o gets built regardless of
CONFIG_SLICE and it will still compile and link when CONFIG_POLLER is disabled.
On the other hand poller.o will fail to link when slice.o is omitted.

I suggest we drop CONFIG_SLICE.

Best regards,

  Daniel


-- 
Dipl.-Math. Daniel Glöckner, emlix GmbH, http://www.emlix.com
Fon +49 551 30664-0, Fax +49 551 30664-11,
Gothaer Platz 3, 37083 Göttingen, Germany
Sitz der Gesellschaft: Göttingen, Amtsgericht Göttingen HR B 3160
Geschäftsführung: Heike Jordan, Dr. Uwe Kracke
Ust-IdNr.: DE 205 198 055

emlix - your embedded linux partner

_______________________________________________
barebox mailing list
barebox@xxxxxxxxxxxxxxxxxxx
http://lists.infradead.org/mailman/listinfo/barebox




[Index of Archives]     [Linux Embedded]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux