On 6/7/19 12:27 PM, Dan Williams wrote:
> @@ -848,15 +848,16 @@ char * __init efi_md_typeattr_format(char *buf, size_t 
> size,
>       if (attr & ~(EFI_MEMORY_UC | EFI_MEMORY_WC | EFI_MEMORY_WT |
>                    EFI_MEMORY_WB | EFI_MEMORY_UCE | EFI_MEMORY_RO |
>                    EFI_MEMORY_WP | EFI_MEMORY_RP | EFI_MEMORY_XP |
> -                  EFI_MEMORY_NV |
> +                  EFI_MEMORY_NV | EFI_MEMORY_SP |
>                    EFI_MEMORY_RUNTIME | EFI_MEMORY_MORE_RELIABLE))
>               snprintf(pos, size, "|attr=0x%016llx]",
>                        (unsigned long long)attr);
>       else
>               snprintf(pos, size,
> -                      "|%3s|%2s|%2s|%2s|%2s|%2s|%2s|%3s|%2s|%2s|%2s|%2s]",
> +                      
> "|%3s|%2s|%2s|%2s|%2s|%2s|%2s|%2s|%3s|%2s|%2s|%2s|%2s]",
>                        attr & EFI_MEMORY_RUNTIME ? "RUN" : "",
>                        attr & EFI_MEMORY_MORE_RELIABLE ? "MR" : "",
> +                      attr & EFI_MEMORY_SP      ? "SP"  : "",
>                        attr & EFI_MEMORY_NV      ? "NV"  : "",
>                        attr & EFI_MEMORY_XP      ? "XP"  : "",
>                        attr & EFI_MEMORY_RP      ? "RP"  : "",

Haha, I went digging in sysfs to find out where this gets dumped out.
The joke was on me because it seems to only go to dmesg.

Separate from these patches, should we have a runtime file that dumps
out the same info?  dmesg isn't always available, and hotplug could
change this too, I'd imagine.
_______________________________________________
Linux-nvdimm mailing list
Linux-nvdimm@lists.01.org
https://lists.01.org/mailman/listinfo/linux-nvdimm

Reply via email to