Thiemo Seufer wrote: > Jason Wessel wrote: > >> The attached patch adds loadvm and savevm support to the e100 driver. >> >> One note about this patch is that the image is not portable across >> big/little endian hosts. Since it already appeared that the save/load >> images in general are not portable between a big/little endian hosts I >> just took the short cut of pushing the struct members that were not >> being saved directly into the save buffer. >> > > Looks like a bug to me. > > >> If it is the case that the >> save/load images are supposed to be endian agnostic, there is definitely >> some more work that needs to be done in all the simulated hw drivers. >> >> Signed-off-by: Jason Wessel >> > > Patch is missing. > > > Thiemo > Since the first patch was missing, I created a new one which does not have any endian issues. In this patch there is a #if 0 which matches the member of the struct that has the #if 0 in it. Once again I tested it to confirm it works with a single or multiple enet cards of different types. Originally I thought I had seen a place in the code where there was a put buffer on a series of long values, but I can no longer find the reference to it. In the original patch I had just saved the structs outright via the put_buffer which clearly would not have been endian agnostic. The patch attached here fixes that. Signed-off-by: Jason Wessel Jason.