diff mbox series

[qemu] spapr: Fix typo in the patb_entry comment

Message ID 20210225032335.64245-1-aik@ozlabs.ru
State New
Headers show
Series [qemu] spapr: Fix typo in the patb_entry comment | expand

Commit Message

Alexey Kardashevskiy Feb. 25, 2021, 3:23 a.m. UTC
There is no H_REGISTER_PROCESS_TABLE, it is H_REGISTER_PROC_TBL handler
for which is still called h_register_process_table() though.

Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
---
 include/hw/ppc/spapr.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Daniel Henrique Barboza Feb. 25, 2021, 2:37 p.m. UTC | #1
On 2/25/21 12:23 AM, Alexey Kardashevskiy wrote:
> There is no H_REGISTER_PROCESS_TABLE, it is H_REGISTER_PROC_TBL handler
> for which is still called h_register_process_table() though.
> 
> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
> ---

Reviewed-by: Daniel Henrique Barboza <danielhb413@gmail.com>

>   include/hw/ppc/spapr.h | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/hw/ppc/spapr.h b/include/hw/ppc/spapr.h
> index ccbeeca1de84..8dceace4b442 100644
> --- a/include/hw/ppc/spapr.h
> +++ b/include/hw/ppc/spapr.h
> @@ -168,7 +168,7 @@ struct SpaprMachineState {
>       SpaprResizeHpt resize_hpt;
>       void *htab;
>       uint32_t htab_shift;
> -    uint64_t patb_entry; /* Process tbl registed in H_REGISTER_PROCESS_TABLE */
> +    uint64_t patb_entry; /* Process tbl registed in H_REGISTER_PROC_TBL */
>       SpaprPendingHpt *pending_hpt; /* in-progress resize */
>   
>       hwaddr rma_size;
>
David Gibson March 22, 2021, 5:44 a.m. UTC | #2
On Thu, Feb 25, 2021 at 02:23:35PM +1100, Alexey Kardashevskiy wrote:
> There is no H_REGISTER_PROCESS_TABLE, it is H_REGISTER_PROC_TBL handler
> for which is still called h_register_process_table() though.
> 
> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>

Applied to ppc-for-6.0.

In future, best to CC me directly, since I only occasionally peruse
the lists.

> ---
>  include/hw/ppc/spapr.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/hw/ppc/spapr.h b/include/hw/ppc/spapr.h
> index ccbeeca1de84..8dceace4b442 100644
> --- a/include/hw/ppc/spapr.h
> +++ b/include/hw/ppc/spapr.h
> @@ -168,7 +168,7 @@ struct SpaprMachineState {
>      SpaprResizeHpt resize_hpt;
>      void *htab;
>      uint32_t htab_shift;
> -    uint64_t patb_entry; /* Process tbl registed in H_REGISTER_PROCESS_TABLE */
> +    uint64_t patb_entry; /* Process tbl registed in H_REGISTER_PROC_TBL */
>      SpaprPendingHpt *pending_hpt; /* in-progress resize */
>  
>      hwaddr rma_size;
Alexey Kardashevskiy March 24, 2021, 4:30 a.m. UTC | #3
On 22/03/2021 16:44, David Gibson wrote:
> On Thu, Feb 25, 2021 at 02:23:35PM +1100, Alexey Kardashevskiy wrote:
>> There is no H_REGISTER_PROCESS_TABLE, it is H_REGISTER_PROC_TBL handler
>> for which is still called h_register_process_table() though.
>>
>> Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
> 
> Applied to ppc-for-6.0.
> 
> In future, best to CC me directly, since I only occasionally peruse
> the lists.



even the qemu-ppc@nongnu.org list? okay :)



>> ---
>>   include/hw/ppc/spapr.h | 2 +-
>>   1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/include/hw/ppc/spapr.h b/include/hw/ppc/spapr.h
>> index ccbeeca1de84..8dceace4b442 100644
>> --- a/include/hw/ppc/spapr.h
>> +++ b/include/hw/ppc/spapr.h
>> @@ -168,7 +168,7 @@ struct SpaprMachineState {
>>       SpaprResizeHpt resize_hpt;
>>       void *htab;
>>       uint32_t htab_shift;
>> -    uint64_t patb_entry; /* Process tbl registed in H_REGISTER_PROCESS_TABLE */
>> +    uint64_t patb_entry; /* Process tbl registed in H_REGISTER_PROC_TBL */
>>       SpaprPendingHpt *pending_hpt; /* in-progress resize */
>>   
>>       hwaddr rma_size;
>
David Gibson March 24, 2021, 6:15 a.m. UTC | #4
On Wed, Mar 24, 2021 at 03:30:57PM +1100, Alexey Kardashevskiy wrote:
> 
> 
> On 22/03/2021 16:44, David Gibson wrote:
> > On Thu, Feb 25, 2021 at 02:23:35PM +1100, Alexey Kardashevskiy wrote:
> > > There is no H_REGISTER_PROCESS_TABLE, it is H_REGISTER_PROC_TBL handler
> > > for which is still called h_register_process_table() though.
> > > 
> > > Signed-off-by: Alexey Kardashevskiy <aik@ozlabs.ru>
> > 
> > Applied to ppc-for-6.0.
> > 
> > In future, best to CC me directly, since I only occasionally peruse
> > the lists.
> 
> 
> 
> even the qemu-ppc@nongnu.org list? okay :)

Yes.  I do try to read qemu-ppc@nongnu.org weekly, but it's still easy
for me to miss things.
diff mbox series

Patch

diff --git a/include/hw/ppc/spapr.h b/include/hw/ppc/spapr.h
index ccbeeca1de84..8dceace4b442 100644
--- a/include/hw/ppc/spapr.h
+++ b/include/hw/ppc/spapr.h
@@ -168,7 +168,7 @@  struct SpaprMachineState {
     SpaprResizeHpt resize_hpt;
     void *htab;
     uint32_t htab_shift;
-    uint64_t patb_entry; /* Process tbl registed in H_REGISTER_PROCESS_TABLE */
+    uint64_t patb_entry; /* Process tbl registed in H_REGISTER_PROC_TBL */
     SpaprPendingHpt *pending_hpt; /* in-progress resize */
 
     hwaddr rma_size;