Skip to content
Projects
Groups
Snippets
Help
Loading...
Help
Support
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in / Register
Toggle navigation
L
linux
Project overview
Project overview
Details
Activity
Releases
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Issues
0
Issues
0
List
Boards
Labels
Milestones
Merge Requests
0
Merge Requests
0
Analytics
Analytics
Repository
Value Stream
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Create a new issue
Commits
Issue Boards
Open sidebar
Kirill Smelkov
linux
Commits
ced3960a
Commit
ced3960a
authored
Feb 09, 2023
by
Rafael J. Wysocki
Browse files
Options
Browse Files
Download
Plain Diff
Merge back cpufreq material for 6.3-rc1.
parents
918c5765
b9e6a2d4
Changes
9
Expand all
Show whitespace changes
Inline
Side-by-side
Showing
9 changed files
with
876 additions
and
254 deletions
+876
-254
Documentation/admin-guide/kernel-parameters.txt
Documentation/admin-guide/kernel-parameters.txt
+7
-0
Documentation/admin-guide/pm/amd-pstate.rst
Documentation/admin-guide/pm/amd-pstate.rst
+72
-2
drivers/acpi/cppc_acpi.c
drivers/acpi/cppc_acpi.c
+67
-0
drivers/cpufreq/Kconfig
drivers/cpufreq/Kconfig
+0
-10
drivers/cpufreq/Makefile
drivers/cpufreq/Makefile
+0
-1
drivers/cpufreq/amd-pstate.c
drivers/cpufreq/amd-pstate.c
+686
-19
drivers/cpufreq/loongson1-cpufreq.c
drivers/cpufreq/loongson1-cpufreq.c
+0
-222
include/acpi/cppc_acpi.h
include/acpi/cppc_acpi.h
+12
-0
include/linux/amd-pstate.h
include/linux/amd-pstate.h
+32
-0
No files found.
Documentation/admin-guide/kernel-parameters.txt
View file @
ced3960a
...
...
@@ -7020,3 +7020,10 @@
management firmware translates the requests into actual
hardware states (core frequency, data fabric and memory
clocks etc.)
active
Use amd_pstate_epp driver instance as the scaling driver,
driver provides a hint to the hardware if software wants
to bias toward performance (0x0) or energy efficiency (0xff)
to the CPPC firmware. then CPPC power algorithm will
calculate the runtime workload and adjust the realtime cores
frequency.
Documentation/admin-guide/pm/amd-pstate.rst
View file @
ced3960a
...
...
@@ -262,6 +262,25 @@ lowest non-linear performance in `AMD CPPC Performance Capability
<perf_cap_>`_.)
This attribute is read-only.
``energy_performance_available_preferences``
A list of all the supported EPP preferences that could be used for
``energy_performance_preference`` on this system.
These profiles represent different hints that are provided
to the low-level firmware about the user's desired energy vs efficiency
tradeoff. ``default`` represents the epp value is set by platform
firmware. This attribute is read-only.
``energy_performance_preference``
The current energy performance preference can be read from this attribute.
and user can change current preference according to energy or performance needs
Please get all support profiles list from
``energy_performance_available_preferences`` attribute, all the profiles are
integer values defined between 0 to 255 when EPP feature is enabled by platform
firmware, if EPP feature is disabled, driver will ignore the written value
This attribute is read-write.
Other performance and frequency values can be read back from
``/sys/devices/system/cpu/cpuX/acpi_cppc/``, see :ref:`cppc_sysfs`.
...
...
@@ -280,8 +299,30 @@ module which supports the new AMD P-States mechanism on most of the future AMD
platforms. The AMD P-States mechanism is the more performance and energy
efficiency frequency management method on AMD processors.
Kernel Module Options for ``amd-pstate``
=========================================
AMD Pstate Driver Operation Modes
=================================
``amd_pstate`` CPPC has two operation modes: CPPC Autonomous(active) mode and
CPPC non-autonomous(passive) mode.
active mode and passive mode can be chosen by different kernel parameters.
When in Autonomous mode, CPPC ignores requests done in the Desired Performance
Target register and takes into account only the values set to the Minimum requested
performance, Maximum requested performance, and Energy Performance Preference
registers. When Autonomous is disabled, it only considers the Desired Performance Target.
Active Mode
------------
``amd_pstate=active``
This is the low-level firmware control mode which is implemented by ``amd_pstate_epp``
driver with ``amd_pstate=active`` passed to the kernel in the command line.
In this mode, ``amd_pstate_epp`` driver provides a hint to the hardware if software
wants to bias toward performance (0x0) or energy efficiency (0xff) to the CPPC firmware.
then CPPC power algorithm will calculate the runtime workload and adjust the realtime
cores frequency according to the power supply and thermal, core voltage and some other
hardware conditions.
Passive Mode
------------
...
...
@@ -298,6 +339,35 @@ processor must provide at least nominal performance requested and go higher if c
operating conditions allow.
User Space Interface in ``sysfs``
=================================
Global Attributes
-----------------
``amd-pstate`` exposes several global attributes (files) in ``sysfs`` to
control its functionality at the system level. They are located in the
``/sys/devices/system/cpu/amd-pstate/`` directory and affect all CPUs.
``status``
Operation mode of the driver: "active", "passive" or "disable".
"active"
The driver is functional and in the ``active mode``
"passive"
The driver is functional and in the ``passive mode``
"disable"
The driver is unregistered and not functional now.
This attribute can be written to in order to change the driver's
operation mode or to unregister it. The string written to it must be
one of the possible values of it and, if successful, writing one of
these values to the sysfs file will cause the driver to switch over
to the operation mode represented by that string - or to be
unregistered in the "disable" case.
``cpupower`` tool support for ``amd-pstate``
===============================================
...
...
drivers/acpi/cppc_acpi.c
View file @
ced3960a
...
...
@@ -1153,6 +1153,19 @@ int cppc_get_nominal_perf(int cpunum, u64 *nominal_perf)
return
cppc_get_perf
(
cpunum
,
NOMINAL_PERF
,
nominal_perf
);
}
/**
* cppc_get_epp_perf - Get the epp register value.
* @cpunum: CPU from which to get epp preference value.
* @epp_perf: Return address.
*
* Return: 0 for success, -EIO otherwise.
*/
int
cppc_get_epp_perf
(
int
cpunum
,
u64
*
epp_perf
)
{
return
cppc_get_perf
(
cpunum
,
ENERGY_PERF
,
epp_perf
);
}
EXPORT_SYMBOL_GPL
(
cppc_get_epp_perf
);
/**
* cppc_get_perf_caps - Get a CPU's performance capabilities.
* @cpunum: CPU from which to get capabilities info.
...
...
@@ -1365,6 +1378,60 @@ int cppc_get_perf_ctrs(int cpunum, struct cppc_perf_fb_ctrs *perf_fb_ctrs)
}
EXPORT_SYMBOL_GPL
(
cppc_get_perf_ctrs
);
/*
* Set Energy Performance Preference Register value through
* Performance Controls Interface
*/
int
cppc_set_epp_perf
(
int
cpu
,
struct
cppc_perf_ctrls
*
perf_ctrls
,
bool
enable
)
{
int
pcc_ss_id
=
per_cpu
(
cpu_pcc_subspace_idx
,
cpu
);
struct
cpc_register_resource
*
epp_set_reg
;
struct
cpc_register_resource
*
auto_sel_reg
;
struct
cpc_desc
*
cpc_desc
=
per_cpu
(
cpc_desc_ptr
,
cpu
);
struct
cppc_pcc_data
*
pcc_ss_data
=
NULL
;
int
ret
;
if
(
!
cpc_desc
)
{
pr_debug
(
"No CPC descriptor for CPU:%d
\n
"
,
cpu
);
return
-
ENODEV
;
}
auto_sel_reg
=
&
cpc_desc
->
cpc_regs
[
AUTO_SEL_ENABLE
];
epp_set_reg
=
&
cpc_desc
->
cpc_regs
[
ENERGY_PERF
];
if
(
CPC_IN_PCC
(
epp_set_reg
)
||
CPC_IN_PCC
(
auto_sel_reg
))
{
if
(
pcc_ss_id
<
0
)
{
pr_debug
(
"Invalid pcc_ss_id for CPU:%d
\n
"
,
cpu
);
return
-
ENODEV
;
}
if
(
CPC_SUPPORTED
(
auto_sel_reg
))
{
ret
=
cpc_write
(
cpu
,
auto_sel_reg
,
enable
);
if
(
ret
)
return
ret
;
}
if
(
CPC_SUPPORTED
(
epp_set_reg
))
{
ret
=
cpc_write
(
cpu
,
epp_set_reg
,
perf_ctrls
->
energy_perf
);
if
(
ret
)
return
ret
;
}
pcc_ss_data
=
pcc_data
[
pcc_ss_id
];
down_write
(
&
pcc_ss_data
->
pcc_lock
);
/* after writing CPC, transfer the ownership of PCC to platform */
ret
=
send_pcc_cmd
(
pcc_ss_id
,
CMD_WRITE
);
up_write
(
&
pcc_ss_data
->
pcc_lock
);
}
else
{
ret
=
-
ENOTSUPP
;
pr_debug
(
"_CPC in PCC is not supported
\n
"
);
}
return
ret
;
}
EXPORT_SYMBOL_GPL
(
cppc_set_epp_perf
);
/**
* cppc_set_enable - Set to enable CPPC on the processor by writing the
* Continuous Performance Control package EnableRegister field.
...
...
drivers/cpufreq/Kconfig
View file @
ced3960a
...
...
@@ -3,7 +3,6 @@ menu "CPU Frequency scaling"
config CPU_FREQ
bool "CPU Frequency scaling"
select SRCU
help
CPU Frequency scaling allows you to change the clock speed of
CPUs on the fly. This is a nice method to save power, because
...
...
@@ -270,15 +269,6 @@ config LOONGSON2_CPUFREQ
Loongson2F and its successors support this feature.
If in doubt, say N.
config LOONGSON1_CPUFREQ
tristate "Loongson1 CPUFreq Driver"
depends on LOONGSON1_LS1B
help
This option adds a CPUFreq driver for loongson1 processors which
support software configurable cpu frequency.
If in doubt, say N.
endif
...
...
drivers/cpufreq/Makefile
View file @
ced3960a
...
...
@@ -111,7 +111,6 @@ obj-$(CONFIG_POWERNV_CPUFREQ) += powernv-cpufreq.o
obj-$(CONFIG_BMIPS_CPUFREQ)
+=
bmips-cpufreq.o
obj-$(CONFIG_IA64_ACPI_CPUFREQ)
+=
ia64-acpi-cpufreq.o
obj-$(CONFIG_LOONGSON2_CPUFREQ)
+=
loongson2_cpufreq.o
obj-$(CONFIG_LOONGSON1_CPUFREQ)
+=
loongson1-cpufreq.o
obj-$(CONFIG_SH_CPU_FREQ)
+=
sh-cpufreq.o
obj-$(CONFIG_SPARC_US2E_CPUFREQ)
+=
sparc-us2e-cpufreq.o
obj-$(CONFIG_SPARC_US3_CPUFREQ)
+=
sparc-us3-cpufreq.o
drivers/cpufreq/amd-pstate.c
View file @
ced3960a
This diff is collapsed.
Click to expand it.
drivers/cpufreq/loongson1-cpufreq.c
deleted
100644 → 0
View file @
918c5765
/*
* CPU Frequency Scaling for Loongson 1 SoC
*
* Copyright (C) 2014-2016 Zhang, Keguang <keguang.zhang@gmail.com>
*
* This file is licensed under the terms of the GNU General Public
* License version 2. This program is licensed "as is" without any
* warranty of any kind, whether express or implied.
*/
#include <linux/clk.h>
#include <linux/clk-provider.h>
#include <linux/cpu.h>
#include <linux/cpufreq.h>
#include <linux/delay.h>
#include <linux/io.h>
#include <linux/module.h>
#include <linux/platform_device.h>
#include <linux/slab.h>
#include <cpufreq.h>
#include <loongson1.h>
struct
ls1x_cpufreq
{
struct
device
*
dev
;
struct
clk
*
clk
;
/* CPU clk */
struct
clk
*
mux_clk
;
/* MUX of CPU clk */
struct
clk
*
pll_clk
;
/* PLL clk */
struct
clk
*
osc_clk
;
/* OSC clk */
unsigned
int
max_freq
;
unsigned
int
min_freq
;
};
static
struct
ls1x_cpufreq
*
cpufreq
;
static
int
ls1x_cpufreq_notifier
(
struct
notifier_block
*
nb
,
unsigned
long
val
,
void
*
data
)
{
if
(
val
==
CPUFREQ_POSTCHANGE
)
current_cpu_data
.
udelay_val
=
loops_per_jiffy
;
return
NOTIFY_OK
;
}
static
struct
notifier_block
ls1x_cpufreq_notifier_block
=
{
.
notifier_call
=
ls1x_cpufreq_notifier
};
static
int
ls1x_cpufreq_target
(
struct
cpufreq_policy
*
policy
,
unsigned
int
index
)
{
struct
device
*
cpu_dev
=
get_cpu_device
(
policy
->
cpu
);
unsigned
int
old_freq
,
new_freq
;
old_freq
=
policy
->
cur
;
new_freq
=
policy
->
freq_table
[
index
].
frequency
;
/*
* The procedure of reconfiguring CPU clk is as below.
*
* - Reparent CPU clk to OSC clk
* - Reset CPU clock (very important)
* - Reconfigure CPU DIV
* - Reparent CPU clk back to CPU DIV clk
*/
clk_set_parent
(
policy
->
clk
,
cpufreq
->
osc_clk
);
__raw_writel
(
__raw_readl
(
LS1X_CLK_PLL_DIV
)
|
RST_CPU_EN
|
RST_CPU
,
LS1X_CLK_PLL_DIV
);
__raw_writel
(
__raw_readl
(
LS1X_CLK_PLL_DIV
)
&
~
(
RST_CPU_EN
|
RST_CPU
),
LS1X_CLK_PLL_DIV
);
clk_set_rate
(
cpufreq
->
mux_clk
,
new_freq
*
1000
);
clk_set_parent
(
policy
->
clk
,
cpufreq
->
mux_clk
);
dev_dbg
(
cpu_dev
,
"%u KHz --> %u KHz
\n
"
,
old_freq
,
new_freq
);
return
0
;
}
static
int
ls1x_cpufreq_init
(
struct
cpufreq_policy
*
policy
)
{
struct
device
*
cpu_dev
=
get_cpu_device
(
policy
->
cpu
);
struct
cpufreq_frequency_table
*
freq_tbl
;
unsigned
int
pll_freq
,
freq
;
int
steps
,
i
;
pll_freq
=
clk_get_rate
(
cpufreq
->
pll_clk
)
/
1000
;
steps
=
1
<<
DIV_CPU_WIDTH
;
freq_tbl
=
kcalloc
(
steps
,
sizeof
(
*
freq_tbl
),
GFP_KERNEL
);
if
(
!
freq_tbl
)
return
-
ENOMEM
;
for
(
i
=
0
;
i
<
(
steps
-
1
);
i
++
)
{
freq
=
pll_freq
/
(
i
+
1
);
if
((
freq
<
cpufreq
->
min_freq
)
||
(
freq
>
cpufreq
->
max_freq
))
freq_tbl
[
i
].
frequency
=
CPUFREQ_ENTRY_INVALID
;
else
freq_tbl
[
i
].
frequency
=
freq
;
dev_dbg
(
cpu_dev
,
"cpufreq table: index %d: frequency %d
\n
"
,
i
,
freq_tbl
[
i
].
frequency
);
}
freq_tbl
[
i
].
frequency
=
CPUFREQ_TABLE_END
;
policy
->
clk
=
cpufreq
->
clk
;
cpufreq_generic_init
(
policy
,
freq_tbl
,
0
);
return
0
;
}
static
int
ls1x_cpufreq_exit
(
struct
cpufreq_policy
*
policy
)
{
kfree
(
policy
->
freq_table
);
return
0
;
}
static
struct
cpufreq_driver
ls1x_cpufreq_driver
=
{
.
name
=
"cpufreq-ls1x"
,
.
flags
=
CPUFREQ_NEED_INITIAL_FREQ_CHECK
,
.
verify
=
cpufreq_generic_frequency_table_verify
,
.
target_index
=
ls1x_cpufreq_target
,
.
get
=
cpufreq_generic_get
,
.
init
=
ls1x_cpufreq_init
,
.
exit
=
ls1x_cpufreq_exit
,
.
attr
=
cpufreq_generic_attr
,
};
static
int
ls1x_cpufreq_remove
(
struct
platform_device
*
pdev
)
{
cpufreq_unregister_notifier
(
&
ls1x_cpufreq_notifier_block
,
CPUFREQ_TRANSITION_NOTIFIER
);
cpufreq_unregister_driver
(
&
ls1x_cpufreq_driver
);
return
0
;
}
static
int
ls1x_cpufreq_probe
(
struct
platform_device
*
pdev
)
{
struct
plat_ls1x_cpufreq
*
pdata
=
dev_get_platdata
(
&
pdev
->
dev
);
struct
clk
*
clk
;
int
ret
;
if
(
!
pdata
||
!
pdata
->
clk_name
||
!
pdata
->
osc_clk_name
)
{
dev_err
(
&
pdev
->
dev
,
"platform data missing
\n
"
);
return
-
EINVAL
;
}
cpufreq
=
devm_kzalloc
(
&
pdev
->
dev
,
sizeof
(
struct
ls1x_cpufreq
),
GFP_KERNEL
);
if
(
!
cpufreq
)
return
-
ENOMEM
;
cpufreq
->
dev
=
&
pdev
->
dev
;
clk
=
devm_clk_get
(
&
pdev
->
dev
,
pdata
->
clk_name
);
if
(
IS_ERR
(
clk
))
{
dev_err
(
&
pdev
->
dev
,
"unable to get %s clock
\n
"
,
pdata
->
clk_name
);
return
PTR_ERR
(
clk
);
}
cpufreq
->
clk
=
clk
;
clk
=
clk_get_parent
(
clk
);
if
(
IS_ERR
(
clk
))
{
dev_err
(
&
pdev
->
dev
,
"unable to get parent of %s clock
\n
"
,
__clk_get_name
(
cpufreq
->
clk
));
return
PTR_ERR
(
clk
);
}
cpufreq
->
mux_clk
=
clk
;
clk
=
clk_get_parent
(
clk
);
if
(
IS_ERR
(
clk
))
{
dev_err
(
&
pdev
->
dev
,
"unable to get parent of %s clock
\n
"
,
__clk_get_name
(
cpufreq
->
mux_clk
));
return
PTR_ERR
(
clk
);
}
cpufreq
->
pll_clk
=
clk
;
clk
=
devm_clk_get
(
&
pdev
->
dev
,
pdata
->
osc_clk_name
);
if
(
IS_ERR
(
clk
))
{
dev_err
(
&
pdev
->
dev
,
"unable to get %s clock
\n
"
,
pdata
->
osc_clk_name
);
return
PTR_ERR
(
clk
);
}
cpufreq
->
osc_clk
=
clk
;
cpufreq
->
max_freq
=
pdata
->
max_freq
;
cpufreq
->
min_freq
=
pdata
->
min_freq
;
ret
=
cpufreq_register_driver
(
&
ls1x_cpufreq_driver
);
if
(
ret
)
{
dev_err
(
&
pdev
->
dev
,
"failed to register CPUFreq driver: %d
\n
"
,
ret
);
return
ret
;
}
ret
=
cpufreq_register_notifier
(
&
ls1x_cpufreq_notifier_block
,
CPUFREQ_TRANSITION_NOTIFIER
);
if
(
ret
)
{
dev_err
(
&
pdev
->
dev
,
"failed to register CPUFreq notifier: %d
\n
"
,
ret
);
cpufreq_unregister_driver
(
&
ls1x_cpufreq_driver
);
}
return
ret
;
}
static
struct
platform_driver
ls1x_cpufreq_platdrv
=
{
.
probe
=
ls1x_cpufreq_probe
,
.
remove
=
ls1x_cpufreq_remove
,
.
driver
=
{
.
name
=
"ls1x-cpufreq"
,
},
};
module_platform_driver
(
ls1x_cpufreq_platdrv
);
MODULE_ALIAS
(
"platform:ls1x-cpufreq"
);
MODULE_AUTHOR
(
"Kelvin Cheung <keguang.zhang@gmail.com>"
);
MODULE_DESCRIPTION
(
"Loongson1 CPUFreq driver"
);
MODULE_LICENSE
(
"GPL"
);
include/acpi/cppc_acpi.h
View file @
ced3960a
...
...
@@ -108,12 +108,14 @@ struct cppc_perf_caps {
u32
lowest_nonlinear_perf
;
u32
lowest_freq
;
u32
nominal_freq
;
u32
energy_perf
;
};
struct
cppc_perf_ctrls
{
u32
max_perf
;
u32
min_perf
;
u32
desired_perf
;
u32
energy_perf
;
};
struct
cppc_perf_fb_ctrs
{
...
...
@@ -149,6 +151,8 @@ extern bool cpc_ffh_supported(void);
extern
bool
cpc_supported_by_cpu
(
void
);
extern
int
cpc_read_ffh
(
int
cpunum
,
struct
cpc_reg
*
reg
,
u64
*
val
);
extern
int
cpc_write_ffh
(
int
cpunum
,
struct
cpc_reg
*
reg
,
u64
val
);
extern
int
cppc_get_epp_perf
(
int
cpunum
,
u64
*
epp_perf
);
extern
int
cppc_set_epp_perf
(
int
cpu
,
struct
cppc_perf_ctrls
*
perf_ctrls
,
bool
enable
);
#else
/* !CONFIG_ACPI_CPPC_LIB */
static
inline
int
cppc_get_desired_perf
(
int
cpunum
,
u64
*
desired_perf
)
{
...
...
@@ -202,6 +206,14 @@ static inline int cpc_write_ffh(int cpunum, struct cpc_reg *reg, u64 val)
{
return
-
ENOTSUPP
;
}
static
inline
int
cppc_set_epp_perf
(
int
cpu
,
struct
cppc_perf_ctrls
*
perf_ctrls
,
bool
enable
)
{
return
-
ENOTSUPP
;
}
static
inline
int
cppc_get_epp_perf
(
int
cpunum
,
u64
*
epp_perf
)
{
return
-
ENOTSUPP
;
}
#endif
/* !CONFIG_ACPI_CPPC_LIB */
#endif
/* _CPPC_ACPI_H*/
include/linux/amd-pstate.h
View file @
ced3960a
...
...
@@ -12,6 +12,11 @@
#include <linux/pm_qos.h>
#define AMD_CPPC_EPP_PERFORMANCE 0x00
#define AMD_CPPC_EPP_BALANCE_PERFORMANCE 0x80
#define AMD_CPPC_EPP_BALANCE_POWERSAVE 0xBF
#define AMD_CPPC_EPP_POWERSAVE 0xFF
/*********************************************************************
* AMD P-state INTERFACE *
*********************************************************************/
...
...
@@ -47,6 +52,10 @@ struct amd_aperf_mperf {
* @prev: Last Aperf/Mperf/tsc count value read from register
* @freq: current cpu frequency value
* @boost_supported: check whether the Processor or SBIOS supports boost mode
* @epp_policy: Last saved policy used to set energy-performance preference
* @epp_cached: Cached CPPC energy-performance preference value
* @policy: Cpufreq policy value
* @cppc_cap1_cached Cached MSR_AMD_CPPC_CAP1 register value
*
* The amd_cpudata is key private data for each CPU thread in AMD P-State, and
* represents all the attributes and goals that AMD P-State requests at runtime.
...
...
@@ -72,6 +81,29 @@ struct amd_cpudata {
u64
freq
;
bool
boost_supported
;
/* EPP feature related attributes*/
s16
epp_policy
;
s16
epp_cached
;
u32
policy
;
u64
cppc_cap1_cached
;
bool
suspended
;
};
/*
* enum amd_pstate_mode - driver working mode of amd pstate
*/
enum
amd_pstate_mode
{
AMD_PSTATE_DISABLE
=
0
,
AMD_PSTATE_PASSIVE
,
AMD_PSTATE_ACTIVE
,
AMD_PSTATE_MAX
,
};
static
const
char
*
const
amd_pstate_mode_string
[]
=
{
[
AMD_PSTATE_DISABLE
]
=
"disable"
,
[
AMD_PSTATE_PASSIVE
]
=
"passive"
,
[
AMD_PSTATE_ACTIVE
]
=
"active"
,
NULL
,
};
#endif
/* _LINUX_AMD_PSTATE_H */
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment