forked from suyu/suyu
kernel/resource_limit: Remove the name member from resource limits
This doesn't really provide any benefit to the resource limit interface. There's no way for callers to any of the service functions for resource limits to provide a custom name, so all created instances of resource limits other than the system resource limit would have a name of "Unknown". The system resource limit itself is already trivially identifiable from its limit values, so there's no real need to take up space in the object to identify one object meaningfully out of N total objects.
This commit is contained in:
parent
62860dc0b0
commit
d09e98f566
3 changed files with 6 additions and 14 deletions
|
@ -115,7 +115,7 @@ struct KernelCore::Impl {
|
||||||
|
|
||||||
// Creates the default system resource limit
|
// Creates the default system resource limit
|
||||||
void InitializeSystemResourceLimit(KernelCore& kernel) {
|
void InitializeSystemResourceLimit(KernelCore& kernel) {
|
||||||
system_resource_limit = ResourceLimit::Create(kernel, "System");
|
system_resource_limit = ResourceLimit::Create(kernel);
|
||||||
|
|
||||||
// If setting the default system values fails, then something seriously wrong has occurred.
|
// If setting the default system values fails, then something seriously wrong has occurred.
|
||||||
ASSERT(system_resource_limit->SetLimitValue(ResourceType::PhysicalMemory, 0x200000000)
|
ASSERT(system_resource_limit->SetLimitValue(ResourceType::PhysicalMemory, 0x200000000)
|
||||||
|
|
|
@ -16,11 +16,8 @@ constexpr std::size_t ResourceTypeToIndex(ResourceType type) {
|
||||||
ResourceLimit::ResourceLimit(KernelCore& kernel) : Object{kernel} {}
|
ResourceLimit::ResourceLimit(KernelCore& kernel) : Object{kernel} {}
|
||||||
ResourceLimit::~ResourceLimit() = default;
|
ResourceLimit::~ResourceLimit() = default;
|
||||||
|
|
||||||
SharedPtr<ResourceLimit> ResourceLimit::Create(KernelCore& kernel, std::string name) {
|
SharedPtr<ResourceLimit> ResourceLimit::Create(KernelCore& kernel) {
|
||||||
SharedPtr<ResourceLimit> resource_limit(new ResourceLimit(kernel));
|
return new ResourceLimit(kernel);
|
||||||
|
|
||||||
resource_limit->name = std::move(name);
|
|
||||||
return resource_limit;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
s64 ResourceLimit::GetCurrentResourceValue(ResourceType resource) const {
|
s64 ResourceLimit::GetCurrentResourceValue(ResourceType resource) const {
|
||||||
|
|
|
@ -31,16 +31,14 @@ constexpr bool IsValidResourceType(ResourceType type) {
|
||||||
|
|
||||||
class ResourceLimit final : public Object {
|
class ResourceLimit final : public Object {
|
||||||
public:
|
public:
|
||||||
/**
|
/// Creates a resource limit object.
|
||||||
* Creates a resource limit object.
|
static SharedPtr<ResourceLimit> Create(KernelCore& kernel);
|
||||||
*/
|
|
||||||
static SharedPtr<ResourceLimit> Create(KernelCore& kernel, std::string name = "Unknown");
|
|
||||||
|
|
||||||
std::string GetTypeName() const override {
|
std::string GetTypeName() const override {
|
||||||
return "ResourceLimit";
|
return "ResourceLimit";
|
||||||
}
|
}
|
||||||
std::string GetName() const override {
|
std::string GetName() const override {
|
||||||
return name;
|
return GetTypeName();
|
||||||
}
|
}
|
||||||
|
|
||||||
static const HandleType HANDLE_TYPE = HandleType::ResourceLimit;
|
static const HandleType HANDLE_TYPE = HandleType::ResourceLimit;
|
||||||
|
@ -95,9 +93,6 @@ private:
|
||||||
ResourceArray limits{};
|
ResourceArray limits{};
|
||||||
/// Current resource limit values.
|
/// Current resource limit values.
|
||||||
ResourceArray values{};
|
ResourceArray values{};
|
||||||
|
|
||||||
/// Name of resource limit object.
|
|
||||||
std::string name;
|
|
||||||
};
|
};
|
||||||
|
|
||||||
} // namespace Kernel
|
} // namespace Kernel
|
||||||
|
|
Loading…
Reference in a new issue