| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Date: Mon, 4 Dec 2023 12:23:42 +0000
From: Avri Altman <Avri.Altman@....com>
To: Bean Huo <beanhuo@...pp.de>,
"bvanassche@....org" <bvanassche@....org>,
"alim.akhtar@...sung.com" <alim.akhtar@...sung.com>,
"jejb@...ux.ibm.com" <jejb@...ux.ibm.com>,
"martin.petersen@...cle.com" <martin.petersen@...cle.com>,
"mani@...nel.org" <mani@...nel.org>,
"quic_cang@...cinc.com" <quic_cang@...cinc.com>,
"quic_asutoshd@...cinc.com" <quic_asutoshd@...cinc.com>,
"beanhuo@...ron.com" <beanhuo@...ron.com>,
"thomas@...ch.de" <thomas@...ch.de>
CC: "linux-scsi@...r.kernel.org" <linux-scsi@...r.kernel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"mikebi@...ron.com" <mikebi@...ron.com>,
"lporzio@...ron.com" <lporzio@...ron.com>
Subject: RE: [PATCH v3 2/3] scsi: ufs: core: Add UFS RTC support
>
> > +static void ufs_init_rtc(struct ufs_hba *hba, u8 *desc_buf) {
> > + struct ufs_dev_info *dev_info = &hba->dev_info;
> > + u16 periodic_rtc_update =
> > +get_unaligned_be16(&desc_buf[DEVICE_DESC_PARAM_FRQ_RTC]);
> > +
> > + if (periodic_rtc_update & UFS_RTC_TIME_BASELINE) {
> Is it allowed to ignore the update period as depicted by
> wPeriodicRTCUpdate?
> And choose a different update period, e.g. 10 seconds instead?
> If yes - then It deserve a comment IMO.
Oh - the comment is in the 3rd patch.
Sorry - please ignore.
Thanks,
Avri
>
> Thanks,
> Avri
>
> > + dev_info->rtc_type = UFS_RTC_ABSOLUTE;
> > + /*
> > + * The concept of measuring time in Linux as the
> > + number of seconds
> > elapsed since
> > + * 00:00:00 UTC on January 1, 1970, and UFS ABS RTC is
> > + elapsed
> > from January 1st
> > + * 2010 00:00, here we need to adjust ABS baseline.
> > + */
> > + dev_info->rtc_time_baseline = mktime64(2010, 1, 1, 0, 0, 0) -
> > + mktime64(1970, 1, 1, 0, 0, 0);
> > + } else {
> > + dev_info->rtc_type = UFS_RTC_RELATIVE;
> > + dev_info->rtc_time_baseline = 0;
> > + }
> > +
> > + INIT_DELAYED_WORK(&hba->ufs_rtc_update_work,
> ufshcd_rtc_work);
> > }
> > +
> > static int ufs_get_device_desc(struct ufs_hba *hba) {
> > int err;
> > @@ -8241,6 +8310,8 @@ static int ufs_get_device_desc(struct ufs_hba
> > *hba)
> >
> > ufshcd_temp_notif_probe(hba, desc_buf);
> >
> > + ufs_init_rtc(hba, desc_buf);
> > +
> > if (hba->ext_iid_sup)
> > ufshcd_ext_iid_probe(hba, desc_buf);
> >
> > @@ -8794,6 +8865,8 @@ static int ufshcd_device_init(struct ufs_hba
> > *hba, bool init_dev_params)
> > ufshcd_force_reset_auto_bkops(hba);
> >
> > ufshcd_set_timestamp_attr(hba);
> > + schedule_delayed_work(&hba->ufs_rtc_update_work,
> > +
> > + msecs_to_jiffies(UFS_RTC_UPDATE_INTERVAL_MS));
> >
> > /* Gear up to HS gear if supported */
> > if (hba->max_pwr_info.is_valid) { @@ -9750,6 +9823,8 @@ static
> > int __ufshcd_wl_suspend(struct ufs_hba *hba, enum ufs_pm_op pm_op)
> > ret = ufshcd_vops_suspend(hba, pm_op, POST_CHANGE);
> > if (ret)
> > goto set_link_active;
> > +
> > + cancel_delayed_work_sync(&hba->ufs_rtc_update_work);
> > goto out;
> >
> > set_link_active:
> > @@ -9844,6 +9919,8 @@ static int __ufshcd_wl_resume(struct ufs_hba
> > *hba, enum ufs_pm_op pm_op)
> > if (ret)
> > goto set_old_link_state;
> > ufshcd_set_timestamp_attr(hba);
> > + schedule_delayed_work(&hba->ufs_rtc_update_work,
> > +
> > + msecs_to_jiffies(UFS_RTC_UPDATE_INTERVAL_MS));
> > }
> >
> > if (ufshcd_keep_autobkops_enabled_except_suspend(hba))
> > diff --git a/include/ufs/ufs.h b/include/ufs/ufs.h index
> > e77ab1786856..8022d267fe8a 100644
> > --- a/include/ufs/ufs.h
> > +++ b/include/ufs/ufs.h
> > @@ -14,6 +14,7 @@
> > #include <linux/bitops.h>
> > #include <linux/types.h>
> > #include <uapi/scsi/scsi_bsg_ufs.h>
> > +#include <linux/time64.h>
> >
> > /*
> > * Using static_assert() is not allowed in UAPI header files. Hence
> > the check @@ -551,6 +552,15 @@ struct ufs_vreg_info {
> > struct ufs_vreg *vdd_hba;
> > };
> >
> > +/*
> > + * UFS device descriptor wPeriodicRTCUpdate bit9 defines RTC time
> > baseline.
> > + */
> > +#define UFS_RTC_TIME_BASELINE BIT(9)
> > +enum ufs_rtc_time {
> > + UFS_RTC_RELATIVE,
> > + UFS_RTC_ABSOLUTE
> > +};
> > +
> > struct ufs_dev_info {
> > bool f_power_on_wp_en;
> > /* Keeps information if any of the LU is power on write
> > protected */ @@
> > -578,6 +588,10 @@ struct ufs_dev_info {
> >
> > /* UFS EXT_IID Enable */
> > bool b_ext_iid_en;
> > +
> > + /* UFS RTC */
> > + enum ufs_rtc_time rtc_type;
> > + time64_t rtc_time_baseline;
> > };
> >
> > /*
> > diff --git a/include/ufs/ufshcd.h b/include/ufs/ufshcd.h index
> > 7f0b2c5599cd..7bdda92fcb1c 100644
> > --- a/include/ufs/ufshcd.h
> > +++ b/include/ufs/ufshcd.h
> > @@ -911,6 +911,8 @@ enum ufshcd_mcq_opr {
> > * @mcq_base: Multi circular queue registers base address
> > * @uhq: array of supported hardware queues
> > * @dev_cmd_queue: Queue for issuing device management commands
> > + * @mcq_opr: MCQ operation and runtime registers
> > + * @ufs_rtc_update_work: A work for UFS RTC periodic update
> > */
> > struct ufs_hba {
> > void __iomem *mmio_base;
> > @@ -1071,6 +1073,8 @@ struct ufs_hba {
> > struct ufs_hw_queue *uhq;
> > struct ufs_hw_queue *dev_cmd_queue;
> > struct ufshcd_mcq_opr_info_t mcq_opr[OPR_MAX];
> > +
> > + struct delayed_work ufs_rtc_update_work;
> > };
> >
> > /**
> > --
> > 2.34.1
> >
Powered by blists - more mailing lists