Re: [PATCH 4/7] tests/acceptance: Sun4uMachine: Remove dependency to LinuxKernelTest

2021-05-24 Thread Willian Rampazzo
On Mon, May 24, 2021 at 3:30 PM Willian Rampazzo wrote: > > On Tue, May 4, 2021 at 1:01 PM Philippe Mathieu-Daudé wrote: > > > > Hi Wainer, > > > > On 5/4/21 12:43 AM, Wainer dos Santos Moschetta wrote: > > > The Sun4uMachine class inherit from LinuxKernelTest to effectively only > > > use > > >

Re: [PATCH 4/7] tests/acceptance: Sun4uMachine: Remove dependency to LinuxKernelTest

2021-05-24 Thread Willian Rampazzo
On Tue, May 4, 2021 at 1:01 PM Philippe Mathieu-Daudé wrote: > > Hi Wainer, > > On 5/4/21 12:43 AM, Wainer dos Santos Moschetta wrote: > > The Sun4uMachine class inherit from LinuxKernelTest to effectively only use > > the KERNEL_COMMON_COMMAND_LINE attribute. This change remove that unneeded > >

Re: [PATCH 4/7] tests/acceptance: Sun4uMachine: Remove dependency to LinuxKernelTest

2021-05-24 Thread Willian Rampazzo
On Mon, May 3, 2021 at 7:44 PM Wainer dos Santos Moschetta wrote: > > The Sun4uMachine class inherit from LinuxKernelTest to effectively only use > the KERNEL_COMMON_COMMAND_LINE attribute. This change remove that unneeded > dependency, making Sun4uMachine self-content. > > I took the occasion to

Re: [PATCH 4/7] tests/acceptance: Sun4uMachine: Remove dependency to LinuxKernelTest

2021-05-04 Thread Philippe Mathieu-Daudé
Hi Wainer, On 5/4/21 12:43 AM, Wainer dos Santos Moschetta wrote: > The Sun4uMachine class inherit from LinuxKernelTest to effectively only use > the KERNEL_COMMON_COMMAND_LINE attribute. This change remove that unneeded > dependency, making Sun4uMachine self-content. It is odd because the test b

[PATCH 4/7] tests/acceptance: Sun4uMachine: Remove dependency to LinuxKernelTest

2021-05-03 Thread Wainer dos Santos Moschetta
The Sun4uMachine class inherit from LinuxKernelTest to effectively only use the KERNEL_COMMON_COMMAND_LINE attribute. This change remove that unneeded dependency, making Sun4uMachine self-content. I took the occasion to delint the code: the unused os import was removed, imports were reordered, and