[PATCH 0/8] mini-os: some cleanup patches

Juergen Gross posted 8 patches 1 year, 11 months ago
Failed in applying to current master (apply log)
.gitignore                      |   2 +
Makefile                        |   9 +-
console/xenbus.c => consfront.c |  99 ++++---
console.c                       | 415 ++++++++++++++++++++++++++
console/console.c               | 177 -----------
console/console.h               |   2 -
console/xencons_ring.c          | 238 ---------------
include/console.h               |   1 +
xenbus/xenbus.c => xenbus.c     | 510 +++++++++++++++++++-------------
9 files changed, 778 insertions(+), 675 deletions(-)
rename console/xenbus.c => consfront.c (78%)
create mode 100644 console.c
delete mode 100644 console/console.c
delete mode 100644 console/console.h
delete mode 100644 console/xencons_ring.c
rename xenbus/xenbus.c => xenbus.c (71%)
[PATCH 0/8] mini-os: some cleanup patches
Posted by Juergen Gross 1 year, 11 months ago
Do some cleanups.

No functional change intended, apart from whitespace changes only
minor modifications making code easier to read.

Juergen Gross (8):
  mini-os: drop xenbus directory
  mini-os: apply coding style to xenbus.c
  mini-os: eliminate console/console.h
  mini-os: rename console/xenbus.c to consfront.c
  mini-os: apply coding style to consfront.c
  mini-os: eliminate console directory
  mini-os: apply coding style to console.c
  mini-os: add mini-os-debug[.gz] to .gitignore

 .gitignore                      |   2 +
 Makefile                        |   9 +-
 console/xenbus.c => consfront.c |  99 ++++---
 console.c                       | 415 ++++++++++++++++++++++++++
 console/console.c               | 177 -----------
 console/console.h               |   2 -
 console/xencons_ring.c          | 238 ---------------
 include/console.h               |   1 +
 xenbus/xenbus.c => xenbus.c     | 510 +++++++++++++++++++-------------
 9 files changed, 778 insertions(+), 675 deletions(-)
 rename console/xenbus.c => consfront.c (78%)
 create mode 100644 console.c
 delete mode 100644 console/console.c
 delete mode 100644 console/console.h
 delete mode 100644 console/xencons_ring.c
 rename xenbus/xenbus.c => xenbus.c (71%)

-- 
2.35.3
Re: [PATCH 0/8] mini-os: some cleanup patches
Posted by Samuel Thibault 1 year, 11 months ago
Hello,

Juergen Gross, le lun. 20 juin 2022 09:38:12 +0200, a ecrit:
> Do some cleanups.
> 
> No functional change intended, apart from whitespace changes only
> minor modifications making code easier to read.
> 
> Juergen Gross (8):
>   mini-os: drop xenbus directory
>   mini-os: apply coding style to xenbus.c
>   mini-os: eliminate console/console.h
>   mini-os: rename console/xenbus.c to consfront.c
>   mini-os: apply coding style to consfront.c
>   mini-os: eliminate console directory
>   mini-os: apply coding style to console.c
>   mini-os: add mini-os-debug[.gz] to .gitignore

For the whole series:

Reviewed-by: Samuel Thibault <samuel.thibault@ens-lyon.org>

Thanks!

>  .gitignore                      |   2 +
>  Makefile                        |   9 +-
>  console/xenbus.c => consfront.c |  99 ++++---
>  console.c                       | 415 ++++++++++++++++++++++++++
>  console/console.c               | 177 -----------
>  console/console.h               |   2 -
>  console/xencons_ring.c          | 238 ---------------
>  include/console.h               |   1 +
>  xenbus/xenbus.c => xenbus.c     | 510 +++++++++++++++++++-------------
>  9 files changed, 778 insertions(+), 675 deletions(-)
>  rename console/xenbus.c => consfront.c (78%)
>  create mode 100644 console.c
>  delete mode 100644 console/console.c
>  delete mode 100644 console/console.h
>  delete mode 100644 console/xencons_ring.c
>  rename xenbus/xenbus.c => xenbus.c (71%)
> 
> -- 
> 2.35.3
>
Re: [PATCH 0/8] mini-os: some cleanup patches
Posted by Juergen Gross 1 year, 10 months ago
On 20.06.22 09:38, Juergen Gross wrote:
> Do some cleanups.
> 
> No functional change intended, apart from whitespace changes only
> minor modifications making code easier to read.
> 
> Juergen Gross (8):
>    mini-os: drop xenbus directory
>    mini-os: apply coding style to xenbus.c
>    mini-os: eliminate console/console.h
>    mini-os: rename console/xenbus.c to consfront.c
>    mini-os: apply coding style to consfront.c
>    mini-os: eliminate console directory
>    mini-os: apply coding style to console.c
>    mini-os: add mini-os-debug[.gz] to .gitignore
> 
>   .gitignore                      |   2 +
>   Makefile                        |   9 +-
>   console/xenbus.c => consfront.c |  99 ++++---
>   console.c                       | 415 ++++++++++++++++++++++++++
>   console/console.c               | 177 -----------
>   console/console.h               |   2 -
>   console/xencons_ring.c          | 238 ---------------
>   include/console.h               |   1 +
>   xenbus/xenbus.c => xenbus.c     | 510 +++++++++++++++++++-------------
>   9 files changed, 778 insertions(+), 675 deletions(-)
>   rename console/xenbus.c => consfront.c (78%)
>   create mode 100644 console.c
>   delete mode 100644 console/console.c
>   delete mode 100644 console/console.h
>   delete mode 100644 console/xencons_ring.c
>   rename xenbus/xenbus.c => xenbus.c (71%)
> 

Could someone please commit this series? Samuel gave his R-b nearly 2 weeks
ago.


Juergen
Re: [PATCH 0/8] mini-os: some cleanup patches
Posted by Julien Grall 1 year, 10 months ago
Hi Juergen,

On 04/07/2022 09:37, Juergen Gross wrote:
> On 20.06.22 09:38, Juergen Gross wrote:
>> Do some cleanups.
>>
>> No functional change intended, apart from whitespace changes only
>> minor modifications making code easier to read.
>>
>> Juergen Gross (8):
>>    mini-os: drop xenbus directory
>>    mini-os: apply coding style to xenbus.c
>>    mini-os: eliminate console/console.h
>>    mini-os: rename console/xenbus.c to consfront.c
>>    mini-os: apply coding style to consfront.c
>>    mini-os: eliminate console directory
>>    mini-os: apply coding style to console.c
>>    mini-os: add mini-os-debug[.gz] to .gitignore
>>
>>   .gitignore                      |   2 +
>>   Makefile                        |   9 +-
>>   console/xenbus.c => consfront.c |  99 ++++---
>>   console.c                       | 415 ++++++++++++++++++++++++++
>>   console/console.c               | 177 -----------
>>   console/console.h               |   2 -
>>   console/xencons_ring.c          | 238 ---------------
>>   include/console.h               |   1 +
>>   xenbus/xenbus.c => xenbus.c     | 510 +++++++++++++++++++-------------
>>   9 files changed, 778 insertions(+), 675 deletions(-)
>>   rename console/xenbus.c => consfront.c (78%)
>>   create mode 100644 console.c
>>   delete mode 100644 console/console.c
>>   delete mode 100644 console/console.h
>>   delete mode 100644 console/xencons_ring.c
>>   rename xenbus/xenbus.c => xenbus.c (71%)
>>
> 
> Could someone please commit this series? Samuel gave his R-b nearly 2 weeks
> ago.

I will commit this series and the other you pointed out.

Cheers,

-- 
Julien Grall

Re: [PATCH 0/8] mini-os: some cleanup patches
Posted by Julien Grall 1 year, 10 months ago

On 04/07/2022 09:39, Julien Grall wrote:
> Hi Juergen,
> 
> On 04/07/2022 09:37, Juergen Gross wrote:
>> On 20.06.22 09:38, Juergen Gross wrote:
>>> Do some cleanups.
>>>
>>> No functional change intended, apart from whitespace changes only
>>> minor modifications making code easier to read.
>>>
>>> Juergen Gross (8):
>>>    mini-os: drop xenbus directory
>>>    mini-os: apply coding style to xenbus.c
>>>    mini-os: eliminate console/console.h
>>>    mini-os: rename console/xenbus.c to consfront.c
>>>    mini-os: apply coding style to consfront.c
>>>    mini-os: eliminate console directory
>>>    mini-os: apply coding style to console.c
>>>    mini-os: add mini-os-debug[.gz] to .gitignore
>>>
>>>   .gitignore                      |   2 +
>>>   Makefile                        |   9 +-
>>>   console/xenbus.c => consfront.c |  99 ++++---
>>>   console.c                       | 415 ++++++++++++++++++++++++++
>>>   console/console.c               | 177 -----------
>>>   console/console.h               |   2 -
>>>   console/xencons_ring.c          | 238 ---------------
>>>   include/console.h               |   1 +
>>>   xenbus/xenbus.c => xenbus.c     | 510 +++++++++++++++++++-------------
>>>   9 files changed, 778 insertions(+), 675 deletions(-)
>>>   rename console/xenbus.c => consfront.c (78%)
>>>   create mode 100644 console.c
>>>   delete mode 100644 console/console.c
>>>   delete mode 100644 console/console.h
>>>   delete mode 100644 console/xencons_ring.c
>>>   rename xenbus/xenbus.c => xenbus.c (71%)
>>>
>>
>> Could someone please commit this series? Samuel gave his R-b nearly 2 
>> weeks
>> ago.
> 
> I will commit this series and the other you pointed out.

Done.

Cheers,

-- 
Julien Grall