From: "Aneesh Kumar K.V" <firstname.lastname@example.org> To: Vaibhav Jain <email@example.com>, firstname.lastname@example.org Cc: Vaibhav Jain <email@example.com> Subject: Re: [ndctl PATCH v2 2/6] libncdtl: Add initial support for NVDIMM_FAMILY_PAPR_SCM dimm family Date: Wed, 29 Apr 2020 13:25:47 +0530 [thread overview] Message-ID: <firstname.lastname@example.org> (raw) In-Reply-To: <email@example.com> Vaibhav Jain <firstname.lastname@example.org> writes: > Add necessary scaffolding in libndctl for dimms that support papr_scm > specification. Since there can be platforms that support > Open-Firmware but not the papr_scm specification, hence the changes > proposed first add support for probing if the dimm bus supports > Open-Firmware. This is done via querying for sysfs attribute 'of_node' > in dimm device sysfs directory. If available newly introduced member > 'struct ndctl_bus.has_of_node' is set. During the probe of the dimm > and execution of add_dimm(), the newly introduced add_of_pmem_dimm() > is called if dimm bus reports supports Open-Firmware. > > Function add_of_pmem_dimm() queries the 'compatible' device tree > attribute and based on its value assign NVDIMM_FAMILY_PAPR_SCM to the > dimm command family. In future, based on the contents of 'compatible' > attribute more of_pmem dimm families can be queried. > > We also add support for parsing the dimm flags for > NVDIMM_FAMILY_PAPR_SCM supporting nvdimms as described at . A newly > introduced function parse_papr_scm_flags() reads the contents of this > flag file and sets appropriate flag bits in 'struct > ndctl_dimm.flags'. The mixing of of_pmem and papr_scm is confuring here considering we have two different driver in the kernel. If both can be handled by the same code them possibly function that indicate both? ie, replace add_of_pmem_dimm() with something more generic? > > Also we advertise support for monitor mode by allocating a file > descriptor to the dimm 'flags' file and assigning it to 'struct > ndctl_dimm.health_event_fd'. > > The dimm-ops implementation for NVDIMM_FAMILY_PAPR_SCM is > available in global variable 'papr_scm_dimm_ops' which points to > skeleton implementation in newly introduced file 'lib/papr_scm.c'. > -aneesh _______________________________________________ Linux-nvdimm mailing list -- email@example.com To unsubscribe send an email to firstname.lastname@example.org
next prev parent reply other threads:[~2020-04-29 7:55 UTC|newest] Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top 2020-04-20 7:55 [ndctl PATCH v2 0/6] Add support for reporting papr-scm nvdimm health Vaibhav Jain 2020-04-20 7:55 ` [ndctl PATCH v2 1/6] libndctl: Refactor out add_dimm() to handle NFIT specific init Vaibhav Jain 2020-04-24 3:18 ` Santosh Sivaraj 2020-04-29 7:52 ` Aneesh Kumar K.V 2020-05-04 8:05 ` Vaibhav Jain 2020-04-20 7:55 ` [ndctl PATCH v2 2/6] libncdtl: Add initial support for NVDIMM_FAMILY_PAPR_SCM dimm family Vaibhav Jain 2020-04-29 7:55 ` Aneesh Kumar K.V [this message] 2020-05-04 8:31 ` Vaibhav Jain 2020-04-20 7:55 ` [ndctl PATCH v2 3/6] libndctl: Introduce new dimm-ops dimm_init() & dimm_uninit() Vaibhav Jain 2020-04-20 7:55 ` [ndctl PATCH v2 4/6] libndctl,papr_scm: Add definitions for PAPR nvdimm specific methods Vaibhav Jain 2020-04-20 7:55 ` [ndctl PATCH v2 5/6] libndctl,papr_scm: Add scaffolding to issue and handle PDSM requests Vaibhav Jain 2020-04-20 7:55 ` [ndctl PATCH v2 6/6] libndctl,papr_scm: Implement support for PAPR_SCM_PDSM_HEALTH Vaibhav Jain
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --subject='Re: [ndctl PATCH v2 2/6] libncdtl: Add initial support for NVDIMM_FAMILY_PAPR_SCM dimm family' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).