Re: [PATCH 1/4] drm/amdkfd: Move a constant definition around
Pushed. Thanks. On 2019-02-15 4:14 p.m., Kuehling, Felix wrote: > The series is Reviewed-by: Felix Kuehling > > On 2019-02-14 6:45 p.m., Zhao, Yong wrote: >> The similar definitions should be consecutive. >> >> Change-Id: I936cf076363e641c60e0704d8405ae9493718e18 >> Signed-off-by: Yong Zhao P >> --- >>drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 11 ++- >>1 file changed, 6 insertions(+), 5 deletions(-) >> >> diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h >> b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h >> index 12b66330fc6d..e5ebcca7f031 100644 >> --- a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h >> +++ b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h >> @@ -97,17 +97,18 @@ >>#define KFD_CWSR_TBA_TMA_SIZE (PAGE_SIZE * 2) >>#define KFD_CWSR_TMA_OFFSET PAGE_SIZE >> >> +#define KFD_MAX_NUM_OF_QUEUES_PER_DEVICE\ >> +(KFD_MAX_NUM_OF_PROCESSES * \ >> +KFD_MAX_NUM_OF_QUEUES_PER_PROCESS) >> + >> +#define KFD_KERNEL_QUEUE_SIZE 2048 >> + >>/* >> * Kernel module parameter to specify maximum number of supported queues >> per >> * device >> */ >>extern int max_num_of_queues_per_device; >> >> -#define KFD_MAX_NUM_OF_QUEUES_PER_DEVICE\ >> -(KFD_MAX_NUM_OF_PROCESSES * \ >> -KFD_MAX_NUM_OF_QUEUES_PER_PROCESS) >> - >> -#define KFD_KERNEL_QUEUE_SIZE 2048 >> >>/* Kernel module parameter to specify the scheduling policy */ >>extern int sched_policy; ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
Re: [PATCH 1/4] drm/amdkfd: Move a constant definition around
The series is Reviewed-by: Felix Kuehling On 2019-02-14 6:45 p.m., Zhao, Yong wrote: > The similar definitions should be consecutive. > > Change-Id: I936cf076363e641c60e0704d8405ae9493718e18 > Signed-off-by: Yong Zhao > --- > drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 11 ++- > 1 file changed, 6 insertions(+), 5 deletions(-) > > diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h > b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h > index 12b66330fc6d..e5ebcca7f031 100644 > --- a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h > +++ b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h > @@ -97,17 +97,18 @@ > #define KFD_CWSR_TBA_TMA_SIZE (PAGE_SIZE * 2) > #define KFD_CWSR_TMA_OFFSET PAGE_SIZE > > +#define KFD_MAX_NUM_OF_QUEUES_PER_DEVICE \ > + (KFD_MAX_NUM_OF_PROCESSES * \ > + KFD_MAX_NUM_OF_QUEUES_PER_PROCESS) > + > +#define KFD_KERNEL_QUEUE_SIZE 2048 > + > /* >* Kernel module parameter to specify maximum number of supported queues per >* device >*/ > extern int max_num_of_queues_per_device; > > -#define KFD_MAX_NUM_OF_QUEUES_PER_DEVICE \ > - (KFD_MAX_NUM_OF_PROCESSES * \ > - KFD_MAX_NUM_OF_QUEUES_PER_PROCESS) > - > -#define KFD_KERNEL_QUEUE_SIZE 2048 > > /* Kernel module parameter to specify the scheduling policy */ > extern int sched_policy; ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
[PATCH 1/4] drm/amdkfd: Move a constant definition around
The similar definitions should be consecutive. Change-Id: I936cf076363e641c60e0704d8405ae9493718e18 Signed-off-by: Yong Zhao --- drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h index 12b66330fc6d..e5ebcca7f031 100644 --- a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h +++ b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h @@ -97,17 +97,18 @@ #define KFD_CWSR_TBA_TMA_SIZE (PAGE_SIZE * 2) #define KFD_CWSR_TMA_OFFSET PAGE_SIZE +#define KFD_MAX_NUM_OF_QUEUES_PER_DEVICE \ + (KFD_MAX_NUM_OF_PROCESSES * \ + KFD_MAX_NUM_OF_QUEUES_PER_PROCESS) + +#define KFD_KERNEL_QUEUE_SIZE 2048 + /* * Kernel module parameter to specify maximum number of supported queues per * device */ extern int max_num_of_queues_per_device; -#define KFD_MAX_NUM_OF_QUEUES_PER_DEVICE \ - (KFD_MAX_NUM_OF_PROCESSES * \ - KFD_MAX_NUM_OF_QUEUES_PER_PROCESS) - -#define KFD_KERNEL_QUEUE_SIZE 2048 /* Kernel module parameter to specify the scheduling policy */ extern int sched_policy; -- 2.17.1 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
[PATCH 1/4] drm/amdkfd: Move a constant definition around
The similar definitions should be consecutive. Change-Id: I936cf076363e641c60e0704d8405ae9493718e18 Signed-off-by: Yong Zhao --- drivers/gpu/drm/amd/amdkfd/kfd_priv.h | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h index 12b66330fc6d..e5ebcca7f031 100644 --- a/drivers/gpu/drm/amd/amdkfd/kfd_priv.h +++ b/drivers/gpu/drm/amd/amdkfd/kfd_priv.h @@ -97,17 +97,18 @@ #define KFD_CWSR_TBA_TMA_SIZE (PAGE_SIZE * 2) #define KFD_CWSR_TMA_OFFSET PAGE_SIZE +#define KFD_MAX_NUM_OF_QUEUES_PER_DEVICE \ + (KFD_MAX_NUM_OF_PROCESSES * \ + KFD_MAX_NUM_OF_QUEUES_PER_PROCESS) + +#define KFD_KERNEL_QUEUE_SIZE 2048 + /* * Kernel module parameter to specify maximum number of supported queues per * device */ extern int max_num_of_queues_per_device; -#define KFD_MAX_NUM_OF_QUEUES_PER_DEVICE \ - (KFD_MAX_NUM_OF_PROCESSES * \ - KFD_MAX_NUM_OF_QUEUES_PER_PROCESS) - -#define KFD_KERNEL_QUEUE_SIZE 2048 /* Kernel module parameter to specify the scheduling policy */ extern int sched_policy; -- 2.17.1 ___ amd-gfx mailing list amd-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
