[Xenomai-core] [PATCH] SMI-doc improvement / RTDM paper

[Xenomai-core] [PATCH] SMI-doc improvement / RTDM paper

[Jan Kiszka]

[-- Attachment #1.1: Type: text/plain, Size: 285 bytes --]

Hi Philippe,

this patch tries to clarify where to solve SMI-related latencies. Or
should we even mention the kconfig path?

Moreover, I would like to upload my RTL-Workshop paper about RTDM to
doc/nodist/pdf (~150k) so that it later makes it on the website as well. Ok?

Jan

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: smi-doc.patch --]
[-- Type: text/x-patch; name="smi-doc.patch", Size: 649 bytes --]

Index: ksrc/arch/i386/smi.c
===================================================================
--- ksrc/arch/i386/smi.c	(Revision 539)
+++ ksrc/arch/i386/smi.c	(Arbeitskopie)
@@ -209,8 +209,8 @@
 
 #else /* ! CONFIG_XENO_HW_SMI_WORKAROUND */
 
-    printk("Xenomai: Intel chipset found and SMI workaround not enabled,\n"
-           "       you may encounter high interrupt latencies.\n");
+    printk("Xenomai: Intel chipset found but SMI workaround not enabled in kernel\n"
+           "         configuration. You may encounter high interrupt latencies.\n");
     pci_dev_put(dev);
 
 #endif /* ! CONFIG_XENO_HW_SMI_WORKAROUND */

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 252 bytes --]

[Xenomai-core] Re: [PATCH] SMI-doc improvement / RTDM paper

[Philippe Gerum]

Jan Kiszka wrote:
> Hi Philippe,
> 
> this patch tries to clarify where to solve SMI-related latencies. Or
> should we even mention the kconfig path?
>

The name of the config switch involved should be enough, I guess.

> Moreover, I would like to upload my RTL-Workshop paper about RTDM to
> doc/nodist/pdf (~150k) so that it later makes it on the website as well. Ok?
> 

Yes, go ahead.

> Jan
> 
> 
> ------------------------------------------------------------------------
> 
> Index: ksrc/arch/i386/smi.c
> ===================================================================
> --- ksrc/arch/i386/smi.c	(Revision 539)
> +++ ksrc/arch/i386/smi.c	(Arbeitskopie)
> @@ -209,8 +209,8 @@
>  
>  #else /* ! CONFIG_XENO_HW_SMI_WORKAROUND */
>  
> -    printk("Xenomai: Intel chipset found and SMI workaround not enabled,\n"
> -           "       you may encounter high interrupt latencies.\n");
> +    printk("Xenomai: Intel chipset found but SMI workaround not enabled in kernel\n"
> +           "         configuration. You may encounter high interrupt latencies.\n");
>      pci_dev_put(dev);
>  
>  #endif /* ! CONFIG_XENO_HW_SMI_WORKAROUND */


-- 

Philippe.

Re: [Xenomai-core] Re: [PATCH] SMI-doc improvement / RTDM paper

[Jan Kiszka]

[-- Attachment #1.1: Type: text/plain, Size: 312 bytes --]

Philippe Gerum wrote:
> Jan Kiszka wrote:
>> Hi Philippe,
>>
>> this patch tries to clarify where to solve SMI-related latencies. Or
>> should we even mention the kconfig path?
>>
> 
> The name of the config switch involved should be enough, I guess.

What about this? Please apply if it's ok.

Jan

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.2: smi-doc.patch --]
[-- Type: text/x-patch; name="smi-doc.patch", Size: 689 bytes --]

Index: ksrc/arch/i386/smi.c
===================================================================
--- ksrc/arch/i386/smi.c	(Revision 539)
+++ ksrc/arch/i386/smi.c	(Arbeitskopie)
@@ -209,8 +209,9 @@
 
 #else /* ! CONFIG_XENO_HW_SMI_WORKAROUND */
 
-    printk("Xenomai: Intel chipset found and SMI workaround not enabled,\n"
-           "       you may encounter high interrupt latencies.\n");
+    printk("Xenomai: Intel chipset found but SMI workaround not enabled (check\n"
+           "         CONFIG_XENO_HW_SMI_WORKAROUND). You may encounter high\n"
+           "         interrupt latencies!\n");
     pci_dev_put(dev);
 
 #endif /* ! CONFIG_XENO_HW_SMI_WORKAROUND */

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 252 bytes --]

Re: [Xenomai-core] Re: [PATCH] SMI-doc improvement / RTDM paper

[Philippe Gerum]

Jan Kiszka wrote:
> Philippe Gerum wrote:
> 
>>Jan Kiszka wrote:
>>
>>>Hi Philippe,
>>>
>>>this patch tries to clarify where to solve SMI-related latencies. Or
>>>should we even mention the kconfig path?
>>>
>>
>>The name of the config switch involved should be enough, I guess.
> 
> 
> What about this? Please apply if it's ok.
> 

Done, thanks.

> Jan
> 
> 
> ------------------------------------------------------------------------
> 
> Index: ksrc/arch/i386/smi.c
> ===================================================================
> --- ksrc/arch/i386/smi.c	(Revision 539)
> +++ ksrc/arch/i386/smi.c	(Arbeitskopie)
> @@ -209,8 +209,9 @@
>  
>  #else /* ! CONFIG_XENO_HW_SMI_WORKAROUND */
>  
> -    printk("Xenomai: Intel chipset found and SMI workaround not enabled,\n"
> -           "       you may encounter high interrupt latencies.\n");
> +    printk("Xenomai: Intel chipset found but SMI workaround not enabled (check\n"
> +           "         CONFIG_XENO_HW_SMI_WORKAROUND). You may encounter high\n"
> +           "         interrupt latencies!\n");
>      pci_dev_put(dev);
>  
>  #endif /* ! CONFIG_XENO_HW_SMI_WORKAROUND */


-- 

Philippe.

Re: [Xenomai-core] [PATCH] SMI-doc improvement / RTDM paper

[Hannes Mayer]

Hi all!

Jan Kiszka wrote:
[...]
> +    printk("Xenomai: Intel chipset found but SMI workaround not enabled in kernel\n"
> +           "         configuration. You may encounter high interrupt latencies.\n");

Speaking of printk. Since 2.1 I don't see any messages in the
kernel log. I bet there is a new switch somewhere, but I
can't find it. Any hint appreciated.

BTW, Jan, the SMI seems to be disabled now, even if I haven't
seen the message of disabling SMI in the klog.

Thanks & best regards,
Hannes.

Re: [Xenomai-core] [PATCH] SMI-doc improvement / RTDM paper

[Philippe Gerum]

Hannes Mayer wrote:
> Hi all!
> 
> Jan Kiszka wrote:
> [...]
> 
>> +    printk("Xenomai: Intel chipset found but SMI workaround not 
>> enabled in kernel\n"
>> +           "         configuration. You may encounter high interrupt 
>> latencies.\n");
> 
> 
> Speaking of printk. Since 2.1 I don't see any messages in the
> kernel log. I bet there is a new switch somewhere, but I
> can't find it. Any hint appreciated.
> 

dmesg | grep -i xenomai: ?

> BTW, Jan, the SMI seems to be disabled now, even if I haven't
> seen the message of disabling SMI in the klog.
> 
> Thanks & best regards,
> Hannes.
> 
> 


-- 

Philippe.

Re: [Xenomai-core] [PATCH] SMI-doc improvement / RTDM paper

[Hannes Mayer]

Philippe Gerum wrote:
[...]
> dmesg | grep -i xenomai: ?

Yeah, got it now!
Sorry for the silly question!

I forgot that as of 2.1 everything is compiled into the kernel
by default, so when running a testsuite example there are no
messages that xeno is loaded or unloaded.

Thanks and best regards,
Hannes.